LayoutLib API: fixed wrong camelcase in API.

Change-Id: Iddc8681b841d445747f583380c136310d64c41fc

LayoutLib API: move add/moveChild API back to using index.

Also add an optional data bundle to SceneResult.

Change-Id: I9f9d4ca1f1f05d536a87a005a7939a6d42d0d8a4

Merge "Fix 9-patch padding"

Merge "LayoutLib API: New SceneStatus enum."

Fix 9-patch padding

Forgot to change one line that test against 1 for
left index, when lines now start at 0.

Change-Id: Ib9a7779d8fd70b3b0c6b243bf67d77aa986b13a6

LayoutLib API: New SceneStatus enum.

Change-Id: I1c2ac6ddcabe1faeffe3753735b9f87fde683fd3

Merge "Layoutlib API: updated API for insert/move/remove child."

Layoutlib API: updated API for insert/move/remove child.

Also added some new SceneStatus enums.

Change-Id: I225c58201d81cb1109d1d533fea48b6eacb2e286

Merge "ADT: The image given to layoutlib now shares data with ImageData"

Merge "ADT: fix java/class name mismatch."

Merge "ADT: cleanup RenamePackage action."

ADT: The image given to layoutlib now shares data with ImageData

We now create a custom BufferedImage that also includes a reusable
ImageData to generate SWT images. While there is still the AWT to
SWT conversion, at least most buffers aren't re-allocated every
time during animation. The only new allocation is the one done
in the Image constructor to pass the ImageData content to the native
image object.

This also fixes the tearing issue introduced in the previous
commit by calling ImageOverlay.setImage from the animation
listener callback. This should not be done in the UI thread
runnable as each rendering reuse the same bitmap.
When the animation listener callback returns, the image
will be reused for a new rendering so it's content will
change.

Also fixes the windows rendering by computing the proper
band offset based on the SWT palette Data.

Change-Id: Id57a3696bd92441271f6bee9681edefd12140b02

Merge "ADT/Layoutlib: New API to let the caller instantiate the bitmap."

ADT/Layoutlib: New API to let the caller instantiate the bitmap.

This allows us to use a bitmap more compatible with SWT.

In ADT's case, because the bitmap needs to be converted to SWT
before being displayed, we create a BufferedImage using a byte[]
instead of a int[] so that we can simply do an array copy.

Also, we reuse the generated BufferedImage unless the size changed,
which lets us see less GC during animation playback.

Change-Id: I0062a4f4442ff6469cf0ad4f501c1fbe8c719400

ADT: fix java/class name mismatch.

Change-Id: Id0e539be1873904245f50b3bcd861f171360a990

Merge "Move WST doc to sdk/docs."

Move WST doc to sdk/docs.

Change-Id: I53d7838b557da43a9194ddb13ec1bcd0f1119a36

Merge "ADT string refactoring: replace in all files."

ADT string refactoring: replace in all files.

When doing an extract string either from Java or XML:
- can scan/replace in all other Java files.
- can scan/replace in all other XML files.
- in Java, also replace in assignements.
- in XML, also replace existing string name if already defined.

Change-Id: Ifeef5fd444c2c18b9c071955b8e8567d6515ea95

Merge "Replace reflection with Sdk.makeRelativeTo"

Replace reflection with Sdk.makeRelativeTo

Replace reflection-based invocation of IPath.makeRelativeTo
(which is not available in Eclipse 3.4) with usage of
Sdk.makeRelativeTo, our own copy of the equivalent code.

Change-Id: If67d985b96b845d85cf7fc13d47e9adeee49e8a8

Merge "The rendering target mode should be preserved across IDE sessions"

The rendering target mode should be preserved across IDE sessions

This changeset fixes the bug where the rendering target is not
preserved across IDE sessions the way the other configuration settings
seem to be.

Change-Id: I56fc82b1de18c162f62e80d1ba96b54d697b2607

Merge "ADT: more animation stuff."

ADT: more animation stuff.

Change-Id: I020342a4fe205161328f22e9dabb0b03343677b5

Merge "Rename some layout editor classes"

Rename some layout editor classes

This changeset contains only renaming of some classes (and a couple
of >100 column adjustments), no semantic changes.

The name changes are:

BaseView => BaseViewRule
BaseLayout => BaseLayoutRule
ScaleInfo => CanvasTransform
CanvasSelection => SelectionItem
PropertySheetPage2 => PropertySheetPage
OutlinePage2 => OutlinePage

Change-Id: I14d8c711b12154f4fcb2169129fd553e31fdab84

Merge "ADT: rework extract string refactoring."

merge from tools_r8

Change-Id: I7553de468542889488ac4dba53a23ae876ff5526

Merge "Win SDK: Split the win-sdk makefile and script into 2 parts. DO NOT MERGE." into tools_r8

Win SDK: Split the win-sdk makefile and script into 2 parts. DO NOT MERGE.

[Merge Change Ic94d261b from master_open into tools_r8 to fix broken SDK build.]

The main makefile+scripts to build the Windows SDK is still
in development/build/tools. However it defers to a new
matching set of files here in sdk/build to build and
package things that depend on the sdk.git or external/qemu.git.

This will make it easier for us to prepare SDKs based on
a tools_rN branch that isn't cut at the same time than the
platform branch.

This is a multi-part changeset. The other part is in
development.git.

Change-Id: I8072bf59ee1af57826a8bafa0103afd6fe3fc858

ADT: rework extract string refactoring.

This updates the code that inserts new <string> elements
in a <resources> XML, replacing my micro-XML stream-oriented
parser by one based on the WST SSE region model.

Also added a mini doc to explain the WST region model, since
there's no doc anywhere as far as I can see. I choose to place
this ad-hoc doc where I need it rather than in a generic place
-- it's not a generic doc, more a reminder of the API.

Change-Id: Icd39390c4124ce0ec21322ff223390ac4a06f431

Merge "ADT: Animation preparation."

ADT: Animation preparation.

Update layoutlib API to work better with the new scene
locking mechanism (for concurrent renderings), new error
types in SceneResult, and updated Animation listener.

ADT changes to record the view object in CanvasViewInfo,
and the current LayoutScene in ViewHierarchy.

Added a test menu item to start an animation. This
is temporary and will be fixed later.

Change-Id: I67df2d116afdfd23c093e4645d4a0f99695c5d95

Merge "Extract layout constants into a separate constants class"

Merge "Add unit test configuration for moreUnit"

Extract layout constants into a separate constants class

Combine the constants in BaseView and a LayoutConstants class over in
the internal packages into a new LayoutConstants class, and reference
these constants elsewhere (statically imported). This was suggested by
in the feedback to review #18971.

Change-Id: I40b76f8f6045c34a98f7a2363f96d2942360d1f3

Merge "Fix build. Wrong import package!"

Fix build. Wrong import package!

Fix build. Wrong import package!

Change-Id: Id96450a7f5553c88d0d5c0996ff5c378ea03bbc3

Merge "Show images during canvas drags"

Show images during canvas drags

When you drag in the canvas, show a drag image for the view being
dragged. The drag image is using 30% translucency such that it does
not obscure the drag feedback below it.

We're grabbing the image data right out of the rendered scene and just
copying out the rectangles for the bounds of each selected item. This
has to take the canvas zoom into effect too.

While in the zoom code I modified the Zoom In and Zoom Out buttons to
only zoom by a factor of 20% rather than 100%.

I cleaned up the Pair class slightly; in an earlier version of this
fix I was using it to pass back two results from the rectangle
painting function; I don't do that anymore but the Pair might as well
be cleaned up.

(By the way, as suggested in review #19035 I rewrote the FIXME comment
related to resource lookup and put it in this CL rather than revising
the other one.)

Change-Id: Iabe6e29c4073252f5046e3c26b56f4e7674a3f3a

ADT: cleanup RenamePackage action.

Move it to the refactoring package.
Split action code from refactoring code.
No change in functionality nor invocation (via
context menu project > Android Tools.)

Change-Id: Ieec945b37fdd33d497bf745d207eb5d6b5445d9f

Merge "Unit test fixes"

Unit test fixes

First, the AddonsListFetcherTest#testLoadSample_1 was failing on the
Mac, because the source code contained non-ASCII unicode characters,
and the encoding differs between Windows, Linux and Mac -- and on the
Mac the characters were garbage (not the intended Japanese
characters). Fixed by using unicode escape sequences to define the
expected output instead.

Second, rename the AbstractLayoutTest to LayoutTestBase; the test
runner (for plugins, not unit tests) was assuming this was a test case
(even though it's an abstract class!) so it was emitting a warning
about the class containing no test cases. This is simply a base class
for all the individual layout tests defining a bunch of useful
inherited shared behavior, so rename to avoid the warning.

Change-Id: I6e29316c5644db35051218aef8b99ac41ab0cda4

Add unit test configuration for moreUnit

Eclipse doesn't have a builtin mechanism to jump between a class and
its test, as well as a way to run tests associated with a given
class. There is however a plugin for this, "moreUnit".

Our projects have split off our unit tests in a separate project,
which requires some per-project configuration to tell moreUnit where
to look for tests. This information has to live with the project
rather than in my workspace, so I'm checking in the configuration file
- it is harmless if you don't have moreUnit but useful if you do.

Change-Id: Id29c46e2eaa89f2ad530c2b24700db28cc823b3b

Merge "Replace IPath.makeRelativeTo call with reflection"

Replace IPath.makeRelativeTo call with reflection

I broke the build because on Eclipse 3.4 there is no makeRelativeTo
method on IPath. This replaces that call with reflection. This won't
work on 3.4, but it's a quick fix for the broken build.

Change-Id: Ia917cf5a745f76bbb96f59fae93e2a4fc4f5f900

Merge "Open included layout on double click"

Merge "Fix nine patch size."

Merge "Remove some asserts."

Open included layout on double click

On double click, if the clicked area originates from an <include>'ed
XML file, open the included layout in the editor. Also opens
files in the @android: namespace if available, for example if you
inculde @android:layout/select_dialog_multichoice.

Change-Id: I215c411257717f7b97e7b0ee1d5498c318cdb04d

Remove some asserts.

Change-Id: I8458dc9237506c542c766e6119261ae4e13b453b

Merge "ADT: ensure <application> is last in manifest."

ADT: ensure <application> is last in manifest.

ADT descriptors have no notion of XML ordering.
We modify the concept of "mandatory" descriptor to
have normal mandatory descriptors versus "mandatory last" ones.
("mandatory" elements are virtual reserved slots that are
 always present in the UiModel even if they have no real
 counterpart in the XML model. These elements ensure they can
 be manipulated in the UI even before their XML is created.)

Then we try our best to ensure the "last" ones remain at the
end of the UiNode model. There are 2 cases:
- in UiElementNode, when constructing the UINode model when
  parsing an existing XML model, we try to reorder the nodes
  at the end if possible.
- in UiActions, when adding a new element to the UiModel, we
  try to place it before the first "mandatory last" element.

SDK Bug 3197310

Change-Id: I6a7d9502a95ebe92ff82e07f3f3249a0d25c2154

Fix nine patch size.

Since we moved the 9-patch chunk out of the NinePatch, the image
content portion is not in our control (it's owned by the default
implementation of NinePatchDrawable through an Android Bitmap),
and therefore should not include the control outer line.

This changeset makes the bitmap contains only the extracted
content and makes the computation of the patches take into account that
the draw method will be applied on a bitmap that is only
the content without the control line.
Basically the computation of the patches/fixed rectangle exclude the
outer control line.

Change-Id: I29c2224ee2311962b30d1fb155b85d5731d4e6ec

Merge "Refactor selection painting to fix highlighting of included views"

Merge "Move the ninepatch info into its own class."

Merge "Fix live manipulation of <include> elements"

Merge "Fix unit test log usage"

Merge "Fix -v logging level flag."

Fix -v logging level flag.

- Rather than ignoring the flag, actually make use of it.
- Change the default logging level to SEVERE
- Change the "command slow" exception print from ddmlib to INFO

Change-Id: Iade4700b32ed7b4a55bcd3336c74b2127693bebe

Merge "Fix URL for eclipse download (for auto build plugin)"

Fix URL for eclipse download (for auto build plugin)

Change-Id: I1eba4b240869a222a0eb1c8676cd120acfb0fba4

Merge "Fix potential NPE when closing Eclipse/ADT"

Fix potential NPE when closing Eclipse/ADT

Seems like it was trying to finish a build and closing
the plugin ADT at the same time. How's that even possible?
In any case I've never seen that before and I was in
debug mode, so I bet it's a rare race condition.

SDK Bug 3222178

Change-Id: I28a9bdfcbe56e2a6b73eafb5cadea611c5177f7f

Move the ninepatch info into its own class.

The chunk is what is used separately by the default NinePatch class,
through serialization to make a byte[] out of it.

Since we are moving away from replacement classes to delegate, splitting
the chunk info away from the image allows us to use the normal
NinePatch(Drawable).

Change-Id: Ifd86dc2aa9b485d0e97a2d4a248621cfcddda9ab

Fix unit test log usage

There were some test failures because unit tests were relying on
AdtPlugin's logger, which appears to be null during unit test
runs. Use a new test logger instead, which fails the current test if
anyone logs an error message, and dumps warnings to standard error and
prints to standard output.

Change-Id: I8e52fed554d49e98f7d6c8990d41831998f44640

Fix live manipulation of <include> elements

This changeset fixes bugs related to dragging included views around in
the canvas:

First, the <include> tag would get rewritten to <null>. This happened
because the layout editor treats the FQCN and the XML node name as
equivalent, but in the case of the include tag the FQCN was set to
null instead of "include".

Second, the "layout" attribute would not get copied because it is NOT
in the Android namespace, and the code to copy attributes ended up
with an empty-string namespace which was not handled correctly.

Third, when copied the "layout" attribute would end up with the
namespace "ns:" because the code to create attribute nodes always
created namespaced attribute nodes rather than a plain attribute node
when the namespace is null.

Change-Id: Ibd34212517615aa8ec79abe14bca765cdca525f6

Refactor selection painting to fix highlighting of included views

This changeset moves the painting of selection bounds and selection
hints out of the view rules and into the core IDE. The reason for this
is that the visual appearance of the selection shouldn't be up to each
rule; for one thing the selection highlight should be consistent and
not vary from view to view (and in fact there was only a single
implementation of the paint selection method among the view rules),
and for another the view rules are in theory sharable among IDEs
whereas the selection appearance is going to be IDE specific. There
was also painting of "hints" in the RelativeLayout; rather than having
the visual appearance of this dictated by the rule, this is also moved
into the IDE such that the rules only provide the hint text and the
hints are displayed by the IDE itself.

The above refactoring also fixes selection feedback for <include>'ed
views, which were not visually selectable because there was no
corresponding ViewRule, so nobody to paint selection. With these
changes selection painting is now independent of the rules.

Change-Id: I22dd926102128634a443b8bafb54d4764f1eda41

Merge "Fix the other SwtUtilsTest methods under Windows."

Fix the other SwtUtilsTest methods under Windows.

Fixes the following issues:
- All methods must use the palette mask to compare RGB values.
- Use a non-trivial default color for testing. The previous
  choice of 0xFF00FF00 would not allow detecting RGB <-> BGR
  mixups and the 0xFF alpha mask prevented from noticing the
  fill operation was pre-multiplying the colors.
- Assert the converted image as the expected transparency type.

Change-Id: I1dc7518a7d7000bce75e1c4166a9921ba40a43cb

Merge "Fix SwtUtils conversion for no-alpha case."

Fix SwtUtils conversion for no-alpha case.

Change-Id: I275b526670ca6e60cfc4f8749631dff2ef240b96

Merge "GLE: Remove obsolete IGraphicalLayoutEditor interface."

GLE: Remove obsolete IGraphicalLayoutEditor interface.

Change-Id: I96f929ce9386e88f80d15195b2ff6a498e374554

Merge "ADT: support for editing embedded layouts."

ADT: support for editing embedded layouts.

This is a very first rough changeset. The context in which the layout
is edited is hardcoded until we have the UI for it.

The basics are the following:
- if there's a context, don't pass the parser on top of the model to
  the layout lib.
- instead create a KXml based parser that will return the model-based
parser when queried for a custom parser (if the layout name matches the
file being edited).

Change-Id: I2d106def9c64e82ab50c8fae96cb050338422a85

Merge "ADT/Layoutlib: update API to control how layout expands."

ADT/Layoutlib: update API to control how layout expands.

Previous API was a single on/off to let the layouts expand at
render time depending on how much space they needed.

The new API can now control expansion is horizontal and/or
vertical (or not at all)

Basic implementation in the editor, with a manual detect of
"ScrollView" as top element. We should make the ViewRule handle
this somehow.

Change-Id: Idc503bc0d1d3df98fbf01cc84625952ca55a8afb

Merge "Add per-view custom initialization logic"

Add per-view custom initialization logic

This changeset adds support for adding custom-logic to initialize
views, both to add children and default attributes and to customize
layout attributes when added to a new parent.

First, there is a new "onCreate" hook which is called to notify a view
rule that an instance of its corresponding view has been created. This
lets the ViewRule perform custom initialization of the object. The
ViewRule is told what type of insertion occurred, such that it can
distinguish between a newly created view, a view that is the result of
a copy/paste, and a view that is part of a move operation.

The changeset adds a number of new ViewRules which take advantage of
this:
- A TabHost rule creates the various skeleton children that are
  required, such as a TabWidget child with id @android:id/tabs and a
  FrameLayout child with id @android:id/tabcontent
- A DialerFilter rule creates the mandatory EditText children ("hint"
  and "primary")
- The HorizontalScrollView rule creates a horizontal LinearLayout child
- The ImageButton and ImageViewButtons initialize the "src" attribute
  to a sample image
- The MapViewRule initializes the apiKey attribute

In addition, views are also notified when a new view is added as a
child, such that they can perform additional customizations, in the
form of an "onInsert" event.

The most important application of this is LinearLayoutRule, which uses
this to set reasonable defaults for the layout_width and layout_height
parameters. It uses metadata (which is currently built into ADT but
would ideally migrate into our XML config files) to determine whether
a given child prefers to grow horizontally, grow vertically, both, or
neither, depending on the surrounding parent context.  For example, an
EditText will default to filling the parent width if it is in a
vertical LinearLayout, but it will not grow vertically in a horizontal
linear layout. And so on. Various other rules also use the onInsert
event to tweak children attributes. A ScrollView will for example
always initialize its single child to match parent.

Views can now also add plain menu items into the context menu, and the
TableViewRule adds one such action: "Add Row", which appends a new row
into the table.

The Palette Preview code also invokes these creation hooks, such that
if you for example drag a DialerFilter it can properly render since
the mandatory children are created up front. This required various
changes to the preview code to be able to handle XML edits by the
rules.

Finally, this changeset includes various other misc changes that I
performed at the same time:
- Removed SWT dependency from the ViewRule classes (SWT Rectangle use
  in Rect)
- Fixed AbsoluteLayout unit test (issue 3203560)
- Fixed positioning of the preview outline in LinearLayout when only
   one of the dimensions are clipped due to a smaller target layout

Change-Id: I5956fe4e7a31a20b8dd2f9d9b0c1f90e2f75d68a

Merge "Fix PDT. Extension class of toolsLocator is now ToolsLocator"

Fix PDT. Extension class of toolsLocator is now ToolsLocator

Change-Id: Ic2b65e07755339fc277392e4acdfe1e880a83a3d

Merge "Cleanup and comment the refresh code in GLE2."

Merge "Cleanup GRE preload."

Merge "Remove non-api references from AttrsXmlParser."

Merge "Fixed a problem that physical address is inconsistent."

Cleanup and comment the refresh code in GLE2.

Change-Id: I95a922c463992600edfc8c25edfc688dc38f79bd

Cleanup GRE preload.

GRE used to preload the groovy BaseView rule to improve
the first selection speed. We don't need that now that we
switched to a non-groovy engine.

Change-Id: Ib29272f28df6285137d62c45441cd9ac72efd10f

Remove non-api references from AttrsXmlParser.

Change-Id: Ic139e6f942e835dda4b7ef0303556aef014a60d3

Merge "ADT/GLE: call LayoutScene.dispose() when needed."

ADT/GLE: call LayoutScene.dispose() when needed.

Change-Id: I95819d95827164d3e57dc179d3f321d26a649599

Merge "Remove AdtPlugin dependency from AttrsXmlParser."

Remove AdtPlugin dependency from AttrsXmlParser.

The AdtPluin was used just for logging. Instead the
AttrsXmlParser takes an ILogger (AdtPlug implements
ILogger and can be used directly in unit tests too).

For unit tests there is a new StdSdkLog convenience
class that prints to stdout/stderr (formerly MockStdLogger
from the Sdk Manager was doing that.)

Change-Id: I658af61d04efb19ad6e3bf9c0bf471452372885a

Merge "Update ADT changelog with current changes."

Update ADT changelog with current changes.

Change-Id: Id02026a70882d260855b283693ab93c9810fa2e3

Merge "ADT/GLE2: Add callback before the rendering target changes."

ADT/GLE2: Add callback before the rendering target changes.

Change-Id: If02fd18c39b06042615735b55190e9f03fef5b19

merge from tools_r8

Change-Id: I7e81a594ba2f83e787c288709d14d2b97d89074e

Merge "Fix HV.bat for swt layout issue." into tools_r8