Merge "Sort XML attributes logically"

Merge changes I984ae397,I24f74378

* changes:
  SDK Man2: Clear local source caches when reloading.
  A few minor UI tweaks to SDK Manager 2.

Merge "SDK Manager2: revamp package diff logic."

Merge "Fix code completion handling for text selection"

Merge "Fix 18539: "Extract Android String" mangles the replacement str"

Fix 18539: "Extract Android String" mangles the replacement str

The refactoring could end up processing the current Java file twice -
first using the normal code path, and a second time if the "Replace in
All Java Files" checkbox was checked. This second pass would add a
second set of edits which would mangle the file.

Change-Id: I99ed50ab477af12d7fb54a7de6401689dad5d104

Fix code completion handling for text selection

This CL fixes
18607: ADT 12 problems with xml files

The bug is that if there is a text selection and you invoke code
completion and select an alternative, the wrong number of characters
is deleted as part of the replacement.

The fix is simple: remove the code which adds in the selection count
from the replacement string, since the complete replacement count is
now already computed properly without regard for the selection (we
should -always- replace the complete value token, not just the
selection).

Most of the diffs actually pertain to new unit tests for this because
the testing infrastructure needed some changes to handle references to
selection ranges rather than just caret offsets.

There are also a couple of updates to some test goldenfiles unrelated
to this fix, which updates the golden files to reflect recent changes
like the package name of GestureOverlayViews and the @attr resource
type in completion.

Change-Id: Ie4f680f201930414457a79846f722db6efc89f93

SDK Man2: Clear local source caches when reloading.

Change-Id: I984ae397ada014ff8869d5775be1bc779ef22319

A few minor UI tweaks to SDK Manager 2.

- Make the tree filter checkboxes (new, installed, obsolete)
  work again but this time by using a proper TreeViewer filter.
- Fix missing name column icons in the TreeViewer.
- Fix display of obsolete items, trimming the duplicated platform
  name when possible.

Change-Id: I24f743788b01fe8c9c5f9a7f9b301f725e32f666

SDK Manager2: revamp package diff logic.

This revamps the diff algorithm used to merge
the local and remote packages found during a
repository load into the PkgItems displayed in
the tree viewer.

FYI all the Package and Archive sub-classes are getting
an equals() and hashCode() methods to make them behave
correctly when put it in a HashTable or Set. These are
the auto-generated stuff from Eclipse.

Change-Id: I1494a0ed44cd768eed252e3a81b9e74bf86d563c

Merge "SDK Man2: New UTs for install/delete cases."

SDK Man2: New UTs for install/delete cases.

Change-Id: I1f1e95592f4946ab61ad512f187f87be60ea5834

Merge "Fix possible NPE in prop page if ProjectState is null."

Merge "Add support for aapt CrunchCache in ADT and Ant"

Add support for aapt CrunchCache in ADT and Ant

This change lets ADT and Ant use the crunch cache system
set up in change I58271fb2. This commit is separate because
sdk and framework are separate git repositories.

Change-Id: If2828d157acad0b5adb812001e777c199e3c62fe

Merge "Add support to Ant for packaging dependency checks"

Fix possible NPE in prop page if ProjectState is null.

If default.prop is not present then ProjectState is null,
but the property page is required to set the target (to create
the file)...

Change-Id: I8bb9337476c4de1d20feca946a6f2361dc8c9af3

Merge "Fix to avoid NPE in ADT PreCompiler"

Fix to avoid NPE in ADT PreCompiler

This change removes the possibility of throwing an NPE when calling
AAPT from the PreCompilerBuilder.

Change-Id: I521e4088d1f0942e575d6edff91049c78d9c1eb8

Merge "SDK Manager 2: incremental sort when sorting by source + UT"

Merge "Add support to Ant for dependency checking"

Merge "Adding support to ADT and ant for smart R.java gen"

Merge "Properly unload the target data when unloading an sdk."

Properly unload the target data when unloading an sdk.

Change-Id: I927071585db46de12635b483b3a466e671a3bff6

Merge "Add 7" tablets to device list used in the layout editor."

Merge "Android project's output for JDT is now bin/classes/"

Android project's output for JDT is now bin/classes/

This matches the bin structure of the Ant build system where
the javac output is in bin/classes/ and the Android specific files
are in bin/
Also, this will allow us better control of android output files
as they won't conflict with javac's output.

New projects are created with this setup and existing projects
are converted as they are opened.

Change-Id: I83339906b0d43d7a988a75927c60b664c183d27f

Add support to Ant for packaging dependency checks

This change adds in support for making aapt generate a dependency
file for the .ap_ package generated during resource packaging. Ant
will then check this dependency file before calling aapt again and
will only repackage resources if the dependencies have been modified
in some way.

Change-Id: I56462163c5dd064c1416bc43913f044df8ee9be1

Add support to Ant for dependency checking

This change adds dependency checking to the Ant generation of
R.java. If no resources have been changed or added since the
last generation of R.java then that call to aapt will be skipped.

Change-Id: I262b94111316496fac54da8030a6d474dc30b584

Adding support to ADT and ant for smart R.java gen

This change lets ADT and ant make use of the new
--extra-packages flag in aapt to allow for all the R.java
files to be generated using only one call rather than one call
per library.

Change-Id: Ibc060b1218010cfbae108f4f1aba2c25fe69a964

Merge "More features for adb bash_completion"

Merge "Various fixes to the DDMS Native heap panel."

Various fixes to the DDMS Native heap panel.

Change-Id: I468be358b937f63adca3528fb66aa5d70e2e6827

More features for adb bash_completion

* Aborts early if "adb" isn't in the path.

* Clean up formatting to be more friendly to vim's syntax highlighter.

* Add vim mode line

* Adds these adb commands:
    install
    uninstall
    push
    pull
    reboot

* Rename "adb" to "adb.bash" so we can filter out more effectively.

Change-Id: I76cff06a12cd827ba1d901ac3005059635454822

Merge "Read the default skin from sdk.properties."

Read the default skin from sdk.properties.

Change-Id: Ifad6f2186e499f579df1608055dda4eca1084830

SDK Manager 2: incremental sort when sorting by source + UT

Also fixes some edge cases with the sort-by-api and ensure
switching between sorts doesn't mess too much with the
packages shown.

Change-Id: I7da43915f4e48bd6cd80760a6a15d657777fa0f7

Merge "Add new InsertType for widget moves within the same layout"

Merge "Fix to build dependencies during delayed PostComp"

Merge "Adjust API level for phone comparator"

Merge "Add bash_completion scripts for adb"

Add bash_completion scripts for adb

Adds rudimentary support for bash_completion using adb. It supports all
the optional arguments for adb, completion of individual commands, and
support for calling into the "shell" function and filename completion
within the shell.

Change-Id: I8e5ca53f3988140dc169a2e353dd92ff0438d6b7

Sort XML attributes logically

This changeset modifies the layout editor such that it writes
attributes in a certain order:

* id
* style
* layout_width
* layout_height
* other layout_ attributes, sorted alphabetically
* other attributes, sorted alphabetically

The layout editor will produce attributes in this order when
- New widgets are dragged into the layout
- Widgets are moved in the layout
- It will also insert attributes in the right place when they are set
  as the result of (for example) using the context menu actions.

Note that this ordering is applied unconditionally - there is no user
setting to turn it off. However, note that the current behavior is
random - moving a view for example will scramble the attributes (in an
order which is related to hashkeys in a map), so the option would be
"sort attributes logically" versus "sort attributes randomly"; if we
want an option to "leave attribute order alone" that will need to be
implemented.

Limitations:
- This does not yet modify the formatter to reorganize attributes.
  Thus, Ctrl-Shift-F to reformat the XML will not change attribute
  order.
- It does not fix the problem that the XML model updater does not
  respect the formatting settings (such as one newline per attribute)
  when manipulating attributes.

This will be addressed/worked around in subsequent CLs.

Implementation Note:
The Eclipse XML model updater will unconditionally *append* any new
attributes. We take advantage of this to perform attribute sorting by
ensuring that we always insert new attributes in the right order. We
also check for existing attributes and any which fall
lexicographically later than the new attributes are removed and
reinserted in the right sequence. In order to avoid performing these
removals and additions repeatedly on a node when we set multiple
attributes, and to avoid flushing attribute changes *immediately*
(which was the case until this), we now queue up all pending attribute
values in the nodes and apply them at the end when all attribute
changes for a given node are known.
Change-Id: If39f8c29f26e281f7c6622a31e11ba49724d274a

Add new InsertType for widget moves within the same layout

For some layouts, such as LinearLayout, when you drag widgets into the
layout, we automatically adjust the widths and heights of some
widgets, such as text fields (to make them fill the full width in a
vertical linear layout, and so on).

However, this was also done when dragging a widget from one place to
another within that same layout. In that case, we should leave the
dimensions and weights alone, since the user may have deliberately
changed these.

This requires splitting the InsertType.MOVE into two, since we still
want to apply default sizing when you move a widget from a different
layout into this one. The InsertTypes are now MOVE_INTO and
MOVE_WITHIN, and we only apply default size styling if we are not
moving within the same layout.

Change-Id: I73ace89aff8111417bd6635e13397e7f027bab0a

Add 7" tablets to device list used in the layout editor.

Change-Id: Ib46d23c1ad85e2b0b4b77040cd3498051c79e926

Merge "SdkManager: fix issue with double-packages on reload."

Adjust API level for phone comparator

Change-Id: Ia8220aca9c4bb338d8a26d9878f56449060cd51d

Merge "Adjust the match position description to be relative to the layout"

SdkManager: fix issue with double-packages on reload.

Rewrote the merge in place + sort of packages by API.
Did some refactoring + added some unit tests.

Change-Id: I168b306853185866606ed76c324a8db6051b15bb

Adjust the match position description to be relative to the layout

Currently the matches displayed for absolute X and Y locations will
list the coordinates relative to the entire screen, not to the
surrounding GridLayout. This CL changes the feedback to be relative to
the GridLayout bounds.

Also fix a bug in updating tooltips/messages which would call tooltip
flicker (because the message update method was called too early,
before the message object was ready, causing the tooltip to be
removed, then reshown.)

Change-Id: I665586dbcb1f3f6cdcb3a3f475cf363cd258bf83

Merge "Fix android.bat for Windows SDK."

Merge "Support XML closed elements"

Merge "Properly support spaces in ant project directories."

Merge "Fix resize feedback for cell spanning mode"

Merge "Describe GridLayout matches in drag tooltip"

Support XML closed elements

When you create new elements (by for example dragging a view into a
layout), ADT would *always* create an open element, e.g.
    <Button ...></Button>

This CL uses the element descriptors to create closed elements like
this:
    <Button .../>

For elements that support children it will continue to create open
elements:
    <LinearLayout ...></LinearLayout>

Change-Id: I885fc4bb2611e20d511a3a40c81e202de00b7ac2

Fix resize feedback for cell spanning mode

When resizing in a GridLayout you can toggle whether you are resizing
the widget or the cell bounds with the Shift key (and the resize
feedback tells you this).

However, the drawn resize feedback was always showing the widget
bounds. This CL fixes this such that when you are resizing the cell
bounds, the current cell bounds are displayed instead.

(Also contains a simple fix to ensure feedback is drawn when you
click and hold (without dragging yet) on a resize handle.)

Change-Id: I7cf24f4508aa5bbafd45f36b13be41e66ecbc6cc

Describe GridLayout matches in drag tooltip

This CL adds a description of the current best match (horizontal and
vertical) in a drag tooltip, such as "Insert into bottom of row 5" or
"Align right at x=120".

Change-Id: I08d6b158c5e48d1881ec946037389609d8be9d83

Properly support spaces in ant project directories.

After applying https://review.source.android.com/24515 for
the dalvik/dx change, this allows ant project to build
correctly on Windows if there's a space in their directory
path.

Change-Id: I4c6d5f87fae2dcf2955698633a16a0bde501694f

Merge "Grid Layout Support."

Grid Layout Support.

This changeset adds basic support for grid layouts:

- Freeform mode where you can drag & drop anywhere in the layout, and
  guidelines are offered to align with other left boundaries, right
  boundaries, baseline vertical alignment and center layout horizontal
  alingment.  In addition, it also offers "preferred spacing"
  positioning, using the recommended distance between widgets.

  Also, during a drag a regular 16x16 grid is also (faintly) shown and
  (when there are no alignment matches as explained above) the
  position snaps to this grid.

  We also show the grid-structure (rows and columns, not the regular
  snapping grid) during drags to make it more obvious that behind the
  scenes there is a grid (because some other operations might shift
  rows and columns so it's important to present the right user model.)

  The freeform grid layout editor will automatically create rows and
  columns and size them using spacers which are hidden from the user,
  and on deletion these are automatically cleaned up as well.

- Grid mode where you can drag to any given cell, or between any two
  rows or columns.

- Some layout actions for manipulating the grid (add/remove row and
  column, adjust the gravity, toggle modes.)

This CL also contains various diagnostics code for the grid mode, and
does not yet correctly support dragging multiple views simultaneously.

Change-Id: Ie9ec54805039645e3db78d19095da86b04e44ca0

Merge "Fix merge error"

Fix merge error

Change-Id: Iac2dcc3a504ab2dbc500bcd882d61b460879c7eb

Merge "Add AdapterView handling for Spinners"

Add AdapterView handling for Spinners

This CL adds AdapterView handling similar to what we already provide
for ListViews and GridViews for Spinners.  This means that it will
provide an adapterbinding which provides a layout to use at
designtime. It also adds the ability to pick a different layout to use
at designtime.

More importantly, this fixes rendering of Spinners on Theme.Holo.

Change-Id: I30cef98d22ed57b2e698c4590948e25bf3fecedd

Merge "Add resize feedback tooltip"

Add resize feedback tooltip

This changeset adds a "tooltip" during resize operations close to the
mouse cursor which displays feedback about the current resize
dimensions.

The feedback window is similar in appearance to a tooltip, but unlike
SWT tooltips it moves with the cursor and updates the text
dynamically, and appears immediately rather than being tied to mouse
motion timeouts. It is also partially translucent and uses a slightly
smaller font.

Change-Id: I6b663510f078f325d2f7b168c887aeff14de31b8

Fix to build dependencies during delayed PostComp

This patch adds a boolean flag to ProjectHelper.build()
that when set ensures that all projects which are dependencies
of the project being built will be built as well.
This fixes buggy behavior where
a package would not be fully built when its JUnit tests
were built/run.

Change-Id: Iab19299ea980564345f2a7f30bf61a69d77d96ee

Merge "Only set the text attribute on views that define it"

Merge "Add support for hidden views"

Merge "Simple insets support"

Only set the text attribute on views that define it

Ensure that we only set the text attribute on widgets that define
it. Now that we preserve unknown attributes this meant that we'd end
up setting text attributes on all widgets rather than just those that
define it.

Change-Id: Ief67a777ca40149e0be370b7a561f80fbffa3d5d

Merge "When using prerelease SDK versions, use codename instead of API level"

Add support for hidden views

There are some views which should hide in the UI to avoid clutter,
such as <Space> widgets. This changeset adds support for hidden views,
such that they don't show up in the outline, don't show up for hovers,
aren't included in drag selections and so on.

Change-Id: Icb18f4530b8aff584e72280f5bd09e776af9204c

Simple insets support

This changeset adds the basic plumbing for supporting insets, such
that selection handles can reflect the true bounds of a widget, and
such that guidelines which show adjacent matches can show and measure
the correct visual distance between the widgets.

The actual insets data is not available (though there are some
commented out measurements for some of the most important widgets,
like buttons), so the purpose of this changeset is to put the APIs in
place such that the various view rules can correctly account for these
deltas when they become available.

Change-Id: I326bfa22f0d239d76685b371d38bb8eac594a53b

When using prerelease SDK versions, use codename instead of API level

The New Project wizard now automatically fills in the Minimum SDK text
field based on the selection in the SDK target chooser. However, it
was hardcoded to always pick the API level number. That's not correct
when using SDK versions with a code name; for example, when used with
a dev SDK build the minimum SDK version should be "AOSP".

Change-Id: I28273007be7e56f3a34b683f01234d5d1046aa0a

Merge "Change scheme for picking tablet or phone configurations"

Merge "Add deletion hooks for view rules"

Merge "Add "android create avd --abi" parameter."

Add "android create avd --abi" parameter.

SDK bug 4965341

Change-Id: Iaff42d1272a9c10ba62534c5d478e35e6c2df6e5

Merge "Remove System.exit from traceview code running in Eclipse."

Remove System.exit from traceview code running in Eclipse.

These are remnants of the standalone traceview version and they
can bring Eclipse down if the trace file can't be read.

Change-Id: Ic9a0171f307bc991cd000fa4d35d5c37b47e93b2

Merge "Fix locale handling of linear layout weights"

Fix locale handling of linear layout weights

Fix 18298: ADT r12 layout editor uses locale specific decimal
    separator leading to unparsable XML

Change-Id: Ideae319e41f8a7e34075065c63247d50cdf830c5

Change scheme for picking tablet or phone configurations

Change the code which automatically picks tablet configurations when
the target SDK is 11+ such that it only does this for API 11 and API
12.

Change-Id: Ic59cdd4d7ac2d616069bf79e517987e5ce2517cc

Add deletion hooks for view rules

This changeset adds a deletion hook to the view rule interface which
lets specific layouts perform cleanup when some of its children are
removed. This is vital for GridLayout where various rows and columns
need to be cleaned up.

In this changeset, this is used by RelativeLayout to remove constraint
references to widgets that have been deleted. Without this, you could
have Button2 reference Button1, then delete Button1, and when you drop
a new button, it will be assigned the now available name "Button1" and
the old constraint would kick back in!

Change-Id: Ic5829228cff38bfdc291988b1a2ef172e9aa4de5

Fix android.bat for Windows SDK.

android.bat was not properly copying swtmenbar.jar
to the temp folder, so it wasn't found at runtime.

Also added some debug info wrapping the menu bar usage
and a comment on how to remotely debug the swtmanager.

Change-Id: Ibe3d4ad0534b0e276808a77a051ae279a315da06

Merge "Fix some override issues."

Fix some override issues.

SDK tools must compile using Java 1.5

Change-Id: I39b708fcb6f1e225bece6b12c1b5909865825ac9

Merge "Integrate unknown attributes in the normal UiElementNode workflow."

Integrate unknown attributes in the normal UiElementNode workflow.

Somehow this should have been the default.
There's a few cases where this will allow us to simplify the code
and that will be for another CL.

Also fixes http://code.google.com/p/android/issues/detail?id=17762

Change-Id: Ieccd36f5f4042f414311f09339ed18fc73d7b122

Merge "SdkManager: pretty-print the 'android' command line help."

SdkManager: pretty-print the 'android' command line help.

This reflows the command line help so that it wraps nicely
at 78 chars. Also properly finish sentences with dots.
Mere details...

Also added a proper unit test to test various edge cases.

Change-Id: Id2f9eab5014ea9c5f4aae5ed4427b9aebc3c70bf

Merge "Switch to "android sdk" and "android avd" command line."

Merge "SdkMan2: Links to select new/updates or deselect all."

Merge changes I82eb0776,I72fdca12

* changes:
  Use GridData/LayoutBuilder in PackagesPage.
  SdkManager2: fixes for asynchronous sources UI.

Switch to "android sdk" and "android avd" command line.

The default with no argument is "android sdk".

Change-Id: I208d7b78dcbeab44a135e4d5bbb4b568b9ce09d7

SdkMan2: Links to select new/updates or deselect all.

Change-Id: I9c831138342574b0462e39055be86bde7114f54a

Use GridData/LayoutBuilder in PackagesPage.

Change-Id: I82eb07760e264884c510250245405aa2dfc1cdbe

SdkManager2: fixes for asynchronous sources UI.

- Properly refresh display list when installing/deleting
  a package.
- Gray install/delete buttons during an install/delete
  to avoid nesting operations.

Change-Id: I72fdca1252c447b046040afb70e67dfae77188e4