Merge "Refactoring quick fix adjustments to caret and selection handling"

Refactoring quick fix adjustments to caret and selection handling

This changeset changes the logic in the refactoring quickfix (which
adds the Android refactorings based on the current caret position,
lexical context and selection).

It now allows some of the refactorings to be run when there is a
selection (this fixes "20393: Extract string functionality in Android
XML files"), and it makes other refactorings work when there is no
selection (it implicitly selects the surrounding element).

It also ensures that the Extract Style refactoring won't be listed at
the top if the attribute under the cursor is not a stylable attribute,
and it makes some other adjustments to the proposal order.
Change-Id: I0714bb652032aa19e6c2502dc1af7bb5e4adf308

am 8f2b9fbd: (-s ours) Provide a way to show/hide the filters column. Do not merge.

* commit '8f2b9fbd5f8f298a05f09e0b523c4824e5da0e29':
  Provide a way to show/hide the filters column. Do not merge.

Provide a way to show/hide the filters column. Do not merge.

(cherry picked from commit c9192e375844e45ccb74ee69150717a31385564f)

Change-Id: Iadfc8be7d20a85c79ff94cf62f14345392cf999b

am bd79a007: (-s ours) SDK: fix missing \n in \'android create avd\'. Do not merge.

* commit 'bd79a007af08ce3b4e41c6602cf7b8bf945c6fa8':
  SDK: fix missing \n in 'android create avd'. Do not merge.

SDK: fix missing \n in 'android create avd'. Do not merge.

SDK Bug: 20430

(cherry picked from commit c649a334d7bc656ea3c00cd9d3d102c88fbf8f80)

Change-Id: I8b4370b0d4ec4b96e9b65c214a9e06506b615a41

Merge "SDK: fix missing \n in 'android create avd'"

SDK: fix missing \n in 'android create avd'

SDK Bug: 20430

Change-Id: Id427390c799165a1dff07530c6fce48488193055

Merge "Create FileListingService#getChildrenSync"

Provide a way to show/hide the filters column.

Change-Id: I7c94daa006987a0983cdb3cee02a35f6e678d86e

Create FileListingService#getChildrenSync

Useful for cases where callers want to be informed of adb communication errors
when retrieving child files.

Bug 5392097

Change-Id: I82087d24d08f4e1b2fcda67156835367dc1bb824

am 3f05dce5: (-s ours) Cherrypick fcd339 from  do not merge.

* commit '3f05dce571bfedfdabf6b1f7d8f8a1c05c62317e':
  Cherrypick fcd339 from master. do not merge.

am 0f2c3056: (-s ours) Cherrypick d7b602 from  do not merge.

* commit '0f2c305640a83b480691dc9002eb2ff5c808dff2':
  Cherrypick d7b602 from master. do not merge.

am 8a061633: (-s ours) Cherrypick 4cd282 from  do not merge.

* commit '8a06163380b3df70071df08a17f2f8c2aa77513c':
  Cherrypick 4cd282 from master. do not merge.

am fdef32c0: (-s ours) Cherrypick e92f81 from  do not merge.

* commit 'fdef32c01440dda37650ddf534c2b17093c30531':
  Cherrypick e92f81 from master. do not merge.

am 2c64b8c6: (-s ours) Cherrypick f37031 from  do not merge.

* commit '2c64b8c6855a1a4d512ca72abb6f16a9818da29c':
  Cherrypick f37031 from master. do not merge.

am d2213597: (-s ours) Cherrypick 025af8 from  do not merge.

* commit 'd2213597a8f6adeb177d9ddf777112788261e527':
  Cherrypick 025af8 from master. do not merge.

Cherrypick fcd339 from master. do not merge.

Find non-constant expr errors and offer migration explanation

In ADT 14, library projects will generate R fields that are not
constant, which means you cannot use them as case labels in a switch
statement. This means some existing projects may not compile. However,
the reason for this isn't obvious until you know the details of how
library projects work. Also, converting switch statements to if/else
statements is easy in Eclipse but that may not be known to all users.

This changeset adds a quickfix processor which recognizes the specific
compilation error (non-constant expressions cannot be used in case
statements), and if it looks related to Android R fields, then it
creates a fix proposal.

The fix proposal is simple: it just opens a dialog which explains the
problem, and explains how to fix it (along with a hyperlink to more
information on the ADT 14 update process).

The main advantage of this approach is that it shows up as a lightbulb
right in the editor margin (for those users who are affected by this
error only), and even just hovering over the error will show the fix
display string which talks about Migrating Android code and the extra
info states that the fix can be invoked for more info.

Change-Id: Ieb208ac459454310f5a6cce6e933d9115a135322

Cherrypick d7b602 from master. do not merge.

Remove the Fancy and Glossy icon styles

The generator and stencils for the fancy and glossy launcher icon
styles don't match the icon guidelines, so disable them for now until
we have new stencils for alternate shapes etc that also matches the
recommended style.

Change-Id: I17402309467348738b3582d0501ca93d4fed9673

Cherrypick 4cd282 from master. do not merge.

Grid Layout and Convert to Grid Layout improvements

First, some improvements to Grid Layout handling:

(1) When dropping a new widget, look up the sizing metadata and use it
    to determine what fill gravity to set. For example, a button will
    use gravity left, and a text field will use gravity
    fill_horizontal.

(2) Don't warn about reflection problems when failing to find
    GridLayout layout data; this probably means the layout is being
    attempted opened in an unsupporting SDK. There's a fallback case
    to compute the data instead already.

Second, improvements to the conversion to GridLayout handling:

(1) It now looks at the layout_gravity values to see whether each row
    and column is flexible, and if there's no flexible column in each
    of the horizontal and vertical dimensions, it will insert a
    special <Space> element to absorb any available extra space. This
    avoids constraints warnings from GridLayout.

(2) It treats layout_width or layout_height attributes of match_parent
    or fill_parent as the same as a fill gravity (and removes it) and
    uses this in the flexibility computation above.

(3) It removes unsupported layout params for all children (earlier
    this would only remove unsupported layout params on the direct
    children of the layout, which isn't enough when a hierarchy is
    being flattened.)

(4) It's smarter about computing implicit rows and columns, so it
    avoids writing out redundant layout_row and layout_column
    attributes in some cases.

(5) It avoids throwing refactoring errors in cases where an attribute
    is removed twice

(6) Fixes a bug where the root layout was included when computing the
    set of used x and y coordinates, which means you'd often end up
    with a blank row and column 0.

(7) Various refactoring to make the code cleaner.

(8) More unit tests and updates to existing unit tests to reflect the
    new behavior such as an absorbing spacer and removal of redundant
    attributes.

Change-Id: Ifaca43458a4961fa032dfe22dc033b6def3c38d3

Cherrypick e92f81 from master. do not merge.

Update eclipse/changes.txt

Change-Id: I7870b939c9276e112e0754e2ad18a6fd6f2b1236

Cherrypick f37031 from master. do not merge.

Clarify SDK loading message

When SDKs are still loading there is a "please wait" message in the
status window. It would say "The project target (Android 2.3) ..."
when loading rendering targets, which was a bit confusing. The message
will now say "project target" when loading the target that is the
current project target, and otherwise the "rendering target".

Also change the refresh message to just say "The layout will refresh"
instead of mentioning it by name since when the layout name is
lowercase (as it usually is) the sentence begins with a lowercase
letter which looks a bit wrong (and besides there is no ambiguity
about which layout is going to be reloaded).

Change-Id: I8d6a456c2c8ba5fe539d07eef1f5a017c9b8454a

Cherrypick 025af8 from master. do not merge.

Update Asset Studio launcher icon stencils per UX spec

Per discussion with the Android UX team, we've decided to update the
spec for launcher icons. In a future update we will change some of
the underlying behavior to provide more shapes and allow freeform
shapes.

Change-Id: I8dcdd5884d093fd4a7713c5796dd2dc83784be83

am e8201712: (-s ours) Cherry-Pick 3543c3 from  do not merge.

* commit 'e8201712bbfd6aaad00c36e41bfd93e7428f47db':
  Cherry-Pick 3543c3 from master. do not merge.

am 82b6fbd0: (-s ours) Cherrypick 25f755 from  do not merge.

* commit '82b6fbd067b06382da9c69e79c56915e608bf99f':
  Cherrypick 25f755 from master. do not merge.

am 048a3637: (-s ours) Cherrypick 88a0c4 from  do not merge.

* commit '048a363719a6c9020135843036fef4b5ad37356f':
  Cherrypick 88a0c4 from master. do not merge.

Cherry-Pick 3543c3 from master. do not merge.

Use a text box instead of a browser in tooltip area.

Change-Id: I8c6969c9efda7aca3b77c3ca739463cd695b6c82

Cherrypick 25f755 from master. do not merge.

Display tooltips only for long messages.

Currently, the tooltips are displayed for all logcat messages.
This patch makes them show up only if the logcat messages are longer
than the width of the column in which they are displayed, i.e.,
tooltips will be displayed only for those messages that could be clipped.

Change-Id: Iade7f6628a93a9344b337e45b957ca53e6fa4595

Cherrypick 88a0c4 from master. do not merge.

Nyandroid icon for the logcat Eclipse view.

Bug: 5390877
Change-Id: I945ed2854e6b405f9f851f5444312d7bfa56f54c

Merge "Find non-constant expr errors and offer migration explanation"

Merge "Use a text box instead of a browser in tooltip area."

Merge "Display tooltips only for long messages."

Use a text box instead of a browser in tooltip area.

Change-Id: Ica610ef4cefffaef3b6404a425c8b4551e741676

Merge "Nyandroid icon for the logcat Eclipse view."

Display tooltips only for long messages.

Currently, the tooltips are displayed for all logcat messages.
This patch makes them show up only if the logcat messages are longer
than the width of the column in which they are displayed, i.e.,
tooltips will be displayed only for those messages that could be clipped.

Change-Id: Ib5bbce7d055656d2cc8063efdf5fed5910b24ad0

am 2bd58150: (-s ours) Merge 440fea from  do not merge.

* commit '2bd58150cdac3603552241fe70a73388e7094670':
  Merge 440fea from master. do not merge.

am 4348dca9: (-s ours) Cherrypick 78c89f from  do not merge.

* commit '4348dca9124f42af3ac3a6372831fcab66e99c52':
  Cherrypick 78c89f from master. do not merge.

Merge 440fea from master. do not merge.

Don't add Android referenced project to library output.

The jar file output of a library can receive compiled code
from referenced Java projects but if the project is an Android
project then we ignore it. This is probably a bad setup anyway.

Change-Id: Idc1ec531a14c9f91beed26c4084b3956c3999930

Cherrypick 78c89f from master. do not merge.

Also process and cache the png files in libraries.

The png in the libraries were not processed by the crunch step.
Because aapt never processes png files anymore (relying on the crunch
step to do it), the png files in libraries were never processed.

While this is less a problem for standard png files, this completely
breaks 9-patches that must be processed to actually behaves as 9-patch.

Change-Id: I1d9e32e2c0409ec779150de0ad897e210a9be524

Find non-constant expr errors and offer migration explanation

In ADT 14, library projects will generate R fields that are not
constant, which means you cannot use them as case labels in a switch
statement. This means some existing projects may not compile. However,
the reason for this isn't obvious until you know the details of how
library projects work. Also, converting switch statements to if/else
statements is easy in Eclipse but that may not be known to all users.

This changeset adds a quickfix processor which recognizes the specific
compilation error (non-constant expressions cannot be used in case
statements), and if it looks related to Android R fields, then it
creates a fix proposal.

The fix proposal is simple: it just opens a dialog which explains the
problem, and explains how to fix it (along with a hyperlink to more
information on the ADT 14 update process).

The main advantage of this approach is that it shows up as a lightbulb
right in the editor margin (for those users who are affected by this
error only), and even just hovering over the error will show the fix
display string which talks about Migrating Android code and the extra
info states that the fix can be invoked for more info.

Change-Id: Ia90cdf308db87d31c1e6a663613d4cfd0bf675dc

Merge "Don't add Android referenced project to library output."

Merge "Also process and cache the png files in libraries."

Merge "Grid Layout and Convert to Grid Layout improvements"

Merge "Remove the Fancy and Glossy icon styles"

Don't add Android referenced project to library output.

The jar file output of a library can receive compiled code
from referenced Java projects but if the project is an Android
project then we ignore it. This is probably a bad setup anyway.

Change-Id: I6432d4704c2357c64e2c9597eea75328695ff822

am a4790d7a: (-s ours) Cherrypick 09aefc from  do not merge.

* commit 'a4790d7a523cc7c4eecb1fab2ba0fe9da18a9042':
  Cherrypick 09aefc from master. do not merge.

Nyandroid icon for the logcat Eclipse view.

Bug: 5390877
Change-Id: I81e77cf57a358e6c0583d592d104a93475ab37ae

Also process and cache the png files in libraries.

The png in the libraries were not processed by the crunch step.
Because aapt never processes png files anymore (relying on the crunch
step to do it), the png files in libraries were never processed.

While this is less a problem for standard png files, this completely
breaks 9-patches that must be processed to actually behaves as 9-patch.

Change-Id: I0a1fe14ea34f6d36a368b456494410945afc3c44

Cherrypick 09aefc from master. do not merge.

Check for scrolling before refreshing the table.

Change-Id: I14e596a79120284b0c1b8e4bc23439b10a37c2ff

Check for scrolling before refreshing the table.

Change-Id: Id65b11216a8240e3297072f0af094d69d924a8b3

Remove the Fancy and Glossy icon styles

The generator and stencils for the fancy and glossy launcher icon
styles don't match the icon guidelines, so disable them for now until
we have new stencils for alternate shapes etc that also matches the
recommended style.

Change-Id: I521c3c645dee4bc1766e24cf49ee49f19d66ee6f

Merge "Update Asset Studio launcher icon stencils per UX spec"

Update Asset Studio launcher icon stencils per UX spec

Per discussion with the Android UX team, we've decided to update the
spec for launcher icons. In a future update we will change some of
the underlying behavior to provide more shapes and allow freeform
shapes.

Change-Id: Idf575230c7c649299c7368209dfee5e7a0638fad

Grid Layout and Convert to Grid Layout improvements

First, some improvements to Grid Layout handling:

(1) When dropping a new widget, look up the sizing metadata and use it
    to determine what fill gravity to set. For example, a button will
    use gravity left, and a text field will use gravity
    fill_horizontal.

(2) Don't warn about reflection problems when failing to find
    GridLayout layout data; this probably means the layout is being
    attempted opened in an unsupporting SDK. There's a fallback case
    to compute the data instead already.

Second, improvements to the conversion to GridLayout handling:

(1) It now looks at the layout_gravity values to see whether each row
    and column is flexible, and if there's no flexible column in each
    of the horizontal and vertical dimensions, it will insert a
    special <Space> element to absorb any available extra space. This
    avoids constraints warnings from GridLayout.

(2) It treats layout_width or layout_height attributes of match_parent
    or fill_parent as the same as a fill gravity (and removes it) and
    uses this in the flexibility computation above.

(3) It removes unsupported layout params for all children (earlier
    this would only remove unsupported layout params on the direct
    children of the layout, which isn't enough when a hierarchy is
    being flattened.)

(4) It's smarter about computing implicit rows and columns, so it
    avoids writing out redundant layout_row and layout_column
    attributes in some cases.

(5) It avoids throwing refactoring errors in cases where an attribute
    is removed twice

(6) Fixes a bug where the root layout was included when computing the
    set of used x and y coordinates, which means you'd often end up
    with a blank row and column 0.

(7) Various refactoring to make the code cleaner.

(8) More unit tests and updates to existing unit tests to reflect the
    new behavior such as an absorbing spacer and removal of redundant
    attributes.

Change-Id: Iee44c3ca749eefc107b83545669cc9d7f84615b1

am 981fb61d: (-s ours) Cherrypick 0a6192 from  do not merge.

* commit '981fb61d258d58d91a98734ffcfa7242ded90632':
  Cherrypick 0a6192 from master. do not merge.

Cherrypick 0a6192 from master. do not merge.

Add support to make identity files from the command line.

Change-Id: Ieb11a6ac46790f244786a20ede930768946abc91

Merge "Clarify SDK loading message"

Clarify SDK loading message

When SDKs are still loading there is a "please wait" message in the
status window. It would say "The project target (Android 2.3) ..."
when loading rendering targets, which was a bit confusing. The message
will now say "project target" when loading the target that is the
current project target, and otherwise the "rendering target".

Also change the refresh message to just say "The layout will refresh"
instead of mentioning it by name since when the layout name is
lowercase (as it usually is) the sentence begins with a lowercase
letter which looks a bit wrong (and besides there is no ambiguity
about which layout is going to be reloaded).

Change-Id: I2497ba3a35289e9c3f37021cd8a17983e18146cb

am 8760c8e3: (-s ours) Cherrypick 9dd123 from  do not merge.

* commit '8760c8e3ad680c4bcba25701eea2fd3b403db055':
  Cherrypick 9dd123 from master. do not merge.

Cherrypick 9dd123 from master. do not merge.

Fix migration to new library projects.

Also fixed a minor NPE.

Change-Id: Ie76f0120c63b37579b57b32b3b4af005f388a0b0

Add support for exporting heap data.

Change-Id: Ica8de494934195f30b75b8295d0908e3aacdefff

Add support for Select All & Copy Paste.

Change-Id: I4b41b6ee5a7a5600ca3b37d3addf1dce7d1e072a

Construct NativeStackCallInfo with full symbol address.

Addr2line constructs NativeStackCallInfo for a symbol using its
offset within the library as the address. This patch is a minor change
to make it use the address in the executable instead of the offset
in the library.

Change-Id: I3e3e960f6c5abefac83b670e3ab23a78c57a8e56

Merge "Check for relevant stack frame using source library."

Merge "Update default addr2line to use."

Merge "Fix migration to new library projects."

am 794cb8d0: (-s ours) Merge "SDK repository: support for source packages. Do not merge." into tools_r14

* commit '794cb8d02a94610dda1ebef5f0d72fef345b3894':
  SDK repository: support for source packages. Do not merge.

Merge "SDK repository: support for source packages. Do not merge." into tools_r14

Fix migration to new library projects.

Also fixed a minor NPE.

Change-Id: Ide127e3b9a0aeb1b8468d3f4b973f1ee5a27851d

SDK repository: support for source packages. Do not merge.

(cherry picked from commit d963b52229e2d1a3875698e5d2822f00df3485c9)

Change-Id: I249fafe24795b94294c9f9251713c1028a3d9199

Merge "SDK repository: support for source packages."

am a1760eb0: Tools r14 require ADT 14.

* commit 'a1760eb02098b21f1aa37a293a488a1ab4bdb823':
  Tools r14 require ADT 14.

Tools r14 require ADT 14.

Change-Id: If07afb667f0b748e859e2e8b70aa1bcc74f0acfe

SDK repository: support for source packages.

Change-Id: Ib8df984ddbad0bb6f0eb4c2fb0bd4b9c49b334cc

Check for relevant stack frame using source library.

In a stack trace, skip all frames from libc and from methods matching
malloc/realloc/calloc etc to identify the source frame where heap memory
is allocated.

Change-Id: Ia8661c7ff58b3db502c92b2f48819c3219861a73

Update default addr2line to use.

Change-Id: I17a1b29039e50e0256f37519b3fd8781b88f5e39

Update eclipse/changes.txt

Change-Id: I578e5c6f2197ad445894301fbc8c3780579ae8bc

Tweak usage panel message

Change-Id: I99a2953f11e63f6646133c4727880487fe65f0d8

am 3d73f98d: (-s ours) Welcome Wizard tweaks. Do not merge.

* commit '3d73f98d6ac275a01b3b88adf83b825cd70c986b':
  Welcome Wizard tweaks. Do not merge.

am 0d48d0b6: (-s ours) Change welcome wizard string. Do not merge.

* commit '0d48d0b6481cd642d2eb9652c2b8b898e9b0e784':
  Change welcome wizard string. Do not merge.

Welcome Wizard tweaks. Do not merge.

Use Yes/No radio buttons in the Usage Statistic page rather than a
checkbox, and validate that one of the options are chosen such that
the Finish button on page 1 is not enabled.

Also add the Android icon to the wizard.

(cherry picked from commit f5368be1b6cd95fd0ff065a1c624d812b8a260dd)

Change-Id: I4950f33921ba99ac4ec657adb477395c1cda0a07

Change welcome wizard string. Do not merge.

(cherry picked from commit 1632530362278dd0f0613bb0e7cfc67240cbaf59)

Change-Id: Iabfdc90a6941ffb62adc061af9d4cf5e24505e8e

Welcome Wizard tweaks

Use Yes/No radio buttons in the Usage Statistic page rather than a
checkbox, and validate that one of the options are chosen such that
the Finish button on page 1 is not enabled.

Also add the Android icon to the wizard.

Change-Id: Ia8a7593883a7a6cb4126e644cc627edae77757ec

Change welcome wizard string

Change-Id: I4f900bb20f6a55a44a2f61231524b82d9f96c9ee

am a2091e03: (-s ours) Welcome wizard: validate when install common checkbox changed. Do not merge.

* commit 'a2091e03dd8440aed3ab3618befc67162d7bbedf':
  Welcome wizard: validate when install common checkbox changed. Do not merge.

am a2fc0466: (-s ours) Prevent new install of tools from displaying update msg. Do not merge.

* commit 'a2fc0466f6d784ab31306e4572a8af14fa5b2734':
  Prevent new install of tools from displaying update msg. Do not merge.

am 84cbb7df: (-s ours) First time install: multiple platforms, fix shell disposed. Do not merge.

* commit '84cbb7dfc89081b1c3429ed8c3568977cc354cd1':
  First time install: multiple platforms, fix shell disposed. Do not merge.

am d0357c27: (-s ours) Welcome Wizard. Do not merge.

* commit 'd0357c27e602d6bec239897b8375d642bc590a62':
  Welcome Wizard. Do not merge.

Welcome wizard: validate when install common checkbox changed. Do not merge.

(cherry picked from commit 9e0c3a1110b123bd67befc1dd21a56c5f4bcd5ec)

Change-Id: I0e202549e7b8410bf4fe9c71e872cc3ad1d2fa30

Prevent new install of tools from displaying update msg. Do not merge.

(cherry picked from commit 2a94ac98f9079e083ca88037b38244b5b5a46d7f)

Change-Id: Iee30e19c591cb375104fbcbeb3474c52a47df465

First time install: multiple platforms, fix shell disposed. Do not merge.

Fixes for the first time install support:

- Support installing more than one platform.
  The custom install task was tailored before to one
  install a single package and return one status with
  on archive install path. This has been made more
  generic.

- Fix a couple occurences of the dreaded "SWT Shell disposed"
  error that happen when updating the progress bar and closing
  the window at the same time. The issue is that I was checking
  isDisposed before starting a syncExec but the window can be
  closed between the moment the syncExec is schedule and when it
  happens so code must check again for isDisposed inside the
  syncExec.

(cherry picked from commit 949dc34d6202ec5e318c30992bc11b61e0c64808)

Change-Id: I08976f2fcbba68f5371e410ffe8c741c1fa2c2dd

Welcome Wizard. Do not merge.

This changeset adds a new "Welcome Wizard" which is shown the first
time a user runs Eclipse with the ADT plugin.

The welcome wizard asks for two pieces of information:

(1) The location of the SDK.
(2) Whether the user agrees to gathering usage statistics.

We've needed this information before, but collection of the data had
been more ad-hoc: The usage data permission dialog would show up on
its own, and the SDK information would be requested the first time
some code path touched it (e.g.  opening a layout or opening the
preference dialog's Android page etc).

In addition, the wizard also offers to *install* SDKs if you don't
already have one. It gives the option between the latest available
platform, and one supported by a large majority of devices (currently
API level 7), or both. If you select this option, then when finishing
the wizard the SDK manager is run in a special mode where it installs
the required packages with a progress dialog.

This changeset also starts recording the chosen SDK location in the
~/.android settings file. This allows us to detect when you're running
Eclipse in a brand new workspace and you've already gone through the
SDK selection before, and we don't need to ask again -- we'll just use
the most recently known location.

The wizard will only be shown once. If you bypass or cancel out of the
wizard, you can still configure your SDK the old way - via the
Preference dialog. Note also that the usage permission page is only
shown if the user has not already opted in via say ddms.

NOTE: If you want to test this, make sure you haven't set the
environment variable ADT_TEST_SDK_PATH (as some of us do for running
unit tests) since it is treated as the user having selected the given
SDK root, and in particular it means the wizard won't be shown even if
you've wiped adtUsed=true from your ~/.android/ddms.cfg etc.

(cherry picked from commit 292eefb8faa2f75ddbc5d6e20084c9f9a762da29)

Change-Id: Idb3f8775ef62a84905cde95b25eeb8691ef0afab

Merge "Welcome wizard: validate when install common checkbox changed."

Merge "Prevent new install of tools from displaying update msg."

Merge "First time install: multiple platforms, fix shell disposed."

am 082f833d: (-s ours) Merge "Fix ADT build. Do not merge." into tools_r14

* commit '082f833d2eed8e6cb6e4e75a273f76b796902133':
  Fix ADT build. Do not merge.

am f7078fc8: (-s ours) Merge "Address some comments on the DdmsPreferenceStore. Do not merge." into tools_r14

* commit 'f7078fc825eed703af147ff8573d4ddeb47d35bb':
  Address some comments on the DdmsPreferenceStore. Do not merge.

am 39da114d: (-s ours) Merge "SDK: Fix traceview build. Do not merge." into tools_r14

* commit '39da114da748ccf6c03e6766e0a8245333b49422':
  SDK: Fix traceview build. Do not merge.

am 2acdb04f: (-s ours) Merge "Tweak AdtUpdater dialog to install a fresh new SDK. Do not merge." into tools_r14

* commit '2acdb04f2841b9983151802196ef5e09a6a37c3e':
  Tweak AdtUpdater dialog to install a fresh new SDK. Do not merge.

am ba0575d0: (-s ours) Merge "Extract DDMS PreferenceStore. Do not merge." into tools_r14

* commit 'ba0575d0156a888ec41d380a87d418969f0bd4e5':
  Extract DDMS PreferenceStore. Do not merge.