Merge "Fix selection hint painting on honeycomb"

Fix selection hint painting on honeycomb

The selection hints, which for example display the constraints when
you are in a RelativeLayout, are painted below or to the right of the
canvas. The positioning of this text was based on walking up the
ViewInfo hierarchy and taking the root element's bounds as the bounds
of the canvas image. The text was then placed below or to the right
of this image.

This no longer works with honeycomb since we now paint an action bar
and a system bar, and the bounds of these are not included in the root
ViewInfo, so as a result the selection hints were painted on top of
the system bar.

This changes the algorithm to use the image bounds itself rather than
the view info bounds.

Change-Id: I0e6e04608fb55af476c58a33785d5a7d5ac511c1

Merge "Tools r10 require ADT 10.0"

Tools r10 require ADT 10.0

Change-Id: Ide739f4e3edddb3ab04519262eee34d28e18bd16

Merge "Make sure to quote command line paths on windows."

Make sure to quote command line paths on windows.

The path of the executable should not be quoted.

Linux/MacOS actually don't work if anything is quoted so
only quote on windows.

Strangely aapt is not affected by this so don't touch it for now

Change-Id: I148f229d941baf722542ff72ee683fda212ec327

Merge "Default the render target to Android 3.0 for newly opened layouts"

Default the render target to Android 3.0 for newly opened layouts

When opening new layouts, or opening layouts saved with older
versions of ADT, pick a render target that supports layoutlib 5. Many
features (such as layout rendering, palette previews etc) work much
better with layoutlib 5.

Change-Id: I0c577e76a2447985be0ab9bd2603618e8f152873

Merge "Add Layout Actions toolbar"

Add Layout Actions toolbar

Add a new toolbar above the layout canvas (and to the right of the
palette). This toolbar shows various layout-related actions on its
left, and the canvas zoom controls on the right. The zoom controls
have been moved from the configuration panel, which has also been
reorganized a little with three of the dropdowns moved up to make more
horizontal width and take the place of the old zoom controls.

The toolbar items vary based on which layout is "active". The active
layout is the parent layout of the selection, or the root layout in
the canvas if there is no selection.

Some examples:

- In a LinearLayout, there are radio-button icons for switching
  between horizontal and vertical orientation, and for toggling
  baseline alignment

- There are toggle buttons for toggling between wrap_content and
  match_parent (or fill_parent) for the layout_width and layout_height
  properties

- There is a dropdown menu in LinearLayout and RelativeLayout which
  lets you set the layout_gravity to one of the dozen gravity settings

- There is an action button which brings up a margin chooser dialog
  where you can configure the margin settings for the currently
  selected elements for layouts that support margins

There will be additional actions here in the future.

This changeset also adds a few new MenuAction subclasses and factories
to make it possible to do ordered lists of choices. It also adds
sorting keys to make it easier for parents and children to interleave
their actions with simple sorting keys rather than having to append,
prepend or merge their respective result lists.  In a follow-up
changeset I will replace the context menu code to use these, as well
as attempt to make all actions stateless such that they can be cached
and reapplied for different targets.

Some misc cleanup.

Change-Id: I6a87144fcfd1d359e5561829bd5d63c852f16970

merge from tools_r9

Change-Id: Iae5b1e83937acecc7b31c00d949afb5eee150b19

Merge "Minor layoutlib API fix: rename a Capability"

Merge "Package the plug-ins with debug info." into tools_r9

Package the plug-ins with debug info.

This allows us to get useful stack trace from external users.

Change-Id: I290a255b7bb28d6d32eee9d62cb176414d833a1c

Minor layoutlib API fix: rename a Capability

TRANSPARENT -> CUSTOM_BACKGROUND_COLOR

Change-Id: Iaa521231983de4d1008a7bb86b923f295d295a65

Merge "Added "Save As..." option for .trace files."

Merge "Guard against NPE"

Merge "Only allow palette previews when layoutlib 5 is used"

Merge "Package the plug-ins with debug info."

Package the plug-ins with debug info.

This allows us to get useful stack trace from external users.

Change-Id: I588ba6284f7f62cb7a7b73405e805171f350adab

Added "Save As..." option for .trace files.

Change-Id: I852a3d68299629addf53890a8b42d5c0e72579f3

Guard against NPE

Change-Id: I081c057fd7c87239cdd5b4b6ddccf949a9cb2832

Merge "Improve handling for GestureOverlayView"

Only allow palette previews when layoutlib 5 is used

Disable palette previews when the target SDK does not have layoutlib 5
(or more accurately, the custom background support required by the
palette, which will be backported to earlier SDKs.)

Change-Id: Ib572fd070eae995541a680eaccd58411d35066b2

Improve handling for GestureOverlayView

The layout editor has special handling for the "root" element, which
for example suppresses hover-highlighting for the root and a click
will initiate a marquee selection rather than a drag.

Some layouts wrap the layout inside a GestoreOverlayView. This does
not work well since the "real" root is its child, which will not be
treated as a proper root.

This changeset fixes this situation by handling this scenario such
that both the gesture overlay and its child is treated as the root.

Also make fix such that the icon in the outline also works for the
gesture overlay.

Change-Id: Ia96d2bc402958b44f9f82646ff0fc4552ab0cf14

Merge "Palette previews with custom themes and other palette fixes"

Palette previews with custom themes and other palette fixes

Palette previews are rendered on top of the default theme
background. For certain custom themes, where a gradient or image is
used, this can look terrible, since the preview images will include
portions of the gradient or image.

This changeset improves this situation by taking advantage of
layoutlib 5's ability to override the background. It now uses theme
resolution to find the background color, and paints the previews with
the custom background override to force this color. In addition, if
the theme background is not a color (but a gradient etc), it uses
layoutlib5's new render-drawable support to generate the background,
and it extracts a suitable color from it (currently, the center
pixel.)

This fix also ensures that the foreground color (used to paint labels
for the widgets that cannot be rendered, such as layouts) has a
suitable contrast with the background. After computing the background,
it computes the brightness, and if the foreground and background
differ less than 25% in brightness, then the foreground color is
forced to white or black depending on whether the background is light
or dark.

This changeset also contains a bugfix for a palette initialization
problem where in some cases the palette could come up empty (because
the code which attempts to avoid doing extra work when reloadPalette
is called repeatedly could be tricked by a scenario where the SDK
target does not change but its "have target data" status did.)

The open palette category is now preserved across palette refreshes
(which occur when you change palette modes or category/sorting
options, along with theme and render target changes).

Finally, the changeset also turns off the new window decorations
(system bar, action bar, etc) when generating preview images for both
palette previews and for drag & drop.

Change-Id: I8b8766ad45f2cb6a4a6b79a24c31ed0f08d1e826

Merge "Add support for adding non-String value resources"

Add support for adding non-String value resources

The ResourceChooser only supported adding new Strings. This changeset
lets you add other types of value resources -- dimensions, integers,
etc. It will create a new file in res/values/ if necessary, based on
the plural form of the resource name (e.g. for "string" resources it
will create "strings.xml", etc). For existing files, it will add a new
entry to the existing file, using the same indentation as the last
top-level element in the file.

Change-Id: I09272ff52af38a8a7a059d455f398befbe0d1abc

Merge "Fix possible NPE in legacy callback support."

Fix possible NPE in legacy callback support.

Change-Id: I5faa155393466e1bc4b2fc1e9417cde91a8be456

Merge "Add to layoutlib the ability to simply render a Drawable."

Add to layoutlib the ability to simply render a Drawable.

RenderParams is now a base class. SessionParams extends it
(and contains the layout and the rendering mode which are not
part of the base class).

DrawableParams is used for the new action and adds a reference
to a ResourceValue.

Change-Id: Ieacf4da91fda95df1d25a32ae0953bd9d8028113

Merge "Support onClick method handlers in Hyperlink navigation"

Merge "Add app name/icon and current render locale to the LayoutLib API."

Support onClick method handlers in Hyperlink navigation

This uses the JDT to search for method names matching the required
signature for XML-declarations of method handles, and opens the
method.

It searches first in the Activity classes, then globally.

Change-Id: I378a3abaf16aee28fb6fe6dc304ed061942c3774

Merge "Prevent java.lang.ArithmeticException in draw9patch"

Prevent java.lang.ArithmeticException in draw9patch

Initialize the zoom field to 1 instead of the default 0 since
I ran into a scenario where a division by an uninitialized zoom
caused a divide by zero exception.

Exception in thread "AWT-EventQueue-0" java.lang.ArithmeticException:
    / by zero
at com.android.draw9patch.ui.ImageEditorPanel$ImageViewer.checkLockedR
    egion(ImageEditorPanel.java:894)
at com.android.draw9patch.ui.ImageEditorPanel$ImageViewer.access$1800(
    ImageEditorPanel.java:646)
at com.android.draw9patch.ui.ImageEditorPanel$ImageViewer$3.mouseMoved
    (ImageEditorPanel.java:745)
at java.awt.Component.processMouseMotionEvent(Component.java:6397)

Change-Id: I7d2252f07458864d591be91636477ace4b520855

Add app name/icon and current render locale to the LayoutLib API.

This allows the layoutlib to render system/title/action bars
as a window decor like it would look on the device.
This can be disabled with RenderParams.setForceNoDecor().
(BTW, Params was renamed RenderParams too)

Also minor update to the API by replacing an int with Density
since the enum is now accessible to the API and layoutlib.

Change-Id: Ic37770a9276d12af90c60199a84b04cb64e7c3a1

Merge "draw9patch tool: improving zoom process."

Merge "SDK Manager: support extras/vendor/path"

SDK Manager: support extras/vendor/path

This change makes the SDK manager install new extra
packages into SDK/extras/vendor/path, creating the
folders as needed.

Existing extra packages are not moved when they are updated,
provided that they keep the same 'vendor' and 'path' attributes
in their XML definition.

There is also some support for the form SDK/vendor-path/ that
was introduced by Tools r8 and r9.

SDK Bug: 14493

Change-Id: I28b301a768ea2c8c03573f865520b1b3e85f3868

draw9patch tool: improving zoom process.

Change-Id: Ie4d6408858db5b2bafb76ecb916e6b7c00ba0bb7

Reconcile korg and goog

Change-Id: I10f6dee5046fcf802d5d09668bd61a79506475ae

Merge "Binary XML editor"

Binary XML editor

The change enable to review xml files from the android.jar file.
It works so that actually open file from
<SDK_HOME>/android-<APILEVEL>/data.
The com.android.ide.eclipse.adt.binaryXml (BinaryXMLDescriber)
content type associates Android resource xml files to the
BinaryXMLMultiPageEditorPart editor.
The editor checks if the file is contained in jar and is so,
convert editor input to XmlStorageEditorInput that handles
corresponding file from Android SDK.

Change-Id: I8596f0d3ca5477a12dc25f551243f125f8e6ba7e

Merge "SDK: fix win-sdk build errors, make future errors fatal."

Merge "SDK: fix win-sdk build errors, make future errors fatal." into readonly-korg-master

SDK: fix win-sdk build errors, make future errors fatal.

Change the shell script to make any error fatal and actually
break the build, like it should.

Change-Id: Iee86b79c5b12a109ccb92dbc81d586c54ac68b5f

Merge "Add support for CTRL+click on the method in Traceview."

Add support for CTRL+click on the method in Traceview.

This will open the method declaration in an editor.

Change-Id: I79e160bf565ac563e423132e010b7e51ab531673

Merge "Let the Traceview plug-in open the trace file created through DDMS."

Let the Traceview plug-in open the trace file created through DDMS.

New DDMS extension: traceviewLauncher.

This allows another plug-in (traceview) to provide
a way to open traceview file. If this doesn't work
it revert to the default behavior of DDMS which is to
open the external traceview program.

also reverted the extension of traceview file
from .atv to .trace because earlier versions of
Android would automatically create those files
using this extension.

Change-Id: I2605ad47e501770ae100da2ace781b1d5d8cebc5

Merge "Fix possible NPE in Traceview."

Merge "Fixes problem in "draw9patch tool"."

Merge "Make up better class names."

Fix possible NPE in Traceview.

Change-Id: I7834a4fee69ae103a5a39416aacea689e2010519

Merge "Put the Traceview plugin UI in an editor instead of a view."

Put the Traceview plugin UI in an editor instead of a view.

The editor is associated with the .atv extension (Android TraceView).

Change-Id: I5133932804a0c24a52ba53617942c2ca65b9345b

Make up better class names.

JavaGenerator -> SourceProcessor
GeneratorDeltaVisitor -> SourceChangeHandler
NonJavaFileBundle -> SourceFileData

Change-Id: Idaacbef34007a1af730500d8009241683ee9e37b

Merge "Improvements to RenderScript support:"

SDK: fix win-sdk build errors, make future errors fatal.

Change the shell script to make any error fatal and actually
break the build, like it should.

Change-Id: Iee86b79c5b12a109ccb92dbc81d586c54ac68b5f

Improvements to RenderScript support:

- change dependency file location to be in the gen folder
  in the same package as the source file. This prevents collisions
  if there are two classes named the same in different packages
  or source folders.

- Now that the dependency files are visible, track their deletion
  and force recompilation if this happens

- Missing dependency files during generator initialization force
  a recompilation of the source file.

- Prevent packaging from taking .rs, .rsh, and .d files from the
  source folders. This also impacts the Ant build system.

Change-Id: Iede9193f67370e1784bd8ba661198bbf99238074

Merge "Add Renderscript support to the ADT builders."

Add Renderscript support to the ADT builders.

This uses the new JavaGenerator mechanism so that all that's needed
is to run llvm-rs-cc on a given list and parsing the dependency file
that's created.

Change-Id: Ib4928c980422dfe1944bc720c77bf6ae5be4c34a

Merge "Change the JavaGenerator to handle output and dependencies."

Change the JavaGenerator to handle output and dependencies.

Previously, the generator only handled a list of files to compile
and a list of files that were removed.

Now the Generator provides all that needed to do incremental
builders with known state. On top of providing a default
deltavisitor it handles file dependency and list of output
files.

The aidl generator was updated to use this, and the renderscript
generator will use the same mechanism.

Also fixed the abortOnBadSetup which through CoreException which did
nothing to stop later builders (unlike my comments said) and
poluted the workspace log with worthless messages.

Change-Id: Ib954beb9674f1387c022f926388adb8faf6cdac2

Merge "Fix android.bat to support CWD with & character"

Fix android.bat to support CWD with & character

SDK Bug: 12862

Change-Id: Ib9508ce384b91a234e33834b19aaff6cc77e8ade

Merge "Respect Eclipse classpath when adding files to APK"

Respect Eclipse classpath when adding files to APK

Iterating through the source folder may include files that the user has asked to be ignored.

http://code.google.com/p/android/issues/detail?id=12809

Change-Id: I18915cb8868feffe81499276923fec1415076186

Merge "Guard against possible NPE."

Guard against possible NPE.

Change-Id: I4be9b1659aa1394bea0f24f34fb4681e781b96ae

Merge "Refactor aidl handling in its own class."

Merge "Cluster of improvements for merge tag views"

Cluster of improvements for merge tag views

This changeset contains various improvements around usage of the
<merge> tag. Some of these fixes require layoutlib 5.

* Use the new layoutlib support for rendering multiple children at the
  root level - they now show up in the Outline (provided you are
  running layoutlib 5), can be selected in the layout editor, etc.

* Add a drop handler such that you can drag into the <merge> view and
  get drop feedback (similar to the FrameLayout)

* If the <merge> is empty, we don't get any ViewInfos, so in that case
  manufacture a dummy view sized to the screen. Similarly, if we get
  back ViewInfos that are children of a <merge> tag in the UI model,
  create a <merge> view initialized to the bounding rectangle of these
  views and reparent the views to it.

* Support highlighting multiple views simultaneously when you select
  an include tag that renders into multiple views (because the root of
  the included layout was a <merge> tag).  Similarly, make "Show
  Included In" work properly for <merge> views, and make the overlay
  mask used to hide all included content also reveal only the primary
  selected views (when a view is included more than once.) (Also tweak
  the visual appearance of the mask, and use better icon for the view
  root in the included-root scenario.)

* Improve the algorithm which deals with render results with null
  keys. Use adjacent children that -do- have keys as constraints when
  attempting to match up views without keys and unreferenced model
  nodes. This fixes issue
      http://code.google.com/p/android/issues/detail?id=14188

* Improve the way we pick views under the mouse. This used to search
  down the view hierarchy in sibling order. Instead, search in reverse
  sibling order since this will match what is drawn in the layout. For
  views like FrameLayout and <merge> views, the children are painted
  on top of ech other, so clicking on whatever is on top should choose
  that view, not some earlier sibling below it.

* Fix such that when you drag into the canvas, we *always* target the
  root node, even if it is not under the mouse. This is particularly
  important with <merge> tags, but this also helps if you for example
  have a LinearLayout as the root element, and the layout_height
  property is wrap_content instead of match_parent. In that case, the
  LinearLayout will *only* cover its children, so if you drag over the
  visual screen, it looks like you should be able to drop into the
  layout, but you cannot since it only covers its children. With this
  fix, all positions outside the root element's actual bounds are also
  considered targetting the root.

* Fix broken unit test, add new unit tests.

Change-Id: Id96a06a8763d02845af4531a47fe32afe703df2f

Refactor aidl handling in its own class.

It extends a new base class that will serve as a base
class for the one handling renderscript files.

Change-Id: Ibef0c4b9a792fe52bf7b70bf5d24f76a15cb65c9

Merge "Traceview integration into Eclipse."

Traceview integration into Eclipse.

New plugin which adds a Traceview view to Eclipse.

Change-Id: I3001dfde6ed28db3de8a75715a0ba0743f5c29b0

Merge "Fix in PreferencePage."

Merge "Fix for issue 14189."

Fixes problem in "draw9patch tool".

If a big PNG image will be loading, HelpPanel hides ImageView.
This patch fix that problem.

Change-Id: I615705849b475e5b88daae7bef2af92cb5fa0657

Merge "Scale drag previews with the screen zoom"

Scale drag previews with the screen zoom

If you have zoomed your layout, which is going to be common when
designing for 10" screens, then the drag preview shown when dragging
from the palette is not sized correctly. It assumes a 100% zoom.

This changeset looks at the current canvas zoom and scales the preview
image such that it matches the canvas zoom, so if you're zoomed in
then the preview will be larger, and vice versa.

Change-Id: I471b4d8ae459aad79d50a2b54ecca6bd7d7cf943

Merge "Deemphasize AbsoluteLayout"

Merge "Add support for multiple hyperlink targets"

Add support for multiple hyperlink targets

This changeset makes the Go To Declaration hyperlink resolver produce
a list of possible jump targets. Therefore, if you for example point
at @android:string/ok, you will get a popup listing all the various
string.xml files that define a translation for this string. If there
is a current configuration (which would be the case if you are
pointing at a resource url from a layout file, but not from a Java
file or a manifest file) then the declaration which best matches the
configuration is listed first, followed by everything else in
alphabetical order by the folder name.

This uses the resource maps which the IDE is already maintaining for
rendering, so it can compute the set of hyperlinks cheaply.

In addition, the hyperlink resolver now goes to the direct declaration
of the given resource. This definition can in turn reference another
resource url.

There is one exception to the instant resolution of files: ids. In
layouts it's common to define ids in place (via @+id), and these are
not available in the resource map. Therefore, if no id is found
defined in a value folder, then the search is deferred until the link
is actually clicked. The search will now search not just the base
layout folder, but all layout folders, starting with the same folder
as the link source, followed by all configuration-compatible folders,
followed by all folders.

Change-Id: I20fd4f5836e41e5a9839124b7be3a3230f5198f4

Deemphasize AbsoluteLayout

Move AbsoluteLayout out of the Layout category in the palette, and
into the "Advanced" category.  Also add selection hints for the
absolute which explains that AbsoluteLayout is deprecated.

Change-Id: I2b1633e90794bdec7b7a0d3171f9a29eb25dd258

Fix for issue 14189.

Now the user can set if he wants to change perspective
when he double-clicks in the LogCat view.
If he wants to change perspective, he can set which
perspective shall be opened.

Change-Id: I3ed68f3f8ad3bbfcd6d161b63a0c0f218dd778b6

Merge "Add MipMap to the resource type list."

Add MipMap to the resource type list.

Change-Id: I1f56ab470874e39dad8659d15a3699b83f1ac658

Merge "Move Pair and annoatations into resources.jar now renamed as common.jar"

Move Pair and annoatations into resources.jar now renamed as common.jar

Move all the resource query methods that returned an array of 2 Strings
to return a pair of ResourceType and String.

Change-Id: I6b8447aa27005de786e2defef81ad88a72363523

Merge "Fix compilation errors due to 2 independent changes."

Fix compilation errors due to 2 independent changes.

Change-Id: Ib7aa41aa6c95ec17ffde2644320b851f8f02f85f

Merge "Change APIs using String instead of ResourceType."

Merge "Improve QuickContactBadge designtime"

Change APIs using String instead of ResourceType.

Move ResourceType into resources.jar so that it's accessible
to layoutlib.jar

This is cleaner and allows us to us more efficient EnumMap objects.

Change-Id: If11cbc69ae3ca8bd6c96e6d0ef402570a07af16f

Merge "Fix clean script."

Fix clean script.

The way bash works, the script returns the status of the last command.
In this case, the last failed test's return value is what the script
returns, even when that's what we want. Encapsulating the test in
a function works around this.

Change-Id: I44de75d80b2d16ca4110f078d40db0f71f1aa400