Merge "Java editor save actions for SdkUiLib"

Merge "SDK Manager 2: details vs obsolete, better sort by API."

Merge "Turn off palette previews during unit tests"

Java editor save actions for SdkUiLib

- remove trailing white space
- remove unused NON-NLS tags

Change-Id: I5c02f7b1c46453badbb8d0ad5c8160f579097849

SDK Manager 2: details vs obsolete, better sort by API.

- Remove "details" and go back to the "obsolete" filter.

- Refactor: constants for all icons names.

- Rework sortByAPI to refresh in-place. This allows the table
  to preserve its state (expanded, selected, checked) when
  being refreshed. In-place not done for the sortBySource mode
  yet.

Change-Id: I81560091253f5a3250b7472d050ffa7e8b86fb88

Merge "Put some new icons."

Turn off palette previews during unit tests

Set the default palette mode to icon+text rather than preview during
unit test runs. Since each test run will create a blank workspace, the
previews have to be recomputed each time, which adds a couple of
seconds. Also, there are some problems on Windows when the previewing
code runs under unit tests, which this will work around.

Change-Id: Ic6a7bdaee731a219ff6042075977e12b0eeda111

Put some new icons.

- ddms and (one of) hierarchy viewer perspective.
- feature about for traceview/hierarchyviewer
- use the draw9patch icon for the resource view in ADT
- use the emulator icon for the emulator control view in DDMS
- use the new android icon for the ADT wizard.

Also standardized on <toolsname>-<size>.png for the icon names.
ADT uses android-##.png

Change-Id: I1ce2a0dae5d256049b381552bd89aef717f51da6

Merge "Reset sInitialized when calling AndroidDebugBridge.terminate()."

Merge "Sort elements alphabetically in code completion"

Merge "SDK Manager 2: Refactor window implementation."

Merge "Continuing UI for new SDK Manager Packages Page."

Merge "Remove obsolete emulator modules"

Remove obsolete emulator modules

The corresponding modules have all been moved to development/tools/emulator/system/
already. Note that we use a global variable Make trick to prevent the same module
from being built twice, so removing these files doesn't break the builds.

Another patch should remove the global variable Make trick from
development/tools/emulator/system after this patch.

Change-Id: I547d2a0173ddf58c1870784920ce795f6a0d7bcf

Reset sInitialized when calling AndroidDebugBridge.terminate().

This will allow calling init() again after a terminate().

Change-Id: Icdda999624f13485f419ae53bc2139a5ceaa3229

Sort elements alphabetically in code completion

Attributes were sorted, but not elements. Layout descriptors were
mostly sorted already, but with all views before viewgroups. The
manifest elements were in the order they must appear in the manifest
files.

Change-Id: I03e3fce032949f9df6ffc0ef96386b7458754270

Merge "Minor layoutlib api change for the data binding project callback."

Merge "Move getViewParent/Index from RenderSession to Bridge."

Merge "Fix hyperlinks plugin unit test on Windows."

Minor layoutlib api change for the data binding project callback.

Change-Id: Ib1b39938a459627a6ecf70556bdd6a99342ddf93

Fix hyperlinks plugin unit test on Windows.

Change-Id: I7c48ffe1beb511b0639ae90a26f1a8ee435fad47

Move getViewParent/Index from RenderSession to Bridge.

Also update the IProjectCallback to handle expandableListView.

Change-Id: I36ca8733182bbd9d169fee3e709ebc532fef7b1e

Merge "Make debug key expire in 30 years (instead of 1)."

Merge "New LayoutLib API to handle data binding in ExpandableListView."

New LayoutLib API to handle data binding in ExpandableListView.

Change-Id: Ic59acce7485887b4efde76697a85848e6bf91a97

SDK Manager 2: Refactor window implementation.

The previous patch based the implementation of the
new window on top of the old UpdaterWindowImpl.
Although that works since there are many things in
common, in practice the goal is to get rid of the
old one eventually so it's easier to not add a
dependency that needs to be removed later.

Change-Id: Ic1eac19dcc7e91d6750b7462eb2d28b447acad3a

Continuing UI for new SDK Manager Packages Page.

- Changed application to not display the "page selector" anymore.
  Instead there's only one page, which is is the available+installed
  packages combined page.

- The app behavior depends on the env var.
  - When set, it becomes the "SDK Manager" to only deal with packages.
  - When unset, it has the previous look & pages to deal with both
    AVD and SDK packages.
  - Eventually we'll also have an AVD separate app and we can
    factor out/cleanup the old window implementation.

- Integrated the progress bar + status in the page.
  - This isn't linked to Eclipse yet.
  - The ProgressTask* classes deal with the old behavior of
    a separate progress dialog.
  - The new ProgresView* classes deal with the new behavior
    of the progress status incorporated in the main page.

Other changes:
- Changed layout of the new package page
- Center add-on sites dialog on top of parent.
- Display "details" in tree view (e.g. includes obsolete
  packages + all archives)
- Plug in logic for install/delete buttons

Change-Id: I84c021cb7104681bb39445bd79ab01e161a22796

Merge "Fix layout action bar selection from outline"

Merge "Update changes.txt for ADT 10 and 10.0.1"

Merge "New application icons for the various SDK tools"

Make debug key expire in 30 years (instead of 1).

The 1 year expiration on the default debug key
is made annoying since we don't regenerate it
automatically when it expires.

Also added a simple unit test to check key creation
and expiration date.

SDK Bug: http://code.google.com/p/android/issues/detail?id=15370

Change-Id: Ie1ee14d8888275c2dae282bfb1235af54753ac0e

New application icons for the various SDK tools

This changeset integrates a new set of application icons by Roman
Nurik for the various SDK tools: DDMS, Hierarchyviewer, Traceview, the
SDK manager, and the Eclipse about box icon.

We have new icons with the same visual look for draw9patch and
emulator as well, which are not integrated here. Draw9patch is written
in Swing, so it doesn't have a way to set a dock icon on the Mac
(other than with a startup script modification to pass -Xdock:icon,
which means we also need to bundle the .icns image outside its
.jar). I couldn't find the place the emulator dock image is set so I
will ask around and file a separate CL for that.

Change-Id: Idbb2212b8d0d5f1bd2d41b3576079ff409c7e9c8

Merge "Fix crop tests on Windows (RGB endianess issue)."

Fix crop tests on Windows (RGB endianess issue).

Since endianess can change between an input and output
image in SwtUtils, it's important to not test pixels as
pure int values but instead use the correct RGB palette
mapping.

Change-Id: Icd3aad499a5cc069e8cc804af7511c95849a78a6

Merge "Store plugin tests results in a tmp sub folder."

Merge "Fix AttrParser tests on Windows."

Update changes.txt for ADT 10 and 10.0.1

Change-Id: I8810fd34dc10c4e73787857b078aeac83f0857b1

Fix layout action bar selection from outline

The layout action bar needs to update its selection when the selection
origin is the outline rather than the canvas.

Change-Id: Ibe66efc9f071934989dc4bde7bea0c8342c09ae8

Store plugin tests results in a tmp sub folder.

Change-Id: Ib52b2ae105225f3052cd46690d21a3f81be309d6

Merge "Zoom with scrollwheel in Traceview."

Merge "Added about files for Traceview plugin."

Merge "Changed bundle versions in some manifests."

Merge "Fix in MethodProfilingHandler."

Fix AttrParser tests on Windows.

Change-Id: I7c73b484db07aa06c6c4812672f88242b34c1e3e

Merge "LayoutLib API: update adapter binding callbacks."

LayoutLib API: update adapter binding callbacks.

Change-Id: I85be09fd7541da366c858ea04d7c2c4b53948475

Merge "Pick layout themes from manifest registrations and target SDK"

Pick layout themes from manifest registrations and target SDK

This changeset changes the way the layout editor picks themes when a
layout is opened. It used to just pick the first available theme in
the theme chooser.

Now it performs the following checks, in order:

* Does this layout have a persisted theme setting from a previous run
  of the tool? If yes, use it.
* Can the activity for this layout be determined? If so, look at the
  manifest registration for the corresponding activity, and if it
  specifies a theme, use it.
* If not, does the manifest specify a default theme for the project?
  If so, use it.
* If not, is the target SDK version (or the minimum SDK version, if
  the target is not specified) at least API level 11 or higher? If so,
  the default theme is "Theme.Holo"
* If not, the default theme is "Theme".
* If the file to be opened is included from some other layout, use
  the no-decorations versions of the default layouts, e.g.
  Theme.NoTitleBar or Theme.Holo.NoActionBar.
* At the end of this resolution, the computed theme is stored as the
  persisted theme setting for this layout, so the above algorithm will
  only be computed once. We might want to tweak this such that it
  distinguishes between a default computation of a theme and a manual
  user choice of a theme.
* If the file is opened as "Show Included In" (e.g. rendered within an
  outer file) then the theme chosen is the one for the outer file.

During startup, this information will be asked for each and every
layout being reopened, so there is now a "ManifestInfo" class attached
to the project which keeps manifest information around. It checks the
timestamp of the AndroidManifest.xml file and refreshes its
information lazily if necessary.

All themes mentioned in the manifest are listed in a special section
at the top of the theme chooser (below the entry for the default
computed described above).

The code to look up the associated Activity of a layout is using a
simple heuristic: it looks for usages of the corresponding R.layout
field, and prefers references from methods called onCreate or in
classes whose superclass name ends with Activity. I tried a different
approach where I searched for usages of Activity.setContentView(int)
but this failed to identify a number of cases where the activity was
doing some simple logic and didn't pass the layout id directly as a
parameter in setContentView, so I went back to the basic approach.

Change-Id: Ibd3c0f089fefe38e6e6c607d65524990699c86d3

Merge "Make code completion skip the layout_ prefix if necessary"

Merge "Add frequency count to Extract Style refactoring wizard"

Add frequency count to Extract Style refactoring wizard

The frequency count lists the number of attributes with a given value
out of the total number of elements, to help make it obvious if an
attribute is *not* specified for one or more elements. Without this,
only the elements with a value would be shown or counted.

Change-Id: I7866ea069090a766ab1f1cb4d7004d2763c59c5e

Merge "Dynamically generated resources id were not used in id resolution."

Merge "Add toString to ResourceReference"

Merge "Fix value of dynamic resource IDs used when rendering."

Dynamically generated resources id were not used in id resolution.

Dynamic IDs are generated when requesting an int value from a
(type,name) pair where type==ResourceType.ID and the name doesn't
reference any existing ID (This is possible when IDs are declared inline
and the layout file has not been saved, for instance, so the R class
has not been regenerated yet).

There is a reverse method querying for a (type,name) pair based on
an integer. This methods was not looking for generated id and would
return null instead of the name associated with the generated ID.

Change-Id: I0bc4bc76a5157d56d06c5f4538c4fcc38f1bded7

Add toString to ResourceReference

Change-Id: I9ea9625a839a4dc2c9d03baa5b0d56ddc7f198ee

Fix value of dynamic resource IDs used when rendering.

Change-Id: Ic6c0fad53d06e48dcd8ab59c5aa0c2a530e36767

Make code completion skip the layout_ prefix if necessary

The code completion prefix matcher already lets you skip the
namespace, so you can type "width" and it will expand to
"android:width".

This changeset adds "layout_" to this scheme, so you can type
"margin", and this will match "layout_margin" as well.

Change-Id: I8bf71e91ac8ac71bd2cb0308c6f1d39a9dfaa66a

Merge "Improve painting of combined selection and hover"

Merge "Bundle ninepatch.jar inside ninepatch-tests.jar"

Improve painting of combined selection and hover

When the mouse is over a rectangle, we highlight this "hover"
rectangle by painting a semitranslucent white rectangle on top of it.

When a view is selected, we highlight it by painting a semitranslucent
blue rectangle over it.

This means that if you move the mouse over the selection, you get both
of these effects added together, which dilutes the underlying selected
widget too much.

This changeset tries to improve this situation by defining a different
visual style to be used for the combination of hover and selection,
where the opacity is much lower for the hover in this case.

This changeset also reduces the existing hover by about 30% opacity.

Change-Id: I63ffe8a9d756dcae29b2009a1a1cd6b9ffb6fbe7

Merge "Update test to track layoutlib change"

Update test to track layoutlib change

Change-Id: Ie4ccd0d05a85ebbb03fefb9ba1e31dfbde1a89be

Merge "Add a way to get the adapter binding through the project callback."

Add a way to get the adapter binding through the project callback.

Change-Id: I0de48e3519f38a63c3462f5eb0c7f6c2a29d0e6f

Merge "Improve view cookie handling"

Improve view cookie handling

The layout editor is passed a ViewInfo hierarchy by the layout
library. For older versions of the layout library, it can be handed
hierarchies where the view cookies (which point back to XML model
objects corresponding to the rendered views) that are missing or
ambiguous. For that reason, it has various algorithms to try to piece
things back together, and for example handle <merge> scenarios as best
it can.

This isn't necessary with layout lib version 5 and higher, since as of
version 5 these scenarios are handled on the layout lib side and the
layout editor is passed back special cookies like the MergeCookie to
properly handle the various scenarios.

This fix makes the layout editor look up the layoutlib version, and if
dealing with version 5 or higher, it takes a simpler path to build up
the hierarchy.

This is also necessary to deal with the latest version of layoutlib
which passes a new type of view cookie which the older algorithm
couldn't handle.

Change-Id: I98c3ba5d17ad9d639eb118e4709c0b6bbf815b0a

Merge "Extract Style Refactoring"

Merge "Add initial and final MOVE events on drag."

Extract Style Refactoring

Adds a new refactoring, "Extract Style", which will show the user the
attributes for the current selected elements (or if invoked from an
editor context, the attributes overlapping the current caret or editor
selection). The user can select which attributes to extract, and these
are then added as a new style in the styles.xml file in the project
(which is created if necessary). The user can optionally replace the
attributes that were extracted, and the user can also optionally set
the style attribute on the elements to the new style. (Both are on by
default.) This is integrated with the refactoring quick assistant as
well.

Change-Id: I0504e86a824b00730482607150a879ff28233618

Add initial and final MOVE events on drag.

Change-Id: I88dfc808f34a862941640ad8c9ddd49051e85f72

Merge "Make sure files opened from XML wizard go to the WYSIWYG editor"

Merge "Auto-zoom views the first time they are opened"

Merge "Add repeat count config to layoutlib_api's AdapterBinding."

Auto-zoom views the first time they are opened

Now that we default to the tablet screen size, creating a new layout
usually opens up a view which only shows the top left part of the
design view, and horizontal and vertical scrollbars.

This changeset adds auto-zoom such that the first time a view is
opened, the view also auto-zooms. Note that the type of zooming
performed is zoom-out-only; it never zooms in, so a small phone layout
will continue to be shown at 100%.  We already perform auto-zoom in
several other scenarios (such as changing orientation).

Change-Id: I1e9a9e8357c952f84b541ea799c529d5dc4b3239

Make sure files opened from XML wizard go to the WYSIWYG editor

Change-Id: I51a0d96564b5fa896eb4daef50f193df7cadfa20

Add repeat count config to layoutlib_api's AdapterBinding.

Change-Id: Idbb54de6dee6d6b466476a0e3acdd87dd394ee42

Merge "Add support for data binding in the layout editor."

Add support for data binding in the layout editor.

The Layoutlib_api is changed to allow passing information regarding
adapter content, and querying eclipse to fill the items.

Change-Id: Ie5a047ab9cd0ed7677c13309d95663eae462c3e7

Merge "Test infrastructure fixes"

Test infrastructure fixes

Add methods which allow a test case to specify that it needs a unique
project for the testcase (or even for each individual test
method). This allows tests which modify the project (such as the
quickfix-resource-creation tests) to not cause side effects that
affect other tests (e.g. resources created by the quickfix could show
up in code completion results by the code completion tests, which
meant the goldenfiles would have to either include them or exclude
them, which meant the tests must always be run together or never run
together.

The changeset also adds an environment variable pointing to the git
working copy allowing tests with no golden file to create the golden
files directly. This makes it easy to add new test cases; just write
the test scenarios, run them, which will fail all the tests but also
write out the expected golden files. Verify them and check them in and
running the tests again should pass if the output is stable.

Change-Id: I2e2789c00c84a31a9fbc603851879d8d83342ad3

Merge "15736: Eclipse crashes in layout editor when browsing element's properties"

Merge "Code completion in style definitions & other improvements"

Merge "Better default selection when choosing a device config."

15736: Eclipse crashes in layout editor when browsing element's properties

This changeset attempts to fix issue 15736. The root bug is an SWT bug
which can cause a JVM crash in some scenarios. However, it seems to be
triggered by code which performs a gradient paint in the property
sheet (which is in fact done to work around another SWT bug...).

Using a gradient is not necessary - a plain background paint will do -
so this changeset simplifies the code to only fill the background.  It
also leaves the selection flag on the event alone such that other
parts of the list (such as category items) are not affected.

Change-Id: Ica32fe1e6f49f1e4154c3ffad9cae9799fcc4723

Bundle ninepatch.jar inside ninepatch-tests.jar

Change-Id: Ie659cfdc1bce12f51aca4a636ea117142f8c4be1

Code completion in style definitions & other improvements

This changeset adds code completion support to resource files where
<style> elements are defined. In particular, you get two types of
completion:
- Completion on names:  <style><item name="|" >
  Here you get offered ALL matching attribute names from any view
  descriptors.

- Completion on values: <style><item name="android:gravity">|</item>
  Here you get offered all values that correspond to the attribute
  named as the value of the name attribute of the surrounding item
  element.

Unrelated to styles, code completion will now also move the caret
outside the attribute value quotes when you enter one of the value
alternatives (unless the value is a flag type or a resource token).

Also, run-on attributes are also handled more gracefully: if you
complete properties with the caret immediately following another
attribute, a separating space is added into the match as a prefix such
that the new attribute is separated and the XML valid.

There are a lot of diffs in this changeset. In order to allow the
ResourcesContentAssist to extend code completion to add its style-
specific logic, the content assist code had to be split up a bit.
Doing completion in text nodes, or potential text nodes (e.g. if you
have <item>|</item> you should get text completion at that caret
offset even though there is no text node child of the item element
until you insert actual text there) also required some changes.

There are many new unit tests. I also changed the golden file format
slightly to show the caret in the before-view such that it's more
obvious how the caret was affected by inserting the new item.

Change-Id: Ia985db7e848fea33e93ac8a70b2c3b4dbfd9056a

Merge "Handle AAPT ranges in value resource files"

Handle AAPT ranges in value resource files

The earlier fix to add editor column ranges for AAPT errors handled
the types of errors that occur in layout .xml files. In value
definition files there are several additional error messages that can
be parsed to infer the actual error range on the line.  This changeset
adds handling for these additional cases, plus various unit tests.

It also fixes a couple of corner cases in layout files - in
particular, the case where a resource value is missing, and the case
where there are other -valid- uses of the same value earlier on the
same line where the first reference would be highlighted rather than
the one following the property in error.

Change-Id: I02906ada43a884d7518348924715892de9d33311

Merge "Add more sdk lib tests to sdk build."

Add more sdk lib tests to sdk build.

Change-Id: I09946be68f244922de0ad89d970e53165957fa2b

Better default selection when choosing a device config.

Change-Id: Iacaabec4ad9e9dd55397dc2cc3e6bfadc0c11ff2

Merge "Revert "build/tools.atree: Move x86-specific changes to tools.x86.atree""

Revert "build/tools.atree: Move x86-specific changes to tools.x86.atree"

This reverts commit 253acabecba0d9f649f04499f6dda980fbe7bf49.

Merge "Revert "Fix WIN SDK build: don't break if emulator-x86 is missing.""

Merge "Added a New Folder action in File Explorer."

Merge "Add ddmlib-tests to sdk build in a tests/libtests dir."

Add ddmlib-tests to sdk build in a tests/libtests dir.

Change-Id: Idba9f5b5724926209bbb96bfd75e04dbffa3760c

Revert "Fix WIN SDK build: don't break if emulator-x86 is missing."

This reverts commit ecd2f055e09ebe5a16244b91af1c6f80cae671a0.

Merge "Fix WIN SDK build: don't break if emulator-x86 is missing."