First round of cleaning up the Android layout test runner

This moves all methods to interact with Android devices out of the
test port and driver, into a separate AndroidCommands class. I also
updated the unittests for the Chromium Android layout test runner
script to (1) pass again, (2) work with this new abstraction, (3) be
more relevant to what's actually being used and (4) be readable.

This patch also fixes a bug (and adds a test) when executing the
run-webkit-tests script with the "--adb-device" argument.

BUG=232044
R=eseidel@chromium.org, jochen@chromium.org

Review URL: https://codereview.chromium.org/14188013

git-svn-id: svn://svn.chromium.org/blink/trunk@149345 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Make CodeMirrorTextEditor a default editor

This patch makes CodeMirrorTextEditor a default text editor.
It removes CM from options and patches some layout tests to use TextEditor interface
abstraction instead of DefaultTextEditor class.

BUG=
R=pfeldman@chromium.org

Review URL: https://codereview.chromium.org/14207017

git-svn-id: svn://svn.chromium.org/blink/trunk@149346 bbb929c8-8fbe-4397-9dbb-9b2b20218538
SVGElement destructor may use invalid iterator

When an SVGElement object has rare data, its destructor gets a
hash map iterator for the rare data, uses it to clear resources,
then uses the iterator to delete the rare data. However, the resource
cleanup can delete other SVG elements, thus modifying the hash map
from which the iterator came and hence invalidating the iterator
itself.

The fix is to re-get the iterator before deleting the rare data.

BUG=235638
R=inferno@chromium.org, pdr@chromium.org

Review URL: https://codereview.chromium.org/14533008

git-svn-id: svn://svn.chromium.org/blink/trunk@149347 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Eagerly discard GraphicsContext3D on context loss

BUG=235658
R=jamesr@chromium.org, kbr@chromium.org

Review URL: https://codereview.chromium.org/14283011

git-svn-id: svn://svn.chromium.org/blink/trunk@149348 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: [HeapProfiler] fix for hidpi screens

BUG=none
TEST=none
R= pfeldman@chromium.org, yurys@chromium.org

Review URL: https://codereview.chromium.org/14288003

git-svn-id: svn://svn.chromium.org/blink/trunk@149349 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fix npruntime test by adding it to DumpRenderTree.gypi

TBR=ojan
BUG=76253

Review URL: https://codereview.chromium.org/14288004

git-svn-id: svn://svn.chromium.org/blink/trunk@149350 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Revert "Create errors (especially cancellation errors) internally to WebCore, rather"

This reverts commit 2e0515922df0d78e12e68e117f2f76b372a1ab8a.

TBR=darin
BUG=

Review URL: https://codereview.chromium.org/14495011

git-svn-id: svn://svn.chromium.org/blink/trunk@149351 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Drop redundant Path::fastBoundingRect()

This patch removes the redundant fastBoundingRect function, as
our boundingRect() function is already fast!

No observable effects, just a refactoring.

BUG=227397
R=leviw@chromium.org

Review URL: https://codereview.chromium.org/14463006

git-svn-id: svn://svn.chromium.org/blink/trunk@149353 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Extract code related to CSS properties manipulation from StylePropertySet into a separate class.

StylePropertySet is a container, it should not contain semantic on how to
serialize CSS properties. This is the first step in direction of sharing
CSS properties serialization code inside the codebase. For example
CSSComputedStyleDeclaration also have a bunch of similar code.

BUG=234606
R=eseidel@chromium.org

Review URL: https://codereview.chromium.org/14347007

git-svn-id: svn://svn.chromium.org/blink/trunk@149354 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Removing GraphicsContext::roundToDevicePixels.

Nothing calls it.

BUG=235572
R=pdr@chromium.org

Review URL: https://codereview.chromium.org/14159023

git-svn-id: svn://svn.chromium.org/blink/trunk@149355 bbb929c8-8fbe-4397-9dbb-9b2b20218538