Use typedef to shorten WorkerThreadableWebSocketChannel inner classes.

This CL doesn't change the behavior.

BUG=384238

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

git-svn-id: svn://svn.chromium.org/blink/trunk@178813 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Make WebViewImpl::detectContentOnTouch Shadow DOM aware.

BUG=394106
TEST=none

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

git-svn-id: svn://svn.chromium.org/blink/trunk@178814 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Animations: Fix rem units not calculated in pseudo element keyframes

Before a pseudo-element is created, rem units are not correctly
calculated since the element is null and the document root is found via
this element. Fix simply involves passing the document root to
ElementResolveContext when the element is null.

BUG=392081

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

git-svn-id: svn://svn.chromium.org/blink/trunk@178815 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Rebaseline date format related tests on Mavericks

The OS X API used by datetime formatter relies on ICU/CLDR underneath.
In English locale, a comma is now present between date and time.

This CL does what the following two CLs did for Linux

https://codereview.chromium.org/302323004
https://codereview.chromium.org/403393004

except that the following test was already revised to work regardless of
the presence of a comma between date and time by the first CL of the two
above and is already passing :

fast/forms/datetimelocal-multiple-fields/datetimelocal-multiple-fields-mouse-events.html

The following two tests need rebaselining because Mavericks now has a
comma between date and timein English locale. These new expected results
are identical to the corresponding Linux results.

fast/forms/datetimelocal-multiple-fields/datetimelocal-multiple-fields-value-set-empty.html
fast/forms/datetimelocal-multiple-fields/datetimelocal-multiple-fields-preserve-value-after-history-back.html

Besides, the generic results for the above two tests are copied to
platform/mac-mountainlion.

BUG=314947
TEST=The above 3 tests pass on Mavericks.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@178816 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move ScopedStyleResolver and ScopedStyleTree to oilpan's heap

In oilpan, TreeScope, all Nodes in the TreeScope, StyleEngine, StyleResolver and ScopedStyleTree live and die together. Thus we can make raw pointers in ScopedStyleTree Members. Also we can remove clearScopedStyleResolver() from ScopedStyleTree's destructor, because the TreeScope and ScopedStyleTree live and die together.

BUG=340522

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

git-svn-id: svn://svn.chromium.org/blink/trunk@178817 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Clear window script NPObject before disposing v8 context.

|ScriptController::clearScriptObjects()| calls |disposeUnderlyingV8Object|, which relies on the fact that context is alive.
However, the context should be detached from |V8WindowShell::clearForNavigation| calls, so |clearScriptObjects()| must be done in prior.

Currently, there is a bug in v8 that |v8::Context::DetachGlobal()| does not detach the global and context completely, so the current code works, but it will not after https://codereview.chromium.org/397953009/ land.

BUG=364377

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

git-svn-id: svn://svn.chromium.org/blink/trunk@178818 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Use Traversal<>::firstChild() instead of firstWithin() when iterating over children

Use Traversal<>::firstChild() instead of firstWithin() when iterating over
children. It makes it clearer that we are only iterating over the children and
not traversing the descendants.

There is no behavior change since firstChild() is equivalent to firstWithin()
for ElementTraversal / Traversal<HTMLElement>. However, using firstChild()
improves readability a bit.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@178819 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Oilpan build fix after r178811

TBR=sof@opera.com

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

git-svn-id: svn://svn.chromium.org/blink/trunk@178820 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Change BidiResolver helper methods to take run list as argument

Change BidiResolver and InlineBidiResolver helper methods to take the
bidi run list as an argument instead of using m_runs and mark methods as
const where appropriate.

This is the first step towards reversing the ownership of the run list
and removing the m_runs member from BidiResolver.

BUG=396963
R=dglazkov@chromium.org

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

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