Fix undo problems in the XML editor (issue #15901)

We've had a recurring problem where in difficult-to-reproduce
circumstances the Undo button doesn't seem to work: After making some
edits like dropping a button, each press of the Undo button will each
individual edit of the compound operation, such as undoing each
attribute set, each element add, and so on -- just as if the entire
edit operation did not run under an undo lock.

This changeset fixes this issue - or at least one scenario of it
(tracked in http://code.google.com/p/android/issues/detail?id=15901 )

It turns out that Eclipse in some cases decides to "cancel" the current
Undo operation. This is done in reaction to selection changes, and the
logic for that is related to a user using a form editor to switch
between various attribute fields, and expecting each text field edit
to be separate. In the above issue I've listed a thread dump which
shows one such cancellation which is definitely not desirable.

To fix this bug we add a workaround: In our code which sets up the
undo recording, we call "beginRecording" -repeatedly- (say 4 times).
Since undo units are allowed to be nested, this means we're nesting
our own undos 4 times, and when we're finished with the editing
operation we simply call endRecording a corresponding number of
times. This means that when the offending code comes around and calls
StructuredTextUndoManager.forceEndOfPendingCommand it will subtract
the command count by one, but we'll still be in one of our nested
contexts (and the editor does allow extra endRecording calls to be
called).

Change-Id: Ic4dde029efcb18def3c9daa5d3676c49a815d804

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

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

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

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

Merge "Fix SDK Manager unit tests."

Fix SDK Manager unit tests.

These got broken after I've fixed the sort order
of extra packages in SDK Manager 2.

Change-Id: Ie18f3fb74f800ac855680be29a7932f8c8d9f54c

Merge "Fix instrumentation repetition parsing and multi listener handling."

Merge "Sdkman2: Load sources asynchronously."

Merge "Add a preference for the profiler buffer size."

Merge "Add support for tracking cpu and real time."

Fix instrumentation repetition parsing and multi listener handling.

Change-Id: Ib295bb21c45dd2f52f547f86a5b3c4aaa959c139

Merge "Add a basic properties dialog."

merge from tools_r12

Change-Id: I83911d9ee56e5d44045a5ecbd28647a551f8ea79

Merge "Guard against a possible NPE when the rendering fails." into tools_r12

Guard against a possible NPE when the rendering fails.

Change-Id: I8b0a0c65f2b0d7b7faf10224ac2945081605c5c4

Merge "pullFile exception handling."

Merge "Add a horizontal scrollbar."

Add a preference for the profiler buffer size.

8MB is getting to be too small for some purposes.

Change-Id: Ic711c426febf2e42634bd73a5be6211cfe66f4c0

Merge "Refactored MonkeyRunner to use ChimpChat"

Add support for tracking cpu and real time.

Enhanced TraceView to support extracting cpu and real time profile
data from traces and plotting them accordingly.

In contrast, the original thread-cpu algorithm incorrectly assumed
that threads could not execute in parallel so it would produce
inaccurate representations of the global execution timeline on SMP
systems.

To use this feature, Dalvik must be restarted with appropriate
profiling options.

Collect CPU time only (default).  Global timeline must be inferred.
  adb shell setprop dalvik.vm.extra-opts -Xprofile:threadcpuclock

Collect real time only.  CPU usage information is not available.
  adb shell setprop dalvik.vm.extra-opts -Xprofile:wallclock

Collect CPU time and real time information.  Most accurate but
requires a newer VM.
  adb shell setprop dalvik.vm.extra-opts -Xprofile:dualclock

TraceView remains compatible with traces gathered on older devices
and produces identical output when analyzing traces gathered with
the thread-cpu clock.

Added a feature that displays the exclusive and inclusive time
information while hovering over a method.

Added a feature that displays the time spent in a context switch
by a simple 1 pixel underline within the body of the method call.
This makes it much easier to identify long running methods that
spend most of their time blocked on a wait.

Fixed a bug in the way that the method call highlights were
displayed which could cause them to be truncated down to a
single-pixel width depending on the pixel alignment of the
start and end of each segment.

Change-Id: Id2e6d101d9c604a0029e0920127871f88013faa0

Sdkman2: Load sources asynchronously.

Change-Id: Ic79b987ce8594e7bfe30b32e961b9384e2538b6b

Add a basic properties dialog.

Shows some information about the trace that can be of interest,
such as which timing mode (thread time or wall time) was used.

Change-Id: I0f00b69dc5d3982f346cfc08e12eb5b379430a96

Add a horizontal scrollbar.

Change-Id: I077240805c45f71c8e270cada39bad3168061dfa

Merge "Sdkman2: Simplify packages names, fix extra package sort."

Merge "Sdkman2: Rename "Sort by Source" to "Sort by Repository""

Merge "SdkMan2: sort Tool category at the top."

Merge "Fix value parser to handle attr values not in declare-styleable."

Fix value parser to handle attr values not in declare-styleable.

Also change the layoutlib sample code to properly get the list
of framework attr flag/enum values.

Change-Id: Ie0bf126a0fab574d94d0f86b7b2f8581cf4eaae3

Sdkman2: Simplify packages names, fix extra package sort.

Simplify package names when sorted by API, that is
don't repeat "API nn" or "Android x.y" in the package
names if it matches to category.

Fix sorting of packages to properly sort extras
by vendor+path. The previous sort was broken as
it was making the revision number more important
than the vendor+path for extras.

Change-Id: I51ed0114ab9c77ef5d94710fc2449e5be9cd2ad9

pullFile exception handling.

Make SyncService.pullFile throw SyncException instead of IOException
if it cannot write to local file.

Bug 4600585

Change-Id: Ic3ab9fd7d1a05280ee14a310b9c8053371bb4e37

Sdkman2: Rename "Sort by Source" to "Sort by Repository"

Change-Id: I3f1f0f8781cae29400f7b49d2c92e6f2b83783c2

SdkMan2: sort Tool category at the top.

Change-Id: I8bc57a04a48cd540e6457a0b69b4911475989d8c

Merge "Sample code to use the layout rendering library."

Sample code to use the layout rendering library.

This is very basic sample code showing how to render
a layout.

This explains how to load the resources, create Folderconfig,
ResourceResolver, and how to call the LayoutLibrary to do an
actual render.

There are some big limitations:
- can't render custom views because there's nothing
  compiling them and generating the compiled R.class
  file.
- not all features of ADT are present because there
  are things that don't make sense outside of an editor
  (render in context, expand empty layouts, etc...)

Change-Id: I0c8676ebfbff27f0e9412bb4b13193ce64082372

Merge "Enable SdkManager2 and AvdManager1 windows in ADT."

Merge "SdkManager2: window can be closed during load."

Enable SdkManager2 and AvdManager1 windows in ADT.

Change-Id: I041fe4f6888dcc44f6f81319d05caed9311cc6ff

SdkManager2: window can be closed during load.

Since the sdkmanager2 loads its packages whilst
not blocking the UI, it's entirely possible for the
user to request the app to close whilst it load.
This makes sure we test for disposed widgets after
loading before trying to update them.

Change-Id: I9f0a82f0da5275996fd7275bd55de36986685dad

Merge "Move tools to 13."

Move tools to 13.

Change-Id: Ibedcb5627170d107cacf5e9d212f9da11c8d1f3e

Merge "Extracted ChimpChat from MonkeyRunner"

Refactored MonkeyRunner to use ChimpChat

Change-Id: Icbe5bcb066021d578faae43aa49b58ab1595870b

Extracted ChimpChat from MonkeyRunner

Extracted ChimpChat, the library for communication with Monkey from
MonkeyRunner

Change-Id: Ia9f966549d27abc9f494b2b001099d8130dea376

merge from tools_r12

Change-Id: Ic2a9844fe74da7c873c7264e38d50d568f190530

Merge "Merge bc434d73 from master to r12. do not merge." into tools_r12

Merge bc434d73 from master to r12. do not merge.

Add cpu.model to AVD running armeabi-v7a system images.

Change-Id: I2e94a3bed50936a39eebaffa014427d6ce40983d

Merge "Add cpu.model to AVD running armeabi-v7a system images."

Add cpu.model to AVD running armeabi-v7a system images.

Change-Id: I5b97b115ada432e284ad2dd3c69bcf7009677aec

merge from tools_r12

Change-Id: I60f32f625de3ea932c87747e25366377c07e74a5

Merge "Merge df07f53c from master to r12. do not merge." into tools_r12