Open SDK stats dialog from an existing shell.

This patch addresses Issue #15267. The primary issue is that the SDK
stats permission dialog was opened from a separate Job, and this
dialog shows up when the user opens the preference page for the
 first time. Since both of them happen to be modal dialogs, the
behavior is inconsistent, and many times results in a blocked UI.

The patch fixes this issue by opening up the stats dialog as a child
of the preference page dialog within ADT, and as a child of a new
shell within DDMS.

Change-Id: I8c9ed9e9bbfac855435690f287b4f60975f336fb

Merge "Move the project conversion of bin location from callback to build."

Merge "Change how to find adb."

Merge "Fix CastClassException by using the right class."

Merge "Change way we report Eclipse version in stat ping."

Change how to find adb.

This method is aligned with how hierarchyviewer2 finds adb.

Change-Id: I56a5a12e3e6ff592748a3afee18299d16066a537

Fix CastClassException by using the right class.

Change-Id: I691ded197cfb47acfa1da12f48f6d7e50254bbc9

Merge "Make variable/method names consistent."

Change way we report Eclipse version in stat ping.

This reverts the way Change I14dba0dd was sending
the Eclipse version. Instead of passing a new attribute,
we reuse the existing "app=>version" format, with a
specific app name of "eclipse". Since versions are
reformated in 4 parts, it will report something like
"3.5.0.0" (only major+minor and 2 zero sub parts).
We don't send more details than that (e.g. Milestones).

Change-Id: Id01e3ed6b96ea4ce4e1b4f643d08a050291a6e7b

Merge "Add Unwrap Refactoring"

Add Unwrap Refactoring

This changeset adds a new visual refactoring: "Remove Container". This
is basically the reverse of the "Wrap In" refactoring (which surrounds
the selected views with a new parent layout). Instead it takes a
parent and removes it, in the process transferring the namespace
declaration (if a root element) and any layout params to the children.

Change-Id: Id535b88fcf39a8b7a83fd6feb981fff000140e8a

Merge "Layout editor property menu improvements"

Layout editor property menu improvements

This changeset adds two forms of view attribute metadata:
* First, it records the most commonly used attributes for each
  view. This was determined by gathering statistics on as many layout
  files as I could find and then picking those that are used 10% or
  more.

* Second, it records in the attribute metadata which View defines a
  given attribute.

The context menu uses the above information to present the available
attributes in several ways:

* In the top level menu, where we had "Edit ID", and if applicable
  "Edit Text", it now lists the top attributes instead. For example,
  for a RatingBar the first handful of menu options are "Edit ID...",
  "Edit NumStars...", "Edit StepSize...", "Edit Style..." and
  "IsIndicator" (a boolean pull-right menu).

  Incidentally this automatically handles some cases which were
  manually handled before, so the code in LinearLayoutRule to add an
  "Orientation" menu is no longer needed; it's just one of the two
  common attributes handled by the new attribute list.

* The "Properties" menu is now called "Other Properties", and instead
  of showing all properties, it has a new level of menus:

  * "Recent". This is initially empty, but as you edit other attributes,
    it gets populated (in most recently used order, kept up to date)
    with recently edited properties.

  * One submenu for each defining View super class listing exactly
    the attributes defined by that view. This is useful for browsing
    and editing related attributes. If you are looking at a textual
    view like a Button for example, you can look at the "TextView"
    menu to find all the text related options (TextColor, TextSize,
    etc).  These menus are listed from the nearest to the further
    superclass, so for example if you right click on a CalendarView
    you'll see these menus:

       Recent                     >
       ----------------------------
       Defined by CalendarView    >
       Inherited from FrameLayout >
       Inherited from ViewGroup   >
       Inherited from View        >
       ----------------------------
       Layout Parameters          >
       ----------------------------
       All By Name                >

  * As you can see from the above, there are two more menus below the
    inherited menu items. "Layout Parameters" lists all the layout
    parameters available for the selected nodes (which is defined not
    by the view itself but the view that it is contained within).  And
    finally there is "All By Name", which is a complete menu
    containing all available attributes for the view (and this is what
    the Properties menu used to contain).

* The code which computes a display name from an attribute was also
  tweaked to capitalize not just the first letter but any first word
  letter, so for example when you look at the possible values for
  Gravity you now see "Clip Vertical" instead of "Clip vertical".

* The edit property dialog for the properties menus now uses @string
  or @style resource choosers for the text, hint and style attributes
  (used to just be a plain text box.)

Change-Id: I3b30d48b85fd13f0190c760756bf383a47b3f4a5

Merge "Fix and turn on the auto-format XML option"

Fix and turn on the auto-format XML option

This changeset turns on the option to "Automatically format the XML
edited by the visual layout editor" by default.

It also fixes several issues related to it:

- First, the auto-format code was previously hooked up to synchronous
  insert and delete node events, which meant during a larger edit it
  would apply formatting repeatedly and before the edit was complete
  (which made the DOM model keep reparsing to update document based
  back into its internal model).

- Another problem was that the auto format only applied to added and
  removed nodes; attribute edits were not considered (which made sense
  given that the old formatter did not support reordering of
  attributes, but now that it does attribute changes requires
  reformatting as well.)

- The old reformatter was trying to work with the Eclipse XML
  formatter, which does not handle partial document formats well
  (there were a bunch of workarounds, and even with those there were
  some problems). The new formatter does not, so I've changed the
  option to be tied to the custom formatter; the automatic formatting
  of layout editor changes is now enabled only when the custom editor
  is enabled.

  This also takes advantage of the new formatter's ability to only
  format the attributes section of a document, so if you just tweak
  the attribute of a node, the children's formatting is not affected.

- The formatter would apply to more than just the layout editor; any
  UI editor which used UiElementNodes would be affected. I've added a
  method, by default false, where each UI editor can opt into this.
  I'd like to get Raphael's feedback on the manifest editor before
  possibly adding it for that editor as well (and updating the option
  name to something more generic).

- The mechanism for ignoring XML document changes (already used for
  refactoring) is now also connected to reformatting, so when the
  formatter runs it sets an "ignore" flag such that there is no
  second XML model loading when the changes are just as a result
  of a formatting option.

This changeset also tweaks the formatting behavior of comments;
instead of -always- adding a blank line before line comments (other
than suffix comments), it now considers the previous document and
preserves the spacing used there (though it will collapse multiple
lines into one).

Change-Id: I04c2b80836c4d9874dca789c2f2cbc8c8c7cfa76

Make variable/method names consistent.

Always use mixed case for pid: change PID to Pid.
Rename getPidString() to getPid().
All changes are syntactical only..

Change-Id: I85309d46ff1a14510271f6402debdc1e17747928

Merge "Add support for filtering by application name."

Merge "Store source application name in logcat messages."

Merge "Avoid multiple R.java parsings"

Avoid multiple R.java parsings

This change lets CompiledResourceMonitor avoid multiple
parsings of the same R.java file when reacting to a compilation
of a project that has libraries by adding a better filtering
check for which file changes it reacts to.

Change-Id: I611c581200df3237217f8426d976c859d8848ade

Add support for filtering by application name.

Change-Id: I07be20ac89108c181cfb2793c90f5cf697a304e2

Store source application name in logcat messages.

logcat only provides the pid of the application that generated the
message. Map the pid back to the application's name, and store it
in the parsed logcat message.

Modify the logcat view to display the app name in addition to the pid.

Change-Id: I939ed9ca2389412473dc614bd2e0e1d302295924

Merge "Dispose of font properly."

Dispose of font properly.

Change-Id: Ic00711ebfe13c9f4674e0570bd3dd28f35127d62

Merge "Add support for Select All and Copy to Clipboard actions."

Merge "Wrap messages based on font size and col width."

Merge "Make smart indent handle whitespace indentation better"

Make smart indent handle whitespace indentation better

The smart indent implementation handles indenting within element
content (e.g.  between tag names and attributes) and at element
boundaries (e.g. before and after <>'s.) For other parts of the
document it just defaulted to the normal XML smart indent.

However, this includes handling newlines within blank space.  This
changeset adds special handling for some key scenarios:

Indent the newly inserted line depending on the surrounding context:

<foo>                         <foo>
|                 ====>
</foo>                            |
                              </foo>

Also subtract whitespace on the right hand side of the cursor when
necessary:

<foo>                         <foo>
|    <bar/>       ====>
</foo>                            |</bar>
                              </foo>

There are some other cases as well, described in the unit tests.

Change-Id: I8e679800fe56cae598f430b6f6de409309407708

Merge "Fix failure when non-ASCII characters encountered."

Merge "Fix issue 19057: Drag & drop within canvas broken on Linux"

Fix issue 19057: Drag & drop within canvas broken on Linux

This changeset fixes
  http://code.google.com/p/android/issues/detail?id=19057

by working around the fact that on some Linux/SWT versions, mouse up
events are delivered during a drag and drop. These events would
interfere with the normal drag & drop machinery. Luckily we can tell
whether or not we're in the middle of a drag & drop operation, and if
so ignore the mouse up event completely.

Change-Id: I183e0b5e360865724b87cc200e1899a2b31cc2ef

Merge "ADT: Fix issue 18689 (can't edit/save minSdkVersion)"

Merge "SDK: Expose Layoutlib Version through IAndroidTarget."

Move the project conversion of bin location from callback to build.

It's safer to do it as the first step of the build than in the callback
for opened project callback as there's restriction on what we can do in
the callback.

Change-Id: I2ad819975418180bb70d5613a7b5e923e7f55ec8

Merge "Don't regenerate R.java if a non xml resource is modified."

Add support for Select All and Copy to Clipboard actions.

Change-Id: Idbf3bc093f2a049b5e0e0cd9605a6f8bdc8f7723

Wrap messages based on font size and col width.

Rather than wrapping messages at a certain size, detect the current
width of the column, and wrap them if the message will exceed that
width. Note that the width depends on the font used.

Change-Id: I26441c5b4b7fe3e6ab00b9287c4013493e2352f8

SDK: Expose Layoutlib Version through IAndroidTarget.

This changes PlatformTarget to parse the Layoutlib Version
(API + revision) stored by the SDK Manager in the source.properties
of the platform on disk. Then this introduces an helper method
in SdkManager to find the highest available layoutlib version.

Change-Id: I94990dabed6c7e5ad3f8f06ff3b16696edca8653

Merge "Add flag to disable profiling"

Add flag to disable profiling

Start hierarchyviewer1 with -profiling=false to disable profiling.

Change-Id: Id73022e19d40090a16f850ca20c18b8a2cd59644

Don't regenerate R.java if a non xml resource is modified.

Change-Id: Ie28359234cbb56e14c42274434831e4795bf3ee3

Merge changes I9bd7e20b,I41d6aef9

* changes:
  Added extension specification to dependency check
  Change filename for dependency files

Added extension specification to dependency check

This change adds a parameter 'touchedextensions' that can be qualified
with a colon-separated list of extension to consider during checks to see
if prerequisite files have been modified.
if this parameter is not set, files of all extensions will be checked
to see if they have been modified.

Change-Id: I9bd7e20b6e5341b3a0f3698ffdc2de10d286921c

Change filename for dependency files

This change updates the filenames for the dependency files to check
in Ant to match the new names of the generated files by AAPT.
(R.java.d and <AppName>.ap_.d respectively).

Change-Id: I41d6aef96aec42c8074aa921801482e4c70363b6

Merge "Precompilation only executes AAPT when necessary"

Precompilation only executes AAPT when necessary

This change adds resource tracking to the ResourceManager.
Each ResourceRepository now has new methods:

void markForIdRefresh() to set the repository as "dirty"
boolean needsIdRefresh() to check whether the repository is dirty
void setIdsRefreshed() to set the repository as "clean"

During the precompilation step, the PreCompiler will query the
ResourceManager to see if any of the repositories included in the build
are marked as dirty. AAPT will only be run if one or more dirty repositories
are found.

Repositories are marked as clean when R.java is regenerated and IDs are
set in ProjectResources.

Change-Id: I575ab819702508eacd247b282c3de8979f2f0ab9

Merge "Use Eclipse XML parser instead of JDK parser when formatting"

Merge "Add ability to pass options to IDevice.installPackage."

ADT: Fix issue 18689 (can't edit/save minSdkVersion)

This actually fixes the issue directly, not by a side effect.
For some weird reason, I was always creating UI nodes for
unknown attributes with a setDirty(true). That's obviously
wrong, the attribute should not be set dirty when created
but only when a value is assigned to it, if needed.

Change-Id: I4d4b6dbbc0ab7e837a772994b3e6bf6c2236de65

Use Eclipse XML parser instead of JDK parser when formatting

This changeset adds a utility method which parses an XML string using
the Eclipse structured model instead of the JDK parser. The resulting
document has some extra metadata which the XML formatter can use. In
particular, it distinguishes between empty tags and tags without
children.

Now that this is fixed the formatting on project creation code
includes manifest files, which I excluded previously because it made
the <uses-sdk> element look ugly.

Change-Id: I7069fa43df33d874303435b3625f1c7b20f5b3d3

Merge "Remove from tools.atree a file I just removed."

Remove from tools.atree a file I just removed.

Change-Id: I181655095b2562cf8ddca81678d907ef9233c9bc

Merge "Fix issues related to the new output folder for ADT and JDT."

Merge "Remove some more unused code."

Fix issues related to the new output folder for ADT and JDT.

The android output folder is hard-coded to bin/ for now
The Java output folder is hard-coded to bin/classes.

Projects that are opened and that don't have that exact setup
are converted.

Also make sure bin/res is created when the builders are run.

Change-Id: Id5de887710ed951b53218a73e5962a9b079cb176

Merge "Use PreferenceConverter to set default font."

Use PreferenceConverter to set default font.

Also use platform specific font.

Change-Id: I0bc50c2c79c8e0e21c779ad71f3b11b8c03e99d8

Remove some more unused code.

Change-Id: I6aa1c4f0158fc86d85d922e33dd5bf47de6f377d

Merge "Fix issue setting font, and invalid selection on Mac."

Fix issue setting font, and invalid selection on Mac.

Font: fontdata should be created using PreferenceConverter.getFontData
List selection: On Mac, selecting a list widget outside the list contents
behaves differently than on Linux

Change-Id: I696996c8b92af8211dd767f5dd1552c34b68c327

Merge "Fix missing import"

Merge "Remove obsolete code."

Merge "Package in tools/lib all the new libs required by the new SDKManager."

Fix missing import

Change-Id: I52baf6cf59ff5ab73ad5251bfb6f98bbdadadf15

Remove obsolete code.

Change-Id: I2881d7b5f01229b9da81c6134fc5dfdfb4aacff8

Merge "Rename main_rules.xml to build.xml"

Merge "New library project mechanism."

Rename main_rules.xml to build.xml

This is the final name. I kept main_rules to make the previous
review easier.

Change-Id: I3dfcd36a49f2893665e712ed9391acb4486d713e

Package in tools/lib all the new libs required by the new SDKManager.

Change-Id: If430e817dc82b361c275a98ee38ecbc38348e1ce

New library project mechanism.

Libraries now generate a jar file that is used by the projects
referencing them, instead of having the main projects compile
the library source code themselves.

This means we can remove the link mechanism that created linked
folder in ADT and instead use a container that is lazily initalized
to be all the jar files of the libraries.

Also merged all 3 Ant build files (main_rules, lib_rules, test_rules)
into a single build.xml that can build any kind of projects.
Lot's of improvement in there too. Much cleaner.

Change-Id: I98307e25cd76722e8595938528e6ef57a7e226ad

Fix failure when non-ASCII characters encountered.

Change-Id: I42405b0b1ceb576ae12330e487f0f7dee870e527

Merge "Make new logcat view to be the default in DDMS."

Merge "Goto source when user double clicks on stack trace."

Merge "Add new logcat view to Eclipse."

Merge "Add missing qualifier icons"

Make new logcat view to be the default in DDMS.

The old view can be activated by setting the env variable
ANDROID_USE_OLD_LOGCAT_VIEW.

Change-Id: I006aaba26f323b12b6da70f22f8584ad76fb5041

Goto source when user double clicks on stack trace.

Change-Id: Ia2693424f0106fb3aa1724c211a8e1273f3a0c28

Add new logcat view to Eclipse.

The existing logcat view has been renamed to OldLogCatView. It is
still accessible from the Views menu with the title:
"LogCat (deprecated)".

The new view is the default. The only missing feature compared to
the old view is the lack of double click handler. Subsequent CL's
add this feature.

Change-Id: If8ce594539f3f82c0c47ebb2d46159d6feed5a21

Merge "Use equals for value comparison"

Add missing qualifier icons

The Available Width, Available Height and Smallest Width qualifiers
were missing icons (which are shown in for exmaple the Create
Configuration dialog).  This changeset adds icons for these (which
also makes ResourceHelper.testGetIcon pass.)

Change-Id: I42c4e357245adc0b80320bac5fe1e173d1d0eaf3

Use equals for value comparison

Change-Id: Ic83b797567a044ce08703d036329299ef64bcdea

Add ability to pass options to IDevice.installPackage.

Change-Id: I79328a3ab0c3e113103615b11d9ec85726513adc

Merge "Pull View Rules API into a separate library"

Merge "Show count of unread messages associated with each filter."

Show count of unread messages associated with each filter.

Change-Id: I871ddc550b1d63844b8501a59c079b62f0de4cca

Merge "Refactor code for filtering into LogCatFilter class."

Merge "Add list<received messages> parameter to msgReceived event."

Merge "Save/Restore user created filters."

Merge "Minor refactor: use length() > 0 rather than !equals("")"

Refactor code for filtering into LogCatFilter class.

- Move the code out of LogCatViewerFilter into LogCatFilterSettings
- Rename LogCatFilterSettings to LogCatFilter as it now does the
filtering as well.
- Update tests appropriately.

Change-Id: I68840d8292cc7ad2b3ef5415bd7a0bdd56aa4da2

Add list<received messages> parameter to msgReceived event.

Change-Id: Ib66700806853564901f177694308add9a8d9d2d7

Save/Restore user created filters.

Change-Id: I4e5ba3d6c16889b0a1ac3195e616f0e53026f947

Minor refactor: use length() > 0 rather than !equals("")

Change-Id: Iec62efdd56bbb24fa1c473ccb74e529271f2dd7b

Merge "ADT: Fix manifest editor marking files as dirty on open."

ADT: Fix manifest editor marking files as dirty on open.

Originally this was supposed to be about fixing
issue 18689 (can't edit/save minSdkVersion) but I
can't repro that anymore, so instead I focused on
fixing the marked-dirty not going away. That one is
due to the manifest descriptor lacking the XMLNS
for "android". Upon loading a custom hidden attribute
was created and marked as dirty, which is wrong.

Change-Id: I9f0d3809632fdbeef8c8a39f79c62914818b1efe

Pull View Rules API into a separate library

This changeset pulls out the API classes from the plugin sources and
into a separate standalone .jar library. The library depends on the
common.jar library.

With the separate view API it should be possible to build designtime
helpers (view rules) for custom views to improve editing behavior in
the layout editor.

Change-Id: I20bb511668de2fe52910e5fe0bbd3ec2a18b5a08

Merge "Action API improvements"

Merge "Expose version.name property for aapt task."

Expose version.name property for aapt task.

The new property makes it possible to create ant build scripts
that programmatically determine and set the versionName attribute
during build.  Similar property already exists for versionCode.

Signed-off-by: Tero Saarni <tero.saarni@gmail.com>
Change-Id: Ia36632ae029785563fc7c1bc8937034963016852

Action API improvements

This changeset changes the Actions mechanism for view rules to add
support for the following:

* Delayed computation of submenu contents. Before this, a view rule
  would have to produce the full tree of actions to be shown in menus
  and submenus - for example including all the properties, and in turn
  all the enumerated values for those properties and so on.  Now
  there's a Provider interface which can be used to compute these menu
  items only when the menu is actually opened. The properties menu now
  takes advantage of this.

  This was also necessary to implement the following new feature:

  * The layout editor context menu now also lists not just the
    properties for the currently selected views, but also the
    properties for the parents. For example, if you open the context
    menu, you'll see the properties for the button you just right
    clicked on, but there will also be a "frameLayout1" submenu
    containing the actions for the <FrameLayout> parent of the button,
    and a "linearLayout1" submenu for the parent linear layout. This
    is useful when a parent layout doesn't have blank space on its own
    so it is difficult to target.

  A future CL will use the lazy initialization to add more options to
  the properties menu.

* Support for arbitrary nesting. Submenus can contain submenus can
  contain other submenus etc.

* Custom ordering. This changeset moves the "sort priority" concept
  (which was already used for layout actions) up to all actions, which
  makes it easier for rules to cooperate on ordering because instead
  of appending or prepending to the superclass' context menu result,
  actions can now just be initialized with a sorting priority value
  which makes it trivial to interleave actions regardless of who adds
  them. This also makes it a lot easier to use custom ordering in
  choice menus where the ordering used to be alphabetically sorting on
  keys.

* Improved support for multiselection. The callback interface now
  takes a list of nodes to apply the callback to, and actions can
  indicate whether they support multiple nodes. This makes it possible
  for actions to more directly support the case where you apply an
  action to multiple nodes. As before, the available actions in the
  context menu is limited such that it only shows the actions common
  to all. But now those actions can do something specific.  For
  example, if you select "Edit Text..." on many nodes, you will get
  the input-string dialog once, and then the value is applied to
  all. Similarly, if you select "Edit Id..." it will ask for a
  separate id for each value (and you can cancel out of this loop).

There are various API changes too. Since the Choices action (which had
a map-based set of values) was removed, the OrderedChoices is now
renamed Choices. The Actions subclass of MenuAction which all actions
also extended has simply been moved up to the top level MenuAction.
And MenuAction has been renamed to RuleActions since they are used
not just for menus but for toolbars etc and the key thing about this
interface is that they are intended for use by rules.
Change-Id: If49f75213f2041ebfef7e84254d70d219bb766ab