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

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

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

merge from open-source master

Change-Id: Ife73a3a44823869ddc2536650aa8f68bbb1f5f45

Merge "Exclude BackupAgentHelper classes from Proguard."

Merge "Add drop shadow to dragged items"

Merge "Update the Eclipe plug-ins/features version number."

Update the Eclipe plug-ins/features version number.

Change-Id: Ib5f49d91506de5e218d80af1d05d06ba49155cb6

Add drop shadow to dragged items

Add in a drop shadow to items dragged from the palette. The primary
reason we need this is that some views (in some themes) only render
light content on top of a transparent background, which makes them
nearly impossible to see on a gray background (e.g. over the palette,
when you initiate the drag). The drop shadow helps add contrast. It
also makes the drag look better since it helps the drag preview image
stand out from the background and visually appear to be lifted on top
of it.

Since we're doing more image processing now, I moved the existing
non-SWT image manipulation methods into a separate ImageUtils class.

This changeset also fixes the way we pick the color to crop out when
using older layout libraries (without transparency support). We now
pick a pixel closer to the bounds of the rendered view such that we
for example can handle themes like the dialog theme.

Change-Id: Ifcbb840ef715bf26a9609f6996a4f136de5ca754

merge from open-source master

Change-Id: I73a17a632843a059725334b415445cc892908dca

merge from tools_r8

Change-Id: I7eb330edd8f5fdb70b54ad61e773071a46131984

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

merge from open-source master

Change-Id: I3bdc35f41c5b72e658cf8271edc0df8ffb1a479f

Merge "Use env var com.android.ddms.bindir for ddms."

merge from open-source master

Change-Id: I393184c1706ee9ac5aaac356116385f179dd6df8

Merge "Fix drag image mouse coordinate handling"

Exclude BackupAgentHelper classes from Proguard.

To use the Backup API in Froyo, an app needs to typically
extend this helper class and the class is only referenced
from the AndroidManifest.xml, so ideally we should exclude
such classes from Proguard by default.

Change-Id: I690220d2e89a8ff6d0664be93dad55da8b7b66c2

Fix drag image mouse coordinate handling

Instead of processing drag events using the top left corner of the
drag image, use the mouse cursor coordinate, and fix the absolute
layout rule to compute the top left corner instead.

Change-Id: I4d5096bfd04ac0abb2148196ac7e6d9d52797712

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: I095c9f9d992c0a888eede6b1d585a683d0f8f196

Use env var com.android.ddms.bindir for ddms.

The env var is used if the system property is not defined,
which is convenient to start ddms in debug mode from Eclipse.

Change-Id: I2a33c12399c703cf3194e2e67f39562ab0d59bd9

merge from open-source master

Change-Id: I1603a3ab68a5f0eca583a835d3db730bf3d14649

Merge "Use view bounds to speed up image cropping"

Use view bounds to speed up image cropping

Use the view bounds from the layout result as an initial cropping
size. This saves us from looking at a lot of empty pixels when we
render small views. This also means we can attempt rendering a larger
surface for the preview, which makes accidentally cropping large views
less likely.

(Also fix a couple of other suggestions from previous commit)

Change-Id: I9215f00db14adc42950897637e9f66aa921d484e

merge from open-source master

Change-Id: I77c39ab13953b383641720fd44cc4958e45460e0

Merge "GLE: Yet another reference to event.offsetX/Y."

GLE: Yet another reference to event.offsetX/Y.

Change-Id: I13eeb5ed05a2f62d6afeb7009d934bd60f6f5cfd