Merge "Always refresh logcat table from UI thread."

Merge "SDK Man2: fix bug when deleting a local package."

SDK Man2: fix bug when deleting a local package.

When deleting a local package, the model was still
retaining the info of the previously installed package
and was thus not offering it again for install.

Also fixes a case where selecting something for
install and then cancelling would trigger an
unecessary list refresh.

Change-Id: I0889e2c19c721325f921d8681a429210f76b490b

Always refresh logcat table from UI thread.

Change-Id: I8b11783de412a31baf37fc2d9efb8aafdb73a330

Merge "Fix SdkLib unit test to not print on stderr."

Merge "Add-ons do not support codenames."

Fix SdkLib unit test to not print on stderr.

The SdkLib unit tests had the annoying side effect
of writing an XML schema validation error on stderr.
This patch uses an XML Error Handler to capture the
error and actually test for it too.

Change-Id: I129804c6f9cb4ba3dfc25db4f3313553fb01df63

Add-ons do not support codenames.

There was a typo in the repository XSD and the addon XSD
that had an optional 'codename' element to the addon type.
Since it's optional and (should have) never beend used,
it's safe to remove it and document why we're removing it
without spinning a new XSD version.
Also added a unit-test to enforce it's invalid to have.

Change-Id: Id3c5725c022d86ac0f46532efe25ac5c000c1af7

Merge "Experimental Java-version of the sdk launcher."

Merge "SdkMan2 Log window tweaks."

Merge "SDK Manager: don't retry source if http auth canceled by user."

SdkMan2 Log window tweaks.

- Change the toggle button icon (looks like a little
  log sheet, grayed or enabled). This replaces the ugly
  blue thingy.
- Add support for tooltips to the toggle button widget.
- When logging an error, auto-raise the log window. This
  was done using a crude hack in the first version by
  having the window raise itself. Now this is more properly
  done by having the controller (i.e. the SDK Main Windows)
  raise it when it detects an error in the log. This way it
  can properly adjust the toggle button state.

Change-Id: I4a444651da4ba3b9022484344b526d8a1b3be3c3

SDK Manager: don't retry source if http auth canceled by user.

This fixes a minor workflow issue the case of an
SDK source using a Basic HTTP Auth: if the user cancels
the login dialog, we don't try to be smart and try
alternate URLs. It actually means the URL was just fine
but the user didn't want to authenticate.

Change-Id: If85dc2e01e30ff26decb78088bd912da785a89f9

Merge "SdkMan2, log window comments & auto-scroll"

Merge "Cleanup a few warnings in SdkLib."

Merge "Remove @Override from methods implementing an interface"

Merge "Use fonts as specified in preferences."

Merge "Add tests for LogCatMessageParser."

Cleanup a few warnings in SdkLib.

Change-Id: Ie8429f077b7e3a2a7166b6fcd7b9454c9bc47088

Remove @Override from methods implementing an interface

@Override'ing methods from an interface requires JDK 6.

Change-Id: I78dcf84dc690e5928f05a4d1a20e24a4e10d7eae

Use fonts as specified in preferences.

Update DDMS preference dialog to show only font preference when using
the new logcat view. The font preference key is obtained from
LogCatPanel in the new view so that the Eclipse ddms plugin can
also reuse most of the code.

Change-Id: I61afba27cde45ff8214e83b6fadfe488f6c954bb

SdkMan2, log window comments & auto-scroll

Fixes some missing comments in the LogWindow class.
Also implement a rudimentary auto-scroll based on the
caret position being at the end when we append text.

Change-Id: Ie1dfecc2dc048113f786bc120c8f267eea955e0b

Merge "Fix a few XML comment formatting corner cases"

Merge "Add various getProperty methods."

Add tests for LogCatMessageParser.

Change-Id: I2fc1f14cbf72117112a217b2f9effb00b3e040a0

Merge "Support clearing all logcat messages from the table."

Merge "Support exporting selected logcat messages to a text file."

Merge "Fix logcat table scrolling behavior."

Merge "SdkMan2: Log window."

SdkMan2: Log window.

Change-Id: I97d173679519a1f535ec303ed1c996f16dd33c24

Merge "Add support for authenticated add-on servers."

Merge "Add Apache Http-Client library files to ADT."

Fix a few XML comment formatting corner cases

The cases are covered by the new unit tests.

Change-Id: I165def59dad59de7e8e1c4e5516c567722aa254e

Add various getProperty methods.

Bug 5148643

Change-Id: I8efefdf07e8524f95dcb043a4eae1812b97b2739

Support clearing all logcat messages from the table.

Change-Id: If7fb4b181974ddd0c1c41e54f7c76bfb6b2812bc

Support exporting selected logcat messages to a text file.

Change-Id: Ia9c61a5e16ef12907f4402a076a587cc9f62613c

Add support for authenticated add-on servers.

Displays a login prompt if necessary.
Note: This code depends on Apache http-client libraries.
Build files had been modified in order to add this dependency.
http-client libraries had also be added on prebuilt project.
    prebuilt changeID=I084d78dd09a431bc3a2d77e77810b84c693bdcb7
             GerritLink=https://review.source.android.com/#change,23387

Change-Id: Icada9b41a21fe3aacef9a1eff209a3fe5591a4e0

Merge "Added view introspection to ChimpChat and MonkeyRunner"

Add Apache Http-Client library files to ADT.

This is a follow up to Change I084d78dd from prebuilt.git
to add the files required by sdklib to ADT.

Change-Id: I651ddf8f6dec480843c5bccd7d3296227b262937

Fix logcat table scrolling behavior.

This patch makes the logcat table scroll behavior to be consistent
with other consoles/terminals: The table will scroll to reveal a
newly added message if and only if the last logcat message is
currently displayed (which is the same as saying if the scrollbar
is at the bottom of the table). If the user scrolls up the table
by moving the scroll thumb, then newly added messages will not
trigger a scroll.

Change-Id: Ia9c180b093de91406e8ced020589c71b4d2e1eb1

Merge "Create new project ddmuilib-tests"

Merge "Add support for filtering logcat messages."

Create new project ddmuilib-tests

Change-Id: I62fbf052dead8f9c4378c42c1850851c51b794fc

Add support for filtering logcat messages.

In the logcat view, add a panel on the side where users can
create/edit/delete filters for logcat messages. Users can filter
by the message tag, pid or loglevel.

Change-Id: Ic85397794e5437b761b91a74adb0dd6a58bc74ed

Merge "Get easymock from ANDROID_OUT_FRAMEWORK"

Merge "Save/Restore the width of all columns in the LogCatPanel table."

Merge "Improve partial XML formatting"

Merge "Format newly created files"

Improve partial XML formatting

This changeset improves the behavior of the formatting when formatting
just a part of the document (e.g. the selected region, or more
accurately the nodes overlapping the selected region.)

First, it looks up the current indentation levels in the document of
the start node and its parent and uses that for the indentation, which
means that nodes appear to be pretty printed in line instead of
jumping to whatever the absolute indentation level would correspond
to.

Second, the formatter now has a mode where it can format only the
opening tag of an element. This is a special case of formatting a node
which is particularly useful when you enable automatic formatting by
the layout editor, since just setting a property on a root element
shouldn't reformat all the element children of the edited node.

Third, there are various tweaks to how the whitespace is handled
around the inserted node. It also fixes a couple of bugs.

Change-Id: If1dd2dc5091a08b770927cd4b9fcafe0c032a216

Format newly created files

This changeset hooks the XML formatter up to the New XML File and New
Android Project wizards such that the newly created XML files are run
through the formatter, thereby picking up whatever formatting
customizations the user has made.

It also makes it such that when you finish the New XML File wizard for
a file that does not have a graphical editor, the caret position is
shown and the view receives focus so you can start editing at the
appropriate position.

Finally, it tweaks the defaut root element for color-lists.

Change-Id: I19606a085d6df132009679ddfb4c91152cbdc453

Merge "Ability to suppress rendering fidelity warnings"

Merge "Update XML Mac keybindings for select enclosing and select last"

Merge "Reformat the ADT plugin.xml file"

Merge "Make the plugin build script for for Mac OSX"

Get easymock from ANDROID_OUT_FRAMEWORK

Change-Id: Ife3e5d1f5f1884a2e304dd18e109a6d7325ab90c

Save/Restore the width of all columns in the LogCatPanel table.

Create all columns using TableHelper.createTableColumn which adds
listeners that save the column width whenever they are resized by
the user, and restores them on subsequent creations.

Change-Id: I35d09a981b94aa086899e3bf74f2243029c60b2b

Merge "Fix template creation of ids"

Merge "Fix linear layout and frame layout unit tests"

Make the plugin build script for for Mac OSX

The build_server script used to build the plugin zip file was Linux
specific. This changeset tweaks the scripts such that they will also
run on Mac OSX. It also adds various .gitignore filters such that the
temporary files created by the build script are ignored by git.

Change-Id: I77cfe6bfb65be86c11beb50bcab9c345433b5a8e

Merge "SdkMan2: Correctly handle local items newer than remote."

SdkMan2: Correctly handle local items newer than remote.

This fixes an edge case where a local package is newer
(e.g. has a higher revision number) than a remote one.
This typically happens during development and QA but
not in a normal user scenario.
In this case the newer local package 'hides' the remote
package so that we don't see a dup in the list.

Also fixes the source filter when merging remote packages
and add a comment on the rationale of which, namely that
if two remote repos were to publish the same packages we
DO want to see dups. This is not supposed to happen and
we would want to see it and have it fixed.

Change-Id: Iab51cfd64b4078f3233872973173465a924eac1e

Merge "SdkMan2: Can only install remote packages, not locally installed ones."

SdkMan2: Can only install remote packages, not locally installed ones.

Change-Id: Ia56b5f8a3458c139d79cfdb4acef45f3f89cb5e4

merge from tools_r13

Change-Id: I08d87676cbdcd2908c7157b726d7350058cc5c1e

Ignore from tools_r13

Change-Id: Ieb0d164466d88021a1c22cdbbf1794bb8fc471d9

Fix template creation of ids

We have a quickfix for creating missing resources, such as
@string/foo.  This code was not working correctly for @id resources,
because ids use a different declaration syntax (there is no value to
be specified as a text node child of the item tag, and the type
attribute should be specified).

This changeset also fixes ResourceHelper's isFileBasedResourceType().
ResourceHelper.isFileBasedResourceType recently started returning true
for ResourceType.ID. This seems related to a change in the return
values from FolderTypeRelationship.

In any case, ids should not be treated as #isFileBasedResourceType,
because that method is used to determine whether the id is one-to-one
with a filename, which in turn means that the resource name validator
will insist that the filename can only contain -lowercase- characters
(plus digits and underscores).

Incidentally this change was caught by a unit test, yay!

Change-Id: I0c95c20d666d0744fda73e8230846d24ddd437c3

Ability to suppress rendering fidelity warnings

If you're working with a custom component that uses unsupported
graphics operations, the layout editor will show an error area. This
error is always there. That can be annoying since there's nothing you
can do about it after reporting a bug.

This changeset adds the ability to suppress these errors for the
current IDE session. It adds a "Ignore" hyperlink next to each
fidelity rendering error message, and when you click it, the specific
type of rendering error is suppressed and hidden for the current IDE
session.

These warnings are only suppressed for a single session and not
permanently. This is because you are suppressing an error not for a
specific layout but for any occurrence of that error. If these ignore
settings were persisted you might open some unrelated layout in the
future and be puzzled why it does not look right.

Change-Id: I2d20a36aee7767257b16ad805b0e8da36a6eb97e

Update XML Mac keybindings for select enclosing and select last

One of the most useful selection operations in Eclipse Java editors is
"select enclosing" and "select last".  The keybindings on Mac are
Ctrl-Shift-Up and Ctrl-Shift-Down. However, in XML files, those
keybindings do something different (making the caret jump, without
selection, to the previous element instead of the parent).

It turns out the XML editor also implements those actions, but bound
to other keys on the Mac. The reason for this is that the Java editor
applies special mac specific keybindings for those actions, and the
XML editor does not.

This changeset updates the keybindings, for Mac only, to the same
keybindings as the Mac-only Java editor bindings for the same actions.

Change-Id: I81f7f6d14fe8dd4a79aeace757d6c25c011eedb8

Merge "SdkMan2: Fix edge case when install/delete packages."

Merge "SdkMan2: Show number of packages to install/delete"

Merge "SdkMan2: Fix load when there are no remote sources."

Reformat the ADT plugin.xml file

The plugin.xml file had a lot of very inconsistent formatting,
primarily because the Eclipse Format action seems to be disabled in
it, and most of the contents are the result of copy pasting in
snippets to register various services based on code samples in
documentation.

This changeset simply reformats the file (4 spaces, one attribute per
line) to make the format consistent and easier to adhere to when
inserting new content.

There are NO semantic changes - it's a pure reformat only affecting
whitespace.

Change-Id: I42237f67ada0e2ddeffc5e305a7df1083f0fdb97

Merge "Miscellaneous XML formatting fixes"

Fix linear layout and frame layout unit tests

I changed the code which draws drop feedback recently to use a more
efficient call (calling drawRect(x1,y1,x2,y2) directly instead
of constructing a new Rectangle object and calling drawRect(Rect).
Some golden files checking graphics output needs to be updated.

Change-Id: I369fed9cb0756ff4f3aa970c1bdd552d0f55faaa

SdkMan2: Fix edge case when install/delete packages.

One issue when installing or deleting packages is that
sometimes after an install the display would show both
the uninstalled and installed package. And when deleting
it would also show both states.
The core issue is that it's a bad idea to rely on
Set<>.contains() when your objects equality isn't quite
what the hash code says.

Or said otherwise, there's more than one definition
of "equals()" for Package objects. That is when a
package is installed or deleted, its internal state
changes and its archive changes (it looses or gain an
OS path), which changes the overall hash code but
at that point it's the same object from the user point
of view.

Anyway, long story made short: this fixes that.

Change-Id: I6ec705cc807d96d097cf9ea7faf8234f7ed34416

SdkMan2: Show number of packages to install/delete

Change-Id: I2113998c9a7274d5c468f27f4b51b8d2293d0c17

SdkMan2: Fix load when there are no remote sources.

There was a bug where the package loader would not show
all locally installed packages if it didn't find any
remote source (e.g. when there's no network).
This fixes it. Also adds a UT to cover this case.

Change-Id: I5ccfd91895cfece54f2c2cdff58040c5ae8d1e98

Merge "Fix some UT."

Miscellaneous XML formatting fixes

This changeset contains a number of tweaks to the XML formatter:

- Handle "end-of-line" comments, such as those seen in some drawable
  <selector> files:
    <item android:state_pressed="true"
          android:color="#ffff0000"/> <!-- pressed -->
    <item android:state_focused="true"
          android:color="#ff0000ff"/> <!-- focused -->
    <item android:color="#ff000000"/> <!-- default -->

  Here the comments will continue to live on the same line as the end
  tag.

- If the unformatted XML had a blank line between a comment and the
  next element, then insert a blank line there in the formatted
  XML as well

- Always place the xmlns: attribute on the same line as the opening
  element tag

- In alphabetical attribute sorting, make xmlns: attributes an
  exception and always sort them to the front just as is the case for
  logical attribute sorting

- Fix some cases where turning on "remove all lines" would still leave
  some blank lines, such as in front of the root element closing
  tag and around comments

- Fix unit tests such that they run with the default values for the
  formatter; they were accessing an uninitialized AdtPrefs instance
  (since they are not running as plugin tests) and were inadvertently
  just using "false" for all the flags.

- Add more unit tests.

Change-Id: I537f8afe4ca4d775b5f91574b895ba90a2741f5c

Fix some UT.

Change-Id: Ia3ca9ef68d6435a323959913a6bd9298e9bcf0c9

Experimental Java-version of the sdk launcher.

The goal is to have this in $SDK/tools/lib/sdklauncher.jar
with 2 $SDK/SDK Manager|AVD Manager.exe wrappers (built using
jsmooth) that will run the launcher.jar, which in turns can
run/restart the SDK manager itself or the AVD manager.

The original launcher is still available by doing
  $ USE_MINGW=1 make sdklauncher
whereas the new experimenal one is available by doing
  $ make sdklauncher_exp

Change-Id: Iddb6c59b334a6197865f798b3d12fcf70880a141

Merge "Add eclipse version to usage stat ping."

Merge "Add pushFile/pullFile interface for IDevice."

Merge "Split PackagesPage and PackageLoader inner classes."

Merge "SDK Manager: refactor classes for manager 1 versus 2."

Merge "SdkManager2: UTs for initial platform selection."

Merge "SdkMan2: only select recent new platforms."

Merge "Fix NPE"

Fix NPE

Change-Id: I044342e5aa6b3aaa87efdcc79bbcb3db93190252

Add eclipse version to usage stat ping.

Change-Id: I14dba0dd7cb900c2e700e4da055aee7bb7d1f9cd

Split PackagesPage and PackageLoader inner classes.

PackagesPage and PackageLoader have quite a number of inner
classes. This splits them in separate package-protected
classes, making the page source a tad shorter.

This is a pure refactoring move/rename. No logic is modified.

Change-Id: I88885aed40e34bd28de4b1ea4e1de66ea2ed89b8

SDK Manager: refactor classes for manager 1 versus 2.

This moves SDK Manager 1 versus 2 classes in different
packages, ..repository.sdkman1 versus ..repository.sdkman2.

There is no change to any actual logic. Only edits to fix
imports and make some classes public.

Change-Id: Ia5ea2ef55fb6913816b27d81c496434d80a00c58

Add pushFile/pullFile interface for IDevice.

Implement pushFile/pullFile methods for Device.

Change-Id: I8491f98fc46bbfd5f476b72e002cd97f63c56228

Merge "Fix formatting bug for formatting ranges"

Merge "Add tag-handling for Go To Matching in XML documents"

Merge "Improve smart-indent handling in XML files"

SdkManager2: UTs for initial platform selection.

Change-Id: I4aef0e6ad8993e792a20f95f37a62f44e858b426

SdkMan2: only select recent new platforms.

Try to add some heuristic to select new/updates
so that we only select new stuff from platforms
the user has or the top most recent platform.

Change-Id: I3b4c7a3b515bdee8393a248493d2dc475f2f5745

Merge "Initial implementation of the new logcat panel."

Initial implementation of the new logcat panel.

This logcat panel will show up in ddms if it is launched with -Dcom.android.ddms.useNewLogCatView.
Currently, this implementation only displays a table with a list of logcat messages. It lacks
features such as filters, go-to source of exception, export to log, user preferences, etc.
Future patches should enhance the UI to achieve feature parity with older UI.

Change-Id: I3dde3c590c839318ce57bb5f005627f580ebb06c