merge from tools_r8

Change-Id: I2f8e8e1257fb51c2fda1617804fa6136526b426c

Merge "ADT: Fix display of unknown Manifest elements." into tools_r8

ADT: Fix display of unknown Manifest elements.

SDK Bug: 3135772

Change-Id: I4f7c84c20b31e70db20a202f168158d500b3c09d

merge from tools_r8

Change-Id: I301171df40d831b03ba3563e72a3910dd6d6951e

Merge "ADT: Exclude <include> tag from the GLE palette." into tools_r8

ADT: Exclude <include> tag from the GLE palette.

Change-Id: Ia318556bc88a823fb6eae9048634c824524f3b90

merge from tools_r8

Change-Id: Ibdee0b6cb56b39ccfe1c604f8300f94c81320590

Merge "SDK: Fixed proguard.bat & co." into tools_r8

Merge "ADT: fixes to make Proguard work under Windows." into tools_r8

SDK: Fixed proguard.bat & co.

These are similar to the external/proguard/bin/*.bat
except they use $SDK/tools/lib/find_java.bat to locate the
java location and use it if available. find_java will print
a proeminent error if java cannot be found.

Change-Id: I1fceae838436053031f405de9b3f1ab2d64222e6

ADT: fixes to make Proguard work under Windows.

- the proguard.bat can only pass 9 args, but we have 15+, so
  we'll use a temporary proguard config file for Windows.

- PROGUARD_HOME needs to be defined properly since the default
  proguard.bat cannot infer it properly. We need other fixes
  to that batch file so eventually we're not going to ship the
  default one and we can fix the .bat file, but properly defining
  this env var is probably a good idea anyway.

Change-Id: Id007d5504a5e05d1c4b5a0728e96e0d677978ef4

Merge "Support 3rd party layout rule loading, and rip out Groovy support"

Support 3rd party layout rule loading, and rip out Groovy support

Add support for loading 3rd party .jars providing additional layout
rules. This can be configured by adding a property referencing the
jars to be loaded as part of your build.properties, like this:

default.properties:
...
layoutrules.jars=chart-rules.jar:graph-rules.jar
...

This will create a class loader referencing chart-rules.jar and
graph-rules.jar (as well as the visual editor's plugin class loader as
a fallback), and this class loader is used to load IViewRule
implementations.

In addition, this plugin rips out the various remaining Groovy hooks
and references that were earlier used to load Groovy scripts as layout
rules, and removes groovy from the load path and build symlinking
scripts.

Change-Id: Ia17a60259559ec86270726add258382a879117dc

merge from tools_r8

Change-Id: If3f0a5d9b487e23c8f28fc4db676b0cb70d09c5a

Merge "Fix external jar support when building with proguard." into tools_r8

Fix external jar support when building with proguard.

Change-Id: I3dafb284770f475d70a212cbe22cdae6bff36ff7

Merge "Add layout unit tests"

Add layout unit tests

Add layout unit tests, and some infrastructure for testing.  Also fix
some formatting errors (>100 column lines) in the previous commit.

Change-Id: I3eabf30998ab7deb84df57e4d0c10cf57ee399d5

Merge "Port layout rules to Java"

Port layout rules to Java

We had a number of layout implementations in the tool written in
Groovy; these were hard to deal with because of lack of good tool
support (debugging didn't work, refactoring didn't work, code
completion didn't (always) work, go to declaration didn't work,
semantic checks like unused code didn't work, etc. etc.)

Since these layout helpers are only getting larger, replace them by
equivalent Java code to make development easier.

This checkin also moves the API classes formerly used by Groovy
scripts into a new package (next to the Java layout rules) under
com.android.ide.common (api and layout) since this code isn't Eclipse
specific and could be used by other IDE vendors.

These interfaces were left identical (only the package statements and
directory location changed), with two exceptions: I added a new method
called "in" to IAttributeInfo.java, and I added a parameter to
IViewRule's onInitialize method.

The Groovy code was kept as close to the original as possible; I
copied in the Groovy code, and then replaced the Groovy-specific
constructs (closure-iteration on collections, literal map syntax, etc)
with equivalent Java code. The only tricky part was ensuring that
Groovy's handling of the == and != operators were translated into
.equals calls.

Change-Id: Idf7660ddea3766eac0a4a65ce6524d3f5119f7b2

merge from tools_r8

Change-Id: I5d09a769d021b995afde56faef4d757124bef5e1

Merge "Remove temporary disabling of proguard during ADT export." into tools_r8

Remove temporary disabling of proguard during ADT export.

Change-Id: Iadff55b838b7317f9dcc518d5246a2889f7a5f39

merge from tools_r8

Change-Id: Ic11d6314c05e66d9c60a53dcdb88b8de279a3d5b

Merge "Workaround failure to collect test info for large test suites." into tools_r8

merge from tools_r8

Change-Id: I139ff9bb81b703f7fa305b1234e3c6330012d3f8

Merge "Integrate 6db57208 into tools_r8. DO NOT MERGE." into tools_r8

Merge "Integrate 5397067d into tools_r8. DO NOT MERGE." into tools_r8

Merge "Integrate 17398174 into tools_r8. DO NOT MERGE." into tools_r8

Integrate 6db57208 into tools_r8. DO NOT MERGE.

Initial cut at MonkeyRecorder.

MonkeyRecorder (and MonkeyPlayback) are a set of tools for using MonkeyRunner to record and playback actions.  The current implementation is not very sophisticated, but it works.

Please don't review yet.  Needs a lot of style cleanup.

Change-Id: I79ed1b548d295563a184af5fb945140f93586b17

Integrate 5397067d into tools_r8. DO NOT MERGE.

Fix field visibility by ensuring they stay in __dict__.

Change-Id: I2e93dd4ba8c8cc3f05ca477091932268ec5d78da

Integrate 17398174 into tools_r8. DO NOT MERGE.

Make MonkeyRunner's adb finding more robust.

This includes fixes for running out of the SDK.

Change-Id: Ie73ccd391b34f8a75f23738a97c42902f91af841

merge from tools_r8

Change-Id: I50e3e6ada3a357bd519d5ccd7465ca236a525b84

Merge "Gen folder should not be derived." into tools_r8

Gen folder should not be derived.

This fix the issue where gen is not present after project
creation (it was actually created then deleted as part of a clean
up phase)

Change-Id: I25cd67c076ed401b482d97292640c08a2a7a5830

Workaround failure to collect test info for large test suites.

Add a small delay between tests when collecting test info.

Bug 1796494

Change-Id: I57061d7a21f8c8517c03101e28a3de82cdd61cd8

Merge "Fix ddms filechooser behavior and Mac accelerator"

Fix ddms filechooser behavior and Mac accelerator

Fix a couple of issues in ddms:

First, fix the code to remember the path you last opened the file
chooser with; if there is nothing in $lastImageSaveDir, it should look
at $imageSaveDir (which is the variable the default setting
($user.home) is initialized into).

Second, and this seems to be Mac specific, the
FileDialog.getFilterPath() call does not return the path you have
navigated to, which means that on the Mac it never sets
$lastImageSaveDir correctly - it always sets it to the original
suggestion. The fix is trivial - use File#getParent instead which does
the String manipulation to extract the parent portion of a string
which represents a path.

Finally, on Macs (only), make the keybindings use the Command key
instead of the Control key since that's the norm.

Change-Id: I1b0f381606f5373ddad973754e49ce07856a9bae

merge from tools_r8

Change-Id: I9a270a8b095100a7ff54338d60aa3990ec46b677

Merge "Integrate 9174add7 into tools_r8. DO NOT MERGE" into tools_r8

Merge "Integrate 5b6aa7d0 into tools_r8. DO NOT MERGE." into tools_r8

Integrate 9174add7 into tools_r8. DO NOT MERGE

SDK Manager fix: message to restart after tools update.

The latest CL made this appear after a platform-tools update instead.

Change-Id: Ib52cf88adfb1557c112c3a8153dad536c5932fa3

Integrate 5b6aa7d0 into tools_r8. DO NOT MERGE.

SDK Manager: addon schema version number in about box.

Change-Id: I35d4e1db042282e269e3d1247d36ffa0e44291d5

Merge "SDK Manager fix: message to restart after tools update."

Merge "SDK Manager: addon schema version number in about box."

SDK Manager fix: message to restart after tools update.

The latest CL made this appear after a platform-tools update instead.

Change-Id: I5e27612b3fa6fd7e53cd10a369b2d5eba19a168b

SDK Manager: addon schema version number in about box.

Change-Id: I2a54180f46e52cba931154ae57826c6df22b6a07

Merge "Update application icons for sdkmanager and ddms"

Merge "Initial cut at MonkeyRecorder."

Merge "Fix field visibility by ensuring they stay in __dict__."

Merge "Make MonkeyRunner's adb finding more robust."

merge from tools_r8

Change-Id: I2d68e2e8463c9200f600deed4d7a4b3e55a6dc64

Update application icons for sdkmanager and ddms

Update the dock icon used on Macs for the sdkmanager to one that has
transparency, just as the emulator icon already does. This is the norm
on Macs.

Update the application icon used for ddms to be an android with a
wrench, in a similar style to the other tool icons.

This icon is also transparent, which I hope will work on Windows - it
would be great if Raphael could verify that (and I should probably try
to get a Windows image into my VMWare install so I can do these checks
myself in the future.)

Change-Id: Icd0954cb77ba0845034a79bf9df6dd271019632e

Merge "Integrate 13f4bf0d into Tools_r8. DO NOT MERGE." into tools_r8

Integrate 13f4bf0d into Tools_r8. DO NOT MERGE.

Fix bug in LinearLayout handling of last lines

This fixes a bug where the insert-line for the last position is not
always *drawn* (the suggestion was there and the feedback rectangle
would snap to it, but the dashed line was not painted because the code
to suppress self-dragging was interfering.)

Change-Id: I853ef51a5a160f151119b7c819a3715cbde98b5a

Merge "Fix bug in LinearLayout handling of last lines"

Merge "Integrate 11357696 into tools_r8. DO NOT MERGE" into tools_r8

Merge "Integrate 3c36e33f into tools_r8. DO NOT MERGE." into tools_r8

Integrate 11357696 into tools_r8. DO NOT MERGE

SDK Manager: Restart ADB after platform-tools update.

Change-Id: Ie59201a3f303e1a9a837e82cbf4a83f620b4e7f2

Integrate 3c36e33f into tools_r8. DO NOT MERGE.

Fixing Issue 11557:   SWTError: No more handles exception on Eclipse

Change-Id: Ie5e7d8af8ef0376cd4d3e9afcfdd328c331a5d96

Merge "SDK Manager: Restart ADB after platform-tools update."

Fix bug in LinearLayout handling of last lines

This fixes a bug where the insert-line for the last position is not
always *drawn* (the suggestion was there and the feedback rectangle
would snap to it, but the dashed line was not painted because the code
to suppress self-dragging was interfering.)

Change-Id: I601d2f747357d42b7a797a1ed30b723830d03520

Merge "Fixing Issue 11557:   SWTError: No more handles exception on Eclipse"

SDK Manager: Restart ADB after platform-tools update.

Change-Id: Ie4e21e2922d2b281723a4ee3b5aeb1c3c646beba

Fixing Issue 11557:   SWTError: No more handles exception on Eclipse

Change-Id: I3eb6ddd0b11f8d16c6f441333ddd8c5a8e9dd477

Merge "Set hover fill color"

Merge "DexWrapper is now loaded by the SDK instead of the platforms."

Merge "Updated changelog for ADT."

DexWrapper is now loaded by the SDK instead of the platforms.

Since tools_r8, dx.jar is now located in platform-tools. instead
of platforms/android-<API>/tools/lib/.

There is therefore no need for all the platforms to load their
own dx.jar through reflection into DexWrapper. The SDK now does it
for all, and the builders query the SDK for the wrapper.

Change-Id: Ia79ccaf95237f2b25b43f26241e6335acc7050a3

Merge "Missing platform toosl doesn't mean the SDK is invalid."

Updated changelog for ADT.

Change-Id: I523c4ecf9c037410d23c28bf5ce778f3e3d032b7

Initial cut at MonkeyRecorder.

MonkeyRecorder (and MonkeyPlayback) are a set of tools for using MonkeyRunner to record and playback actions.  The current implementation is not very sophisticated, but it works.

Please don't review yet.  Needs a lot of style cleanup.

Change-Id: Id300a27294b5dc13a842fade900e8b9916b8a17b

Make MonkeyRunner's adb finding more robust.

This includes fixes for running out of the SDK.

Change-Id: Ib14959929bec2b8d5514adbe405207be4ad41264

Set hover fill color

The code which paints the hover didn't actually set the hover fill color
on the graphics context so it was just blending with white. This happens
to be the color the fill was initialized to so the problem wasn't
noticeable.

Change-Id: I95f0eface23f2a772b687ba348997a3eac2d85a9

Missing platform toosl doesn't mean the SDK is invalid.

Change-Id: I623edf5df8bcc9b0b1f806c49e863879fcd4fda4

Merge "SDK Manager: refuse to not install dependencies."

Fix field visibility by ensuring they stay in __dict__.

Change-Id: I0f632f799ac7f554bb524099208385973423a0d0

SDK Manager: refuse to not install dependencies.

In the case the SDK Manager found a package that would fix
a broken dependency of the currently installed packages
(e.g. user has Tools but not Platform-Tools), it will refuse
to not install the package that would fix that dependency.

Bug: 3092907

Change-Id: Iea21c69d726b7c859bfbbffc74ba61b690c4afe0

Merge "Fix typo in precompiler builder."

Fix typo in precompiler builder.

Bug: 3102564

Change-Id: I44d60fc89e91eadf55c4f5451493f33ec5e13c73

Merge "Editors preference page."

Merge "Add in Windows batch file for monkeyrunner."

Merge "SDK Manager: display URL host in source tree."

Merge "SDK Manager: Fix dependencies when installing selected packages."

Add in Windows batch file for monkeyrunner.

Change-Id: Iebe42de2c5c8eb275f6dc4756f6209726f973860

Editors preference page.

For now, only the monitor density, used by the zoom control
in the GLE.

Change-Id: I01219609e6196d845d45e306f5b93c3acf0b2ee2

SDK Manager: display URL host in source tree.

Change-Id: I2fd27d931fe9011cc2894c7179e2e14327867b94

Merge "Connect the resolution chooser to the zoom control in GLE"

Merge "Project property cleanup."

Connect the resolution chooser to the zoom control in GLE

When using the Real-Life zoom control the first time, it
will display the Resolution chooser (from the AVD start dialog)
to let the user figure out their monitor dpi.

Next Step: add the actual density to a prefs panel so that it
can be manually tweaked.

Change-Id: If2322dd077b657386ab0452f9c507cb3c0730c32

SDK Manager: Fix dependencies when installing selected packages.

In the SDK Manager, when you select a specific package from an
available repository to install it, it used to not notice if there
was a broken depedency. This fixes it. It means we have to force
loading all known available repo sources to find them if necessary.

This CL fixes a minor issue: trim all URLs before trying to use
them. The CL is voluntarily a bit defensive for that. This can
happen when we load URLs from external addon sites which we do
not control.

Change-Id: I652e1fa0e74b4384d0e95819970a59519fafa05a

Merge "Make drag and drop create a single undo operation"

Project property cleanup.

Remove obsolete sdk-location on project update, don't
use it as backup location anymore (main_rules.xml won't work with
it anyway).

Remove the old application.package properties since older platforms
will use the new rules anyway.

Change-Id: I5a5ec3d1289cf793dd0f98fb778bd84086976c52

Make drag and drop create a single undo operation

If you drag and drop a widget, you end up with two separate undo
operations: One which inserts the widget in its new location, and one
which deletes it from its former location. If you drag a widget
somewhere and then change your mind and press Undo, you will therefore
end up with *two* versions of the dragged widgets, since undoing just
once only undoes the source removal, not the target addition.

This changeset fixes this. It's not a very clean fix since drag and
drop is a protocol where the drag source and the drop handler are
deliberately not connected. We now create a single undo context before
calling the layout helper's drop handler, and when it has finished we
then remove the source, still within the current undo context. This is
done by having the drag source register a cleanup handler with the
global drag info object, and this handler is invoked from within the
drop target's undo operation.

This means that we end up creating the undo unit on the tool side
rather than in the layout helpers, where more specific undo labels
were assigned. To deal with this, there is a new function which
produces a suitable Undo name. It creates the name by considering
whether it's a move or copy, as well as the unqualified names of the
dragged element and the drop target. For example, when dragging from
the palette you may end up with "Drop Button in LinearLayout", and if
dragging a multiselection you may end up with "Move Widgets in
RelativeLayout", and so on.

Change-Id: I242b51e5a9c7cd9eae55aa4139c510cb26b0e8fb

Merge "Fix various bugs related to linear layout handling"

Merge "Prevent layout interaction on certain layouts"

Prevent layout interaction on certain layouts

Some layouts, like ZoomControls, extend another layout class (in this
case, LinearLayout), presumably for implementation convenience, but
the ZoomControl isn't intended to be used as a generic LinearLayout
where you want to drop new widgets in between its existing children,
where you want to change the orientation, etc.

Therefore, this changeset adds a new layout rule, IgnoredLayoutRule,
which various new widget-specific rules inherit from to prevent
default layout handling from kicking in.

Change-Id: Ia1323866d62859125692207159b9aa855cdf21c1

Fix various bugs related to linear layout handling

This changeset fixes the following issues with LinearLayout:

* Correctly position the last insert position in the linear layout
  when the layout has children.

* When you drag a widget which is already in this LinearLayout, don't
  show insert positions before and after this element, and show a
  match position exactly where it already is.

* Show the best-matching lines not only when you don't have bounds
  (which it was already doing) but also in conjunction with the bounds
  outlines. When the dragged bounds are much larger than the widgets
  around the drop position, it was a bit hard to figure out where
  exactly the proposed drop position is, and this change makes it a
  lot clearer.

* Fix positioning of the drop rectangle. Before this change we were
  positioning the drop rectangle using its existing deltaX,deltaY in
  its old parent; this does not work well both in multi-document cases
  as well as cases where you are dragging from one layout to
  another. We are now simply positioning it at delta=0 in the new
  layout; the next enhancement here would be to consider the
  layoutGravity.

Change-Id: I1c71c3c48a4c104a5cde7382fb1f9a21e57122bc

Merge "Hide anchor lines and tweak text positioning"

Hide anchor lines and tweak text positioning

Temporarily hide the anchor lines (the lines from each selected
element to its parent's bounds edges) until we can find a way to make
it clearer what these mean.

Also tweak the pixel positioning of the selection label text to
account for the padding around the textbox mask since it was partially
overlapping the selection stroke.

Change-Id: Iaa4c7139d4d56288b78a104647d5cbb5752f7888