transforms/2d/hindi-rotated.html has settled in on imageonlyfailure.

BUG=243538
TBR=jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151064 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Fix on reload leaks in NetworkLog and TreeOutline found with allocations tracker.

R=loislo@chromium.org

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

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

This gets rid of more places where Persistent handles are copied
(see bug).

BUG=236290
R=dcarney@chromium.org, jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151066 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: [HeapTracking] Keep recorded snapshot in the selected state after stop record.

When we stop the recording process we removes temporary snapshot (and moves focus to the previous snapshot)
After that we append the new snapshot.
As a result the focus left on an old snapshot.

BUG=none
TEST=none
R=alph@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=150978

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151067 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add an isolate parameter to ScopedPersistent::set.

This gets rid of one call site of v8::Isolate::GetCurrent, which we're trying to
get rid of.

BUG=none
R=dcarney@chromium.org, jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151068 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Create V8PerContextData based on a Local handle, not Persistent.

This gets rid of more places where Persistent handles are copied
(see bug).

BUG=236290
R=haraken@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151070 bbb929c8-8fbe-4397-9dbb-9b2b20218538
animations/play-state* are all over the place. :/

BUG=243353
TBR=jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151071 bbb929c8-8fbe-4397-9dbb-9b2b20218538
editing/execCommand/apply-style-command-crash.html is flaky on mac.

BUG=243701
TBR=jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151072 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move CSSOM classes to using MutableStylePropertySet over StylePropertySet.

Moved ensureCSSStyleDeclaration() and ensureInlineCSSStyleDeclaration() from StylePropertySet
to MutableStylePropertySet.

StyleRule & co now return a MutableStylePropertySet* from mutableProperties() to aid with...

CSSStyleDeclaration now only deals with MutableStylePropertySets. This was already the case
in practice, we just didn't have any compile-time foot-shooting protection.
In other words, if there's a CSSOM wrapper for a given StylePropertySet, it's guaranteed to
be mutable, and the pointer types now reflect this.

This is a backport from WebKit r148396. Patch by Andreas Kling reviewed by Anders Carlsson.

R=eseidel@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151073 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Don't assert when serializing -mask property.

In the case -webkit-mask property contains several layers, it is possible
that one of the layer contains "initial" implicitly set simply because the
repeat property was not specified. However when reconstructing the
mask property from StylePropertySet it is possible to have for example
-mask-repeat-x with a value like 'initial, repeat-x' where the first initial
is implicit (the original -mask first layer did not set it). As
StylePropertySet should not return implicitly set initial values we should
just omit these ones. The ASSERT was triggered because of better checks before
casting to CSSPrimitiveValues but in this example we should not reach that
cast, we should bail out : 'initial' is not a primitive value.

BUG=242931
R=apavlov@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151074 bbb929c8-8fbe-4397-9dbb-9b2b20218538
When querying a primitive css value, use a try catch block.

If the value isn't convertible to a number, this will throw an
exception. That's for example the case if the value is "auto". If we
can't read the css value as float, read it as string instead. This is
unlikely to match the expectation, but still better than just not
catching the exception and timing out.

R=mkwst@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151075 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Updating expectations for plugins/reentrant-update-widget-positions.html

[ Pass Timeout ] on linux and windows, [ Failure Timeout ] on mac. :(

BUG=35387
TBR=jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151078 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Set the timeline window to match the recorded time interval.

When objects tracking is recorded make the timeline overview selection window
set to the recorded interval instead of entire time span.

R=loislo@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@151079 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Revert "Revert 151053 "Implement V8ScriptRunner::callFunction()""

This reverts commit ddb3cc68a62b6776258baf6b79ac56816dfca277.

This was a speculative revert to see if Mac errors would clear up. They
did not, but then magically disappeared a few commits later. Ugh.
Reverting the revert, as this CL wasn't to blame.

> Implement V8ScriptRunner::callFunction()
>
> This patch just factors out the core part of ScriptController::callFunctionWithInstrumentation()
> to V8ScriptRunner::callFunction().
>
> The goal of the refactoring is to aggregate all call sites of Function::Call()
> into V8ScriptRunner::callFunction() and V8ScriptRunner::callInternalFunction().
> I'll implement V8ScriptRunner::callInternalFunction() in the next patch.
>
> BUG=241743
> R=abarth,eseidel
> NOTRY=true
>
> Review URL: https://chromiumcodereview.appspot.com/15975002

TBR=haraken@chromium.org
BUG=241743

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

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