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

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

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."

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

Merge "Select dropped elements"

Select dropped elements

This changeset makes the drop handler select the set of elements that
were just manipulated. This means that if you drop a new element, it
is both more visible (useful if you drop it into a crowded layout) and
immediately available for customization via the property sheet. This
is also important when you drag & drop to move existing elements since
without this, dragging a selected element meant you would *lose*
selection.

There are some timing tricks involved similar to yesterday's focus
checkin because in multi-document drag & drop the view hiearchy (which
we must consult to find the visual objects corresponding to the XML
just added to the model) may not yet be ready.

Change-Id: Ifa2d492a03a02f24303dd7cc95ab653523fb1758

Merge "Prevent accidental dragging of the root layout"

Prevent accidental dragging of the root layout

The layout editor deliberately does not reset the selection if you
start dragging within the bounding rectangle of one (of possibly
multiple) selected widgets. However, since it's pretty easy to have
the root element selected, this sometimes means you end up trying to
drag an item only to realize you can't because everything fits within
the root bounding rectangle.

This changeset fixes this usability issue by making the root widget a
special case. If you are dragging *only* the root, things work as
before. But if you are trying to drag anything else, it will ignore
the root, which means that you can drag one or more items
independently of the root whether or not it is included in the
selection.

Change-Id: Ia7497659e57dcc6431c3760a53847268a8b90815

Merge "Fix cross-canvas refresh issue"

Merge "Check for platform-tools presence."

Fix cross-canvas refresh issue

If you drag & drop a widget from one canvas to another canvas, the
operation seems to have no effect - the new widget is not drawn. If
you actually click on the target canvas it will suddenly appear.

We were calling redraw() on the target, but that does not seem to be
enough; an explicit focus request is necessary too (and has other
benefits; once you drop you expect that element to receive a
subsequent Delete keystroke etc).

Change-Id: I2b8cd2098dd5044fe79fc730771eede5f766d8cf

Check for platform-tools presence.

Change-Id: Ieaf6e42bc67829b01ebb0fa799bc615f85fc1a6d

Merge "Ensure FolderWrapper only lists files and directories."

Merge "Fix drag and drop of unselected item"

Merge "Fix Orientation context menu for LinearLayout."

Fix Orientation context menu for LinearLayout.

If you open a LinearLayout where the orientation has not been
specified in XML, then the context menu will say that the orientation
is vertical, but visually the layout is actually horizontal. (The
default value for LinearLayout is documented to be horizontal.)

Change-Id: Id16797d9226b19eb53c2b59249844e2406b049e4

merge from tools_r7

Change-Id: I2fab5e04b9e93009d1c40fcb887497b588abd8c8

ignore from tools_r7

Change-Id: I4a56d059c30716bbafb794a3d1a741b724ee40ea

Fix drag and drop of unselected item

If you have no selection and start dragging a view, nothing
happens. This fixes this such that when dragging begins, the item you
are dragging becomes selected. This also works where you have a
selection and you start dragging some other (non-selected) item.

Change-Id: I20fc400504f375b75c1ed17ec4850420a8500edd

Ensure FolderWrapper only lists files and directories.

This matches the description of the original File.list better
and avoid us seeing non-file objects that we can't deal with
(e.g. pipes, etc.)

Change-Id: Ie47c9926c2db4cd2605d277fe847c6a60e180d09

Merge "SDK Manager: fix to suggest platform-tools install."

Merge "SDK Manager: fix File.listfiles()"

SDK Manager: fix to suggest platform-tools install.

The way the updater currently works, it will only
generate the source.props of a new package based on the
attributes it knows from that package. That means mean
Tools r7 is updating tools, it will not add the proper
min-platform-tools-rev to the new Tools r8 package.

When "Update All" is selected, we try to do 2 new things:
- make sure to lool at local *existing* packages for
  potential missing dependencies, and suggest them for
  install.
- if a package doesn't have a min-platform-tools-rev set,
  suggest the higest revision available.

Change-Id: I76cdbc2818133429b2726d3127eedd7e65579a7e

SDK Manager: fix File.listfiles()

This fixes a couple instances of File.listfiles() that
were not validating that either a/ the file is a directory
or b/ the list is not null.

This also adds a couple toString() methods to some repo
classes, which are nice when debugging.

Change-Id: I8912d12c5344c8b511d84a58fe4693632315dff0

Merge "Volume keys need to be wake keys in order for them to work when the screen is off." into tools_r7

Merge "Fix drag and drop handler to handle reentry"

Fix drag and drop handler to handle reentry

If you drag from the palette over a linear layout view, you get drop
feedback. However, if you drag outside the linear layout and then back
in, you often don't get any more feedback. This happens pretty easily
with nested layouts.

The problem is that the feedback object is initialized and reset in
onDropEnter, but the actual guideline computation happens in
onDropMove (since only drop move is handed the mouse coordinates). The
bug happens because when you leave and return to the drop zone, the
feedback object is reset, and then the drop feedback is painted - all
before the onDropMove code has been called again.

The fix is simple: On drop enter, also call onDropMove immediately
such that the layout helpers are given a chance to initialize
themselves with the entry mouse position, before they are consulted
for drop feedback data.

This changeset also contains a fix for LinearLayouts: When there are
no children, place the guideline at x=0 or y=0 (depending on whether
the layout is horizontal or vertical). Without this fix, the line is
placed in the center of the view, which is misleading since that is
not where the view will be placed upon drop.

Change-Id: Ib5e17a2d9d3818677e4209126233bbde527207c8

Merge "Fix Issue 11735: The Delete key does not always work"

Merge "Improve visual feedback for relative layout and frame layout."

Merge "SDK Manager: local package list can be empty."

SDK Manager: local package list can be empty.

Change-Id: I44a063ffb69002b10c7db5141036e82f23e7da49

Volume keys need to be wake keys in order for them to work when the screen is off.

Bug: 3083496
Change-Id: I9b456cf002a8a9d953d631c163054f8ccc9572fa

Improve visual feedback for relative layout and frame layout.

Improve the visual feedback for the relative layout and the frame
layouts:

1. When there is multiple selection, don't draw adjacent boxes (since
that's not how things will look once you drop); instead, draw
overlapped boxes, For example if you drag a button and a clock, the
button will be placed on top of the clock, not next to it, in both
layouts.

2. Replace the center oval+x pattern with the stippled orange line
pattern, and make some fixes to relative layout such that (when we
have bounds) the preview rectangle is positioned correctly for the
parent-relative bounds.

Change-Id: Ie4c7b127cb28d9fedf194e9067f2fa9c82ede44e

Fix Issue 11735: The Delete key does not always work

Add Backspace as a key handler for the logical delete action.
Most Macs have a Delete key that actually corresponds to
a backspace action, so we must handle SWT.BS, not just
SWT.DEL. I didn't make this code Mac-specific since having
both backspace and delete work in the canvas editor to delete
the selection seems useful.

Change-Id: I8b0be10cb46ba8a16126bbc3cc59832414d7993a

Merge "Make sure all marker types are removed during clean."

Merge "Display SDK/plug-in incompatibility messages."

Merge "SDK Manager: simplify getenv var to override URLs."

SDK Manager: simplify getenv var to override URLs.

SDK_TEST_URLS is a semicolon-separated list of URLs that can be used to
seed the SDK Updater list. This is only meant as a debugging and QA testing
tool and not for user usage.
To be used, the URLs must either end with the / or end with the canonical
filename expected for an addon list, addon repo or full repo.
This lets QA use URLs ending with / to cover all cases.

Change-Id: Ia7232d5d8a5b6e85d98e735b93a969e42624e919

Merge "Fix a couple of bugs."

Merge "SDK Manager: manage URL sites grouped in categories."

SDK Manager: manage URL sites grouped in categories.

Sources are now grouped in categories:
- android official site
- all 3rd party addons which list is remotely fetched
- local overrides (from getenv SDK_UPDATER_(USER_)URLS)
- user-added add-on URLs

SdkSources (plural Sources) is the collection of all URL
sites (each one being an SdkSource that contains a collection
of packages).
SdkSourceCategory is the category of a given SdkSource.

Each source has its download URL but also an UI-visible name
which is displayed if known -- this is used for our default
android source and for 3rd party remote addons.
For all source overrides (e.g. user added or from the getenv
vars) we'll simply display the URL in the tree.

Change-Id: Idc3e9ba3395818e33fc7505ead7b0bd0be867ea3

Merge "SDK Manager: remove userCanChangeSdkRoot flag."

Merge "SDK Repo: Remove useless xsi namespace from unit-test XMLs."

Merge "SDK Manager: XML schema for addons-list."

Add MonkeyRunner script (and libs) to SDK.

Change-Id: I75d082fc2c69f994197f5887c3f38ad30cfe7bfb

Make sure all marker types are removed during clean.

Change-Id: I2a045488b86d03496d766c5c9ade9bb64fcb9246

Display SDK/plug-in incompatibility messages.

Hmm this used to work. But the lines displaying the error
messages are gone...

Change-Id: I83cf7d3e8a471fe6b2166818db9fad4f60d3d392

SDK Manager: remove userCanChangeSdkRoot flag.

We've never enabled the UI for it as the SDK path
is dicated by where the manager is started from, so
it's time to clean it up.

Change-Id: I97d7ede19e869b970e649ea8a6f9b94045a71269

SDK Repo: Remove useless xsi namespace from unit-test XMLs.

All the sample XMLs still declare a line
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance

We don't use that namespace anymore so we don't need that line.

Change-Id: I620680d1f1fb7e8196ddf0cbc13836439f3e6a6a

Fix a couple of bugs.

Using File.toURL() is dangerous because the reverse operation
(URLDecoder.decode, which for example is used by URLClassLoader to
produce a path from URLs) doesn't always succeed - for example on a
filename like "/tmp/te#st". Use File.toURI().toURL() instead.

The second problem looks like an unintentional || instead of &&.

Change-Id: Ib4e2ac13c914c002eff53455ed6d6ed898ce35fb

SDK Manager: XML schema for addons-list.

Change-Id: I64db42737bf70365c7016c1292087ebbf505cea7

Merge "Proguard default config file."

Proguard default config file.

It is located in $SDK/tools/lib/proguard.cfg and automatically
put in Ant project (create and update if project is not already setup
for proguard).

Change-Id: I9bcb2a5aafec1329b0c001799f4ba34c61569c3a

Fix bug in constraint display.

Need explicit parens to ensure proper Groovy evaluation order.
This short script shows the difference between the old and
new forms:

groovy> def id = "hello"
groovy> def it = "world"
groovy> def strings = []
groovy> strings << (id ? it + "=" + id : it)
groovy> println "strings 1 = $strings"
groovy> strings = []
groovy> strings << id ? it + "=" + id : it
groovy> println "strings 2 = $strings"

strings 1 = [world=hello]
strings 2 = [hello]

Change-Id: Iab1bc50656b1b975a522e5742240075600bab596

Merge "Fix various warnings"

Remove Groovy Closures from the tool integration API.

Remove Groovy Closures from the tool integration API, and replace with
dedicated single-method-interfaces (which can also be invoked as
closures from Groovy - but which also would allow non-Groovy
implementations to work as View handlers).

Also add warning to the javadoc for the interfaces that this is
neither a public nor a final API.

Change-Id: Ia146e1e150c2d47f95ff5bbb0771efc0e3aaa5ad

ADT: use annotations from sdklib

The removes Nullable and VisibleForTesting from ADT
and replaces them by their new versions from sdklib.

Change-Id: I2784e9d044a29a70dcd6258bb45e553246dfd477

Go to problem in source from Logcat via Double-click.

The user can choose in the LogCat Preference Page what
will be the default behaviour (go to method declaration
or go to error line).

There are now 2 available actions in the LogCat View
instead of the unique "Go to Problem" (which is now
called "Go to Problem (method declaration)").

Change-Id: I769771b29d26b625cfd0250fa23e6627821be16d

Merge "Visual adjustments to the layout feedback"

Merge "SDK Manager: support separate addon.xml schema"

Fix various warnings

Change-Id: I4864b89347ed8756b1eae83d97990555144fb93a

Revert premature tools.atree change.

Change-Id: Ic09bccca8bb6dd195e3bc2bdf580da7a3dc5efbb

Merge "Bridge MonkeyRunnerExported tags into pydoc."

Bridge MonkeyRunnerExported tags into pydoc.

This allows the use of pydoc to generate MonkeyRunner
API documentation.

Change-Id: I0af981f2023abf2cbf92cb1d7c9132936414c559

Visual adjustments to the layout feedback

This changeset makes a number of adjustments to the visual feedback
shown during layout dragging & dropping, selection, etc.

These are:

1. Render text on top of a translucent mask (white on a transparent
gray rectangle). This ensures that the text is readable regardless of
what content is under the text. We can't just use the background color
for the normal SWT drawString for two reasons: First, we need to
control the alpha of the background only (since we want the text to be
opaque and the background to be translucent), and second, we often
want to draw multiple lines of text, so we need to manually draw a
background rectangle which accommodates the maximum width of ALL the
lines such taht we don't have a ragged background.

2. Paint the outline mode lines using solid, translucent lines. And
adjust the bounding box computation in this case by 1 pixel such that
when you have adjacent boxes (such as in LinearLayout), you don't end
up with two thin lines next to each other creating a thick line.

3. Change the DrawingStyle internal API from "foreground" and
"background" colors to "stroke" (border) and "fill" (interior), with
individual alphas. This makes the color definitions simpler (for
example we don't need two separate styles, one for the selection
border and one for the selection interior).

4. Make the hover more subtle, using no border and just a light
translucent gray rectangle to only slightly draw attention to the view
under the mouse.

5. Change the appearance of selection to light blue and use a long
dashed border around it. The anchor lines are more faint and more
translucent to make them stand out less.

6. Change the appearance of the drop-preview to use a dash patterned
border identical to the selection border, but colored orange instead.
The matching border pattern will hopefully reinforce that the
drop-preview line is where the selection (also dashed) will go.

7. Use green to show available drop zones (the grid surrounding a
target in RelativeLayout, and the positions between elements in
LinearLayout).

8. For invalid drops, use a white X on a red background as the overlay
fill.

9. I replaced the oval+X pattern from the LayoutHelpers and replaced
them with the orange dashed insert-position lines. I also adjusted the
LinearLayout to draw the bounding boxes such that the middle of the
bounding box, rather than the top left edge, are aligned with the
insert position.

Change-Id: I85c77b9fa84b732a78aac635442f96e7ccfc3983

SDK Manager: support separate addon.xml schema

This splits sdk-repository-3.xsd into 2 XML schemas,
one for the sdk-repository and one for the sdk-addon.

The SDK Manager is then modified to only load <addon>
and <extra> from the sdk-addon whils the <sdk-repository>
supports everything (including <addon> if the old v1 or v2
schemas are being used).

Change-Id: I30b263f2ab48dd3ea7b70d3006fb6516291736d0

Add proguard support in the release build mode of ADT.

This is only activated if default.properties contains a proguard.config
property.

Change-Id: I9921b2796a423330bffa37e795399cf5cec948ab

Fix case where testRunEnded was called twice.

Change-Id: I14be6641c020baf0ad90f2f036e35fb854aed2ab

Merge "Fixing ADT features"

Merge "Add drawing styles to the layout helper API"