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: 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 "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 "GLE: Yet another reference to event.offsetX/Y."

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

Change-Id: I13eeb5ed05a2f62d6afeb7009d934bd60f6f5cfd

Merge "Fix Build: DragSourceEvent has no offsetX/Y in SWT 3.4"

Fix Build: DragSourceEvent has no offsetX/Y in SWT 3.4

Change-Id: I0edf5ed0d6bcfaa8de679203a2bb1c9beb67cc12

Merge "Win SDK: Split the win-sdk makefile and script into 2 parts."

Merge "Preview items on palette drags"

Merge "Update ViewInfo in the layoutlib API."

Preview items on palette drags

This checkin renders previews of Android views as you drag them from
the palette, using the current theme settings you have for the
associated editor. If you are using version 5 or later of the layout
library, you get full alpha-blending on the rendered views (so for
example in an analog clock you see right through it), and on older
version it simply crops out the edge color.

The preview image is semi-translucent, and the image is centered under
the mouse pointer. The drag feedback operations all relate to the top
left corner of the dragged item, so they are passed in the mouse
cursor minus the delta to the top left corner of the image.

The size of the preview is also used to provide bounds for the drag
feedback, such that we can show a preview rectangle rather than just
the insert point over layouts that support drop feedback.

Some views can't be rendered (e.g. QuickContactBadge), and some views
don't have any pixels painted (e.g. all the empty layouts), and in
that case we fall back to showing the palette item itself (the icon
and label) as you're dragging.

There's also a new utility class for SWT common utilities, like image
conversion.

Change-Id: I315c2dd733317bbada8dae89f57280ced2d7bb51

Merge "LinearLayout guide fixes"

Update ViewInfo in the layoutlib API.

- support for View and LayoutParams
- support for default property value map.

Change-Id: I70028710b1f76329a8bd501428fbd68a14fafa1e

Win SDK: Split the win-sdk makefile and script into 2 parts.

The main makefile+scripts to build the Windows SDK is still
in development/build/tools. However it defers to a new
matching set of files here in sdk/build to build and
package things that depend on the sdk.git or external/qemu.git.

This will make it easier for us to prepare SDKs based on
a tools_rN branch that isn't cut at the same time than the
platform branch.

This is a multi-part changeset. The other part is in
development.git.

Change-Id: Ic94d261b16ad595101c6f83cd77055715b45a622

merge from tools_r8

Change-Id: Ife4981cc78290d77662064cd3f6f9317f45a29ec

Merge "Change the adbLocation ddms extension to provide more tools location." into tools_r8

Merge "Fix issue 12251: double click handling"

Change the adbLocation ddms extension to provide more tools location.

Previously DDMS used the adb location to find the location of other tools,
but adb moved to a different tool folder.

adbLocator extension changed to toolsLocation and provide explicit
locations for all the tools DDMS cares about. This way the logic
of the tools location is only in ADT instead of being duplicated
in DDMS.

Change-Id: I87f19c7705cb822dc793264f11e680ba09eb7f40

LinearLayout guide fixes

When we have bounds for the dragged items, we show a preview rectangle
where the item will appear after the drop. We show this rectangle
midway between the two siblings that share the insert
position. However, when you are inserting *after* the last item in the
LinearLayout, there is no reason to show the item midway, since at
this point nothing below the insert position needs to be shifted
down. This checkin changes this such that for the last insert
position, we show the rectangle fully below the insert line (or to the
right of it in case of a horizontal layout).

This changeset also fixes a bug where the last (available, not active)
dropzone would not be shown on a palette drag.

Change-Id: If449b43582e072c9e0ad6d7741afbe8e845e8df9

Fix issue 12251: double click handling

Surprisingly, SWT will deliver a double click event even if you click
button 1 then 3 in rapid succession. This changeset ensures that only
a double click on button 1 warps to XML.

Change-Id: I2b470e2dea5bdb4c9240c9304650ca1c7403db84

Merge "Rename layoutlib_utils into ide_common."

Rename layoutlib_utils into ide_common.

Change-Id: Id243fe66e2ce577bc821ca454bca18444ebb5e7d

Merge "Make ADT use the new layoutlib API."

Make ADT use the new layoutlib API.

ADT now exclusively use the new API.
The older platforms that still use the old API are
accessed through a compatibility layer provided by the class
LayoutBridgeWrapper that converts the old to the new API (both
input and output).

The wrapper and the loading code for the bridge have moved
to layoutlib_utils, but into the ide.common package.
Layoutlib_utils is to be renamed ide-common later.

.sdk.LoadStatus has moved into .ide.common too since
it's used by the bridge loading code. As we'll move
more code into ide-common it's ok to have it there anyway.

Also did some minor fix to the API:
- missing implementation of ViewInfo
- Made a singleton for SUCCESS state of SceneResult.

Change-Id: I5e7130ca03b92ad71dc9c293b2ffc40566df645c

Merge "Improvements to the new Layoutlib API."

Merge "Refactor to make model rendering reusable"

Refactor to make model rendering reusable

This changeset refactors the code in GraphicalEditorPart a bit to make
the part which renders a model into an image reusable. This will be
used by an upcoming changeset to provide previews of palette items as
you drag them.

The refactoring is straightforward, even though the diffs
unfortunately don't show it. I basically split up the large
recomputeLayout() method into smaller separate chunks (validating the
current file, validating the model and validating the sdk and looking
up the bridge) and added a new render method which can take just a
model (separate from the editor's own model) and render it.

Change-Id: I96603c4beb2d56d36416cd2b6448c87dd306217c

Improvements to the new Layoutlib API.

Move to classes instead of interfaces so that the layoutlib
API code can provide default implementation.

This will reduce the testing on the ADT side about whether a method
is present for the current API level or not.

Also moved all the parameters of startLayout (actually renamed createScene)
into a SceneParams class.

Change-Id: I58389cd1bed9e79f6825b7c4e5a79206496ca439

Merge "Minor update to the layoutlib API."

Minor update to the layoutlib API.

Merge "Add transient visibility mode for empty containers"

Add transient visibility mode for empty containers

When you add a new container, such as a LinearLayout, it is usually
invisible. The reason for this is that an empty layout has 0 width and
0 height. There are two existing features in the layout editor to deal
with this: (1) Outline mode, which renders rectangles around all
views, and (2) Padding mode, which adds 10 pixels of padding to all
views. In combination, these two modes will create a rectangle for
empty layouts making them visible and user-manipulatable -- you can
for example select or drop into them.

This changeset attempts to make dealing with these types of containers
easier and more discoverable. It adds a new temporary mode where empty
containers (and only empty containers) are automatically padded and
have their outlines painted. And more importantly, this is only done
in some limited scenarios: When you drag into, or drag within, the
layout canvas. As soon as you finish the drag, empty containers
disappear again.

Unlike padding mode, we don't enlarge the design surface itself, since
this mode comes and goes easily and frequently.

In addition to this, there is special handling for selection. If you
select a zero-sized element (which for example is automatically done
when you drop a new layout, and which can also be done by clicking in
the outline), then the element is also revealed similar to the
show-empty mode, but in this case only the selected item and not any
other invisible containers are shown.

Change-Id: Ibf3ec6a080a50a8f0f55919c3d3e6c4d2890468d

Merge "Fix ADT to build with the new layoutlib API."

Fix ADT to build with the new layoutlib API.

Also make the custom cleaning of the layoutlib looper
only done through API 4. Newer bridge can do their own clean up.

Change-Id: I1ee128e09912df53e110094d8909f81bc6a788e3

Merge "New layoutlib API."

New layoutlib API.

This is the new API to move to a stateful layoutlib, allowing
for faster actions on an inflated layout.

Change-Id: Ice6324c056efc6e82d5760b5f4e3d40b58938368

Merge "Don't unregister listeners from disposed widget"

Merge "Fix icon painting transparency for the palette"

Fix icon painting transparency for the palette

The icons shown in the palette do not have transparency applied to
them, which means that you end up with white squares on a white
background. This looks okay in the outline view where the icons are
shown on a white background, but it does not look good in the palette
where they are on a gray background.

This changeset fixes the icon generation a bit such that it records a
transparent pixel for these icons. It's not an ideal solution since we
only have on/off transparency, not an alpha channel, which means that
we can't have a smoothly anti-aliased edge of the circles regardless
of the background. Luckily, a light color seems to work fine for both
cases.

While I was at it I also adjusted the font positioning slightly on the
Mac platform for some letters that needed it.

Change-Id: Idf5a4f6709096935122bfe05cfecffc63e74a863

Don't unregister listeners from disposed widget

This changeset fixes a couple of exceptions on shutdown which stem
from the fact that once a widget is disposed it's an error to attempt
to remove listeners from it.

Change-Id: Ie4b1049668805c6e9bbedd1f7f0cf480e2d68c9d

Merge "Add autoformatting of XML, and improved manual formatting"

Add autoformatting of XML, and improved manual formatting

This changeset improves the formatting of XML edited by the layout
editor in two ways:

(1) It improves the way the layout editor handles insertion and
deletion into the XML document; it looks up the indentation of the
sibling and parent elements and attempts to correctly indent new
elements based on the existing surrounding formatting, and it also
attempts to clean things up correctly on element deletion.

(2) It adds a new user option for turning on automatic XML
formatting. When this is on, it will invoke the Eclipse XML formatter
on portions of the XML after each edit. This will ensure that the
document adheres to the user's preferred formatting settings (maximum
line width, tabs versus spaces, line breaks before attributes, etc.

Change-Id: I74f9a4240a8c5ca4295c01f3b55751ef10b1c1b0

Merge "Sort palette contents"

Sort palette contents

The palette has many of the Android views in a seemingly random
order. This changeset ensures that they are ordered alphabetically.

Change-Id: I675265747e7835c933fe8e2c7f99dc3c64e6bfaf

Merge "Mark internal strings as NON-NLS in rule classes"

Mark internal strings as NON-NLS in rule classes

Mark internal strings as NON-NLS in rule classes. This code was
initially written in Groovy which is why it didn't have NON-NLS
markers.

Change-Id: I74517771271e54f165332543092a9d29fc2bc52a

Merge "Remove deprecated UiElementNode "editData" property"

Merge "Add gesture support, marquee selection, and refactoring"

Remove deprecated UiElementNode "editData" property

As requested in another review (#18753) the editData property on
UiElementNodes is obsolete (it was used by GLE1) and should be
removed. While there I also updated the naming style of local vars in
that class from underline to camelcase.

Change-Id: Ic5c50d07abedb1177cd018c866901f1e54cd0ec5

Merge "Fix XML editing whitespace handling"

Add gesture support, marquee selection, and refactoring

This checkin adds support for gestures and overlays. Gestures are
sessions of mouse/keyboard activity, and this is documented in the
javadoc for the new Gesture class. Overlays are units of graphics, and
these are documented in the Overlay javadoc. The gesture architecture
lets us isolate the logic for each different type of operation
(marquee, resize, move, etC), and with associated overlays we don't
attempt to for example paint drag feedback during a resize operation,
etc.

The checkin also adds marquee selection (as a second gesture, in
addition to the existing drag & drop based move gesture), along with
some associated changes in how the root view is treated.

As part of isolating the mouse handling and painting related to
gestures, painting etc., I also refactored the code quite a bit.
LayoutCanvas which used to be a large class has been split into a
number of new classes, one for each area of responsibility:

- The mouse listener and drag & drop code has been moved into a
  GestureManager. (A lot of the drop handling code also came from the
  CanvasDropListener class.)

- Code related to maintaining the set of rendered views, and
  performing searches in the views, has been moved into a
  ViewHierarchy class.

- Code related to selection has been moved into a SelectionManager.

- Various individual painting pieces (outline, hover, etc) have been
  moved into individual Overlay classes such as OutlineOverlay,
  HoverOverlay, SelectionOverlay, etc. This also moved associated
  resource allocation and cleanup into the overlays.

- New coordinate classes, ControlPoint and LayoutPoint, are used
  instead of ints and plain Points to make it really clear which
  methods require coordinates in the layout (such as the
  ViewHieararchy search methods) and which ones require coordinates in
  the canvas control (such as paint methods). There are factory methods
  to automatically construct the right kind of coordinate from
  different types of mouse events, as well as methods to convert
  between the two.

I also tweaked the visual appearance of selection a bit more, and
some other misc cleanup.

Change-Id: I666aabdcd36720bebe406b68237e8966d985fb8f

merge from tools_r8

Change-Id: I7bc7385266c44927db9dc66eed3b06e92eaf99e1

Merge "SDK Repo Schema: <vendor> for <extra> elements." into tools_r8

Fix XML editing whitespace handling

This changeset fixes three problems related to the way whitespace is
handled by the visual editor.

(1) The editor would insert a newline text node after a newly inserted
element, but this element was appended rather than inserted
immediately after the element, which meant that if you inserted your
element anywhere in the *middle* of the child list, you would
accumulate blank lines at the end of the parent's element and have no
separator before the current element.

The fix is simple - use insertBefore() rather than appendChild() when
inserting the new node.

(2) On deletion, no XML text nodes were deleted. This means that if
you inserted 4 elements, then deleted them, you would end up with 4
blank text nodes.

The fix here is to look at the sibling when we are about to delete,
and if it looks like a pure whitespace node, remove it as well.

(3) When nested content (like a LinearLayout) was inserted, there
would be no newline inserted before the child, so that first element
would end up on the same line as the parent.

The fix here is to detect this scenario (when we are inserting an
element into an element that has no children) and insert a newline
*before* the element (as well as after, as is the case for all newly
inserted elements).

Longer term we should do more advanced things with formatting, such as
indenting nested content, and making sure that when we insert and
remove whitespace nodes we always adjust the indentation
correctly. But for now, this changeset makes the XML generated by the
editor much more readable and consistent.
Change-Id: I2e65152568092a775074a606e3f5e54c680611c6

SDK Repo Schema: <vendor> for <extra> elements.

Change-Id: I8fa13d28b787e3ecb8bd96896e6410313e2a66a9

merge from tools_r8

Change-Id: If932f5db0eb1d3b8caf49a2f18bc94b84eab9e3f

Merge "SDK Manager: fix suggestions of new platforms." into tools_r8

SDK Manager: fix suggestions of new platforms.

I accidentally broke the way the SDK Manager was suggesting
new platforms when I "optimized" the way it was fetching new
sources when resolving dependencies. The fix is that even if
we don't need to refresh or fetch a source, we still need to
report the packages we know it contains.

Also made the Archive and ArchiveInfo implement Comparable
and defer their comparison to the one of Package. This way
we can sort the archives in the install window.

Change-Id: Ic3b39e49e8143541b19b00de09468c1b3f01b0d7

merge from tools_r8

Change-Id: I90eeb2def2e3d3f0ee6f34b0a3f710f966808a7a

Merge "ADT: support synthetic parent chains in attrs_manifest.xml" into tools_r8

Merge "Select element surrounding caret when switching from editor"

merge from tools_r8

Change-Id: I8792be345899be48cbe7c22f386f09fd4aa13691

Merge "Fix adb location in ant rules." into tools_r8

Select element surrounding caret when switching from editor

If you edit the XML, and then switch back to the visual editor, this
changeset will cause the element surrounding the caret (if any) to be
selected in the visual editor.

Change-Id: I1f03856b3b3946fe23d6e654773ee4318d0d56ed

Merge "Warp to source editor on widget double click"

Fix adb location in ant rules.

Change-Id: Ia8188de0b8af0ec4474f5c4ca2cec294c077e03f

merge from tools_r8

Change-Id: Id7dbdfae280b98547f9a80e07cc5d80d7f9f92f7

ADT: support synthetic parent chains in attrs_manifest.xml

SDK Bug: 3125910

Change-Id: I00450b13d10cbe4f32cbfbd6e1945985b599920f

merge from tools_r8

Change-Id: I907004c9f3c55a7e454e0bca644fa4efe200f15b

Merge "Fix resource issues in ADT." into tools_r8

Merge "ADT: Separate manifest test for AttrsXmlParser." into tools_r8

Fix resource issues in ADT.

Change-Id: I744833ec03d0d379ea526c193549f99f25f6811a

Warp to source editor on widget double click

Update mouse handler such that a double click will look up the
corresponding XML element, front the XML source editor and select the
text range (scrolling if necessary) to reveal the corresponding tag.

Change-Id: Iaa3d6f845c3fea190c304a07fab07314baa3f638

Merge "Add dictionary to codebase"

ADT: Separate manifest test for AttrsXmlParser.

Change-Id: I8ea1a31fd7db849fffb9043c58a91f82444d98d4

merge from tools_r8

Change-Id: I2f8e8e1257fb51c2fda1617804fa6136526b426c

Add dictionary to codebase

Eclipse has the ability to spellcheck comments, and it ships with an
English dictionary. However, many valid terms in our codebase is not
in this dictionary. This checkin adds a dictionary file which contains
many of the valid spelling words used in our codebase.

(Unfortunately, Eclipse does not support "project dictionaries" like
some other IDEs where individual user dictionaries are merged with
shared project dictionaries. However, this new dictionary should be
useful for developers who use a dedicated workspace for Android
development.)

This changeset also fixes some typos.

Change-Id: Ied6647f6cb550460c0087498f8c94fa6624e3b4e

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