merge from open-source master

Change-Id: I0745cd709b0a481f12c29cf8858b51134a1265d7

Merge "Unit test fixes"

Unit test fixes

First, the AddonsListFetcherTest#testLoadSample_1 was failing on the
Mac, because the source code contained non-ASCII unicode characters,
and the encoding differs between Windows, Linux and Mac -- and on the
Mac the characters were garbage (not the intended Japanese
characters). Fixed by using unicode escape sequences to define the
expected output instead.

Second, rename the AbstractLayoutTest to LayoutTestBase; the test
runner (for plugins, not unit tests) was assuming this was a test case
(even though it's an abstract class!) so it was emitting a warning
about the class containing no test cases. This is simply a base class
for all the individual layout tests defining a bunch of useful
inherited shared behavior, so rename to avoid the warning.

Change-Id: I6e29316c5644db35051218aef8b99ac41ab0cda4

merge from open-source master

Change-Id: I66a3b8c183bee0f8efc86f53a7dfd9004ed75867

Merge "Replace IPath.makeRelativeTo call with reflection"

Replace IPath.makeRelativeTo call with reflection

I broke the build because on Eclipse 3.4 there is no makeRelativeTo
method on IPath. This replaces that call with reflection. This won't
work on 3.4, but it's a quick fix for the broken build.

Change-Id: Ia917cf5a745f76bbb96f59fae93e2a4fc4f5f900

merge from open-source master

Change-Id: I0a6441de70a779781b68c12433b2390e600c5d2e

Merge "Open included layout on double click"

merge from open-source master

Change-Id: If98e561ae25793be21b97ef3f16cf0f6f854d6f4

Merge "Fix nine patch size."

Merge "Remove some asserts."

Open included layout on double click

On double click, if the clicked area originates from an <include>'ed
XML file, open the included layout in the editor. Also opens
files in the @android: namespace if available, for example if you
inculde @android:layout/select_dialog_multichoice.

Change-Id: I215c411257717f7b97e7b0ee1d5498c318cdb04d

Remove some asserts.

Change-Id: I8458dc9237506c542c766e6119261ae4e13b453b

merge from open-source master

Change-Id: I559b431e4b45fd51fe5cd230e2b8206469cbf4ca

Merge "ADT: ensure <application> is last in manifest."

ADT: ensure <application> is last in manifest.

ADT descriptors have no notion of XML ordering.
We modify the concept of "mandatory" descriptor to
have normal mandatory descriptors versus "mandatory last" ones.
("mandatory" elements are virtual reserved slots that are
 always present in the UiModel even if they have no real
 counterpart in the XML model. These elements ensure they can
 be manipulated in the UI even before their XML is created.)

Then we try our best to ensure the "last" ones remain at the
end of the UiNode model. There are 2 cases:
- in UiElementNode, when constructing the UINode model when
  parsing an existing XML model, we try to reorder the nodes
  at the end if possible.
- in UiActions, when adding a new element to the UiModel, we
  try to place it before the first "mandatory last" element.

SDK Bug 3197310

Change-Id: I6a7d9502a95ebe92ff82e07f3f3249a0d25c2154

Fix nine patch size.

Since we moved the 9-patch chunk out of the NinePatch, the image
content portion is not in our control (it's owned by the default
implementation of NinePatchDrawable through an Android Bitmap),
and therefore should not include the control outer line.

This changeset makes the bitmap contains only the extracted
content and makes the computation of the patches take into account that
the draw method will be applied on a bitmap that is only
the content without the control line.
Basically the computation of the patches/fixed rectangle exclude the
outer control line.

Change-Id: I29c2224ee2311962b30d1fb155b85d5731d4e6ec

merge from open-source master

Change-Id: I4b261c8bc0635174fb125ee68590b6b9f4c7b5db

Merge "Refactor selection painting to fix highlighting of included views"

merge from open-source master

Change-Id: Ia341d955de13d0c1e7dca0a55173345b11cac745

Merge "Move the ninepatch info into its own class."

merge from open-source master

Change-Id: If699b3e30843890c08e9f5a31fb5c1305cc6ae9e

Merge "Fix live manipulation of <include> elements"

Merge "Fix unit test log usage"

merge from open-source master

Change-Id: I5cf9142f6fa6b876fcdc01446dda8c8d98fed25c

Merge "Fix -v logging level flag."

Fix -v logging level flag.

- Rather than ignoring the flag, actually make use of it.
- Change the default logging level to SEVERE
- Change the "command slow" exception print from ddmlib to INFO

Change-Id: Iade4700b32ed7b4a55bcd3336c74b2127693bebe

merge from open-source master

Change-Id: I944fbeab95d750bf30c224c526027534e2b7d611

Merge "Fix URL for eclipse download (for auto build plugin)"

Fix URL for eclipse download (for auto build plugin)

Change-Id: I1eba4b240869a222a0eb1c8676cd120acfb0fba4

merge from open-source master

Change-Id: Ied18408d470e506c4107941980994d3f1a1ce013

Merge "Fix potential NPE when closing Eclipse/ADT"

Fix potential NPE when closing Eclipse/ADT

Seems like it was trying to finish a build and closing
the plugin ADT at the same time. How's that even possible?
In any case I've never seen that before and I was in
debug mode, so I bet it's a rare race condition.

SDK Bug 3222178

Change-Id: I28a9bdfcbe56e2a6b73eafb5cadea611c5177f7f

Move the ninepatch info into its own class.

The chunk is what is used separately by the default NinePatch class,
through serialization to make a byte[] out of it.

Since we are moving away from replacement classes to delegate, splitting
the chunk info away from the image allows us to use the normal
NinePatch(Drawable).

Change-Id: Ifd86dc2aa9b485d0e97a2d4a248621cfcddda9ab

Fix unit test log usage

There were some test failures because unit tests were relying on
AdtPlugin's logger, which appears to be null during unit test
runs. Use a new test logger instead, which fails the current test if
anyone logs an error message, and dumps warnings to standard error and
prints to standard output.

Change-Id: I8e52fed554d49e98f7d6c8990d41831998f44640

Fix live manipulation of <include> elements

This changeset fixes bugs related to dragging included views around in
the canvas:

First, the <include> tag would get rewritten to <null>. This happened
because the layout editor treats the FQCN and the XML node name as
equivalent, but in the case of the include tag the FQCN was set to
null instead of "include".

Second, the "layout" attribute would not get copied because it is NOT
in the Android namespace, and the code to copy attributes ended up
with an empty-string namespace which was not handled correctly.

Third, when copied the "layout" attribute would end up with the
namespace "ns:" because the code to create attribute nodes always
created namespaced attribute nodes rather than a plain attribute node
when the namespace is null.

Change-Id: Ibd34212517615aa8ec79abe14bca765cdca525f6

Refactor selection painting to fix highlighting of included views

This changeset moves the painting of selection bounds and selection
hints out of the view rules and into the core IDE. The reason for this
is that the visual appearance of the selection shouldn't be up to each
rule; for one thing the selection highlight should be consistent and
not vary from view to view (and in fact there was only a single
implementation of the paint selection method among the view rules),
and for another the view rules are in theory sharable among IDEs
whereas the selection appearance is going to be IDE specific. There
was also painting of "hints" in the RelativeLayout; rather than having
the visual appearance of this dictated by the rule, this is also moved
into the IDE such that the rules only provide the hint text and the
hints are displayed by the IDE itself.

The above refactoring also fixes selection feedback for <include>'ed
views, which were not visually selectable because there was no
corresponding ViewRule, so nobody to paint selection. With these
changes selection painting is now independent of the rules.

Change-Id: I22dd926102128634a443b8bafb54d4764f1eda41

merge from open-source master

Change-Id: I9233097052d1559e620c54a6450eb59e5f6d70e1

Merge "Fix the other SwtUtilsTest methods under Windows."

Fix the other SwtUtilsTest methods under Windows.

Fixes the following issues:
- All methods must use the palette mask to compare RGB values.
- Use a non-trivial default color for testing. The previous
  choice of 0xFF00FF00 would not allow detecting RGB <-> BGR
  mixups and the 0xFF alpha mask prevented from noticing the
  fill operation was pre-multiplying the colors.
- Assert the converted image as the expected transparency type.

Change-Id: I1dc7518a7d7000bce75e1c4166a9921ba40a43cb

merge from open-source master

Change-Id: I96eb6b9ea27b3ce3730f6f962c6abd0853c8fc7a

Merge "Fix SwtUtils conversion for no-alpha case."

Fix SwtUtils conversion for no-alpha case.

Change-Id: I275b526670ca6e60cfc4f8749631dff2ef240b96

merge from open-source master

Change-Id: Ib80d4267e4667901e852a03da42a7f6cc418f116

Merge "GLE: Remove obsolete IGraphicalLayoutEditor interface."

GLE: Remove obsolete IGraphicalLayoutEditor interface.

Change-Id: I96f929ce9386e88f80d15195b2ff6a498e374554

merge from open-source master

Change-Id: I68c1ab2f7dc006fdb91bbfdd7c6a48107db4022c

Merge "ADT: support for editing embedded layouts."

ADT: support for editing embedded layouts.

This is a very first rough changeset. The context in which the layout
is edited is hardcoded until we have the UI for it.

The basics are the following:
- if there's a context, don't pass the parser on top of the model to
  the layout lib.
- instead create a KXml based parser that will return the model-based
parser when queried for a custom parser (if the layout name matches the
file being edited).

Change-Id: I2d106def9c64e82ab50c8fae96cb050338422a85

merge from open-source master

Change-Id: I4d2189f435ea2ef7e66f4c78d703329eed1940bf

Merge "ADT/Layoutlib: update API to control how layout expands."

ADT/Layoutlib: update API to control how layout expands.

Previous API was a single on/off to let the layouts expand at
render time depending on how much space they needed.

The new API can now control expansion is horizontal and/or
vertical (or not at all)

Basic implementation in the editor, with a manual detect of
"ScrollView" as top element. We should make the ViewRule handle
this somehow.

Change-Id: Idc503bc0d1d3df98fbf01cc84625952ca55a8afb

merge from open-source master

Change-Id: I2a6be625ee3c3e2c7963dcb787506e0076397e40

Merge "Add per-view custom initialization logic"

Add per-view custom initialization logic

This changeset adds support for adding custom-logic to initialize
views, both to add children and default attributes and to customize
layout attributes when added to a new parent.

First, there is a new "onCreate" hook which is called to notify a view
rule that an instance of its corresponding view has been created. This
lets the ViewRule perform custom initialization of the object. The
ViewRule is told what type of insertion occurred, such that it can
distinguish between a newly created view, a view that is the result of
a copy/paste, and a view that is part of a move operation.

The changeset adds a number of new ViewRules which take advantage of
this:
- A TabHost rule creates the various skeleton children that are
  required, such as a TabWidget child with id @android:id/tabs and a
  FrameLayout child with id @android:id/tabcontent
- A DialerFilter rule creates the mandatory EditText children ("hint"
  and "primary")
- The HorizontalScrollView rule creates a horizontal LinearLayout child
- The ImageButton and ImageViewButtons initialize the "src" attribute
  to a sample image
- The MapViewRule initializes the apiKey attribute

In addition, views are also notified when a new view is added as a
child, such that they can perform additional customizations, in the
form of an "onInsert" event.

The most important application of this is LinearLayoutRule, which uses
this to set reasonable defaults for the layout_width and layout_height
parameters. It uses metadata (which is currently built into ADT but
would ideally migrate into our XML config files) to determine whether
a given child prefers to grow horizontally, grow vertically, both, or
neither, depending on the surrounding parent context.  For example, an
EditText will default to filling the parent width if it is in a
vertical LinearLayout, but it will not grow vertically in a horizontal
linear layout. And so on. Various other rules also use the onInsert
event to tweak children attributes. A ScrollView will for example
always initialize its single child to match parent.

Views can now also add plain menu items into the context menu, and the
TableViewRule adds one such action: "Add Row", which appends a new row
into the table.

The Palette Preview code also invokes these creation hooks, such that
if you for example drag a DialerFilter it can properly render since
the mandatory children are created up front. This required various
changes to the preview code to be able to handle XML edits by the
rules.

Finally, this changeset includes various other misc changes that I
performed at the same time:
- Removed SWT dependency from the ViewRule classes (SWT Rectangle use
  in Rect)
- Fixed AbsoluteLayout unit test (issue 3203560)
- Fixed positioning of the preview outline in LinearLayout when only
   one of the dimensions are clipped due to a smaller target layout

Change-Id: I5956fe4e7a31a20b8dd2f9d9b0c1f90e2f75d68a

merge from open-source master

Change-Id: I3c993a285154366b5d2dc9893b90df31db0bb510

Merge "Fix PDT. Extension class of toolsLocator is now ToolsLocator"

Fix PDT. Extension class of toolsLocator is now ToolsLocator

Change-Id: Ic2b65e07755339fc277392e4acdfe1e880a83a3d

merge from open-source master

Change-Id: I33b2e6584135c9d6ce875cf27ca66a3dfed640fa

Merge "Cleanup and comment the refresh code in GLE2."

Merge "Cleanup GRE preload."

merge from open-source master

Change-Id: If1ac02fd868400b7b2dfafc200c17b43f42c6aef

Merge "Remove non-api references from AttrsXmlParser."

merge from open-source master

Change-Id: Idf12fbcb5508ca53047fcde3c35a96f9a971d305

Merge "Fixed a problem that physical address is inconsistent."

Cleanup and comment the refresh code in GLE2.

Change-Id: I95a922c463992600edfc8c25edfc688dc38f79bd

Cleanup GRE preload.

GRE used to preload the groovy BaseView rule to improve
the first selection speed. We don't need that now that we
switched to a non-groovy engine.

Change-Id: Ib29272f28df6285137d62c45441cd9ac72efd10f

Remove non-api references from AttrsXmlParser.

Change-Id: Ic139e6f942e835dda4b7ef0303556aef014a60d3

merge from open-source master

Change-Id: I41e43cb191b6cfc8177c40cb2bb191e110ad0d53

Merge "ADT/GLE: call LayoutScene.dispose() when needed."

ADT/GLE: call LayoutScene.dispose() when needed.

Change-Id: I95819d95827164d3e57dc179d3f321d26a649599

merge from open-source master

Change-Id: I39d75545b18ad12e1f1926171a2e7962e157974a

Merge "Remove AdtPlugin dependency from AttrsXmlParser."

Remove AdtPlugin dependency from AttrsXmlParser.

The AdtPluin was used just for logging. Instead the
AttrsXmlParser takes an ILogger (AdtPlug implements
ILogger and can be used directly in unit tests too).

For unit tests there is a new StdSdkLog convenience
class that prints to stdout/stderr (formerly MockStdLogger
from the Sdk Manager was doing that.)

Change-Id: I658af61d04efb19ad6e3bf9c0bf471452372885a

merge from open-source master

Change-Id: Idc0001767c65246da61f91696675373a16e81c79

Merge "Update ADT changelog with current changes."

Update ADT changelog with current changes.

Change-Id: Id02026a70882d260855b283693ab93c9810fa2e3

merge from open-source master

Change-Id: I4dd20b129dca8473d5da7f6b84d90f8674a6bbb1

Merge "ADT/GLE2: Add callback before the rendering target changes."

ADT/GLE2: Add callback before the rendering target changes.

Change-Id: If02fd18c39b06042615735b55190e9f03fef5b19

merge from open-source master

Change-Id: Ia89bfb0ded53d625e9f423215bf0308ff4d69cc3

merge from tools_r8

Change-Id: I7e81a594ba2f83e787c288709d14d2b97d89074e

Merge "Fix HV.bat for swt layout issue." into tools_r8

merge from open-source master

Change-Id: Ide21ec65f1912807b11d016579747f647aa90950

Merge "ADT/GLE: Platform selection when rendering layouts."

Fix HV.bat for swt layout issue.

SDK Bug 3185449

Change-Id: I88dfaeb287fdef5cf82a0bc3da286c6c4493f910

merge from open-source master

Change-Id: Ia5db9ed687e683df73ec341158096726671a6752

Merge "ADT: Extract AttrsXmlParser in com.android.ide.common"

ADT/GLE: Platform selection when rendering layouts.

Change-Id: Iabbd49cdd52419b947b83fb84f9fb3a5d4576471

ADT: Extract AttrsXmlParser in com.android.ide.common

This is a pure-refactoring CL that moves AttrsXmlParser
into an ide.common.resources.platform package. In a next CL,
the parser should be cleanup to remove some references to
external classes (e.g. adtplugin is only used for logging so
it will become an ILog reference.)

The goal of the resources.platform package is to allow other IDEs
to parse the manifest schema. An utility class would be provided
here that would then be used by AndroidTargetParser. The rest of the
data parsing (widgets, resources, etc.) is a non-goal. Maybe later.

Change-Id: I4fb8eb5d168b75ef8bfab57d0b2883aea85b6167

merge from open-source master

Change-Id: I920548d1aa7485e6a154255ad0ee9bc7e62c93ad

merge from tools_r8

Change-Id: I8bb7cf608bda93ea3f13182a5dfe9f88dcc4bbb1

Merge "ADT tools location: traceview is a .bat" into tools_r8

ADT tools location: traceview is a .bat

This prevents adb from starting under windows, since
adt is trying to verify the presence of traceview.exe
when it's actually traceview.bat.

Change-Id: I259d90adb17b0b91beb114319cf88392812fc264

merge from open-source master

Change-Id: I717ec549d6917bc238a3517c091cfcca0609ef8c

Merge "Add preferences to the list of not proguarded classes."

merge from open-source master

Change-Id: Ic0f26cdf72551a108fee8877263346a8c91eab96

Merge "Improvements to LinearLayout feedback"

Improvements to LinearLayout feedback

When you have a small and empty linear layout, and you drag something
larger (such as a button) into it, the drop feedback is a bit
confusing: The drop feedback rectangle is larger than the linear
layout, so the bounds are outside the layout. This changeset addresses
this by forcing the bounds of the drop preview to be at most the
dimensions of the LinearLayout itself.

Second, the fix I applied last week to show the last insert position,
did not work in all cases - in particular when the drag originates
outside the canvas itself. To determine if we are inserting at the
last position, look at the number of target node children, rather than
the number of potential insert positions, since in some cases the
number of insert positions will be smaller than the number of
children.

Finally, there was a theoretical bug that if one of the dragged
elements did not non-zero bounds, then the insert position would be
wrong. This is also fixed by this changeset.

Change-Id: Ia30e99f7a3aa45b8091855b69aaef86ec3699405

Add preferences to the list of not proguarded classes.

Change-Id: Ib7db435bb640b3a3feaa0534ccc77b6f9eda0800