Remove zoom argument from family-setting FontBuilder functions.

A follow-up to https://codereview.chromium.org/200573006/,
does the same thing to font-family.

R=eae
BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169286 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Optimize StyleSheetContents::checkLoaded by keeping track of completed
and loading clients.

For the input-wstyle.html benchmark from the bug report this brings
the difference between the x-input and the x-input-style down to
a factor of two and the time scales linearly with the number of
elements added in the benchmark.

x-input: 52.1520ms
x-input-style: 105.3900ms
input: 27.0840ms

R=adamk@chromium.org, esprehn@chromium.org, haraken@chromium.org, tasak@chromium.org
BUG=350785

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169287 bbb929c8-8fbe-4397-9dbb-9b2b20218538
[CSS Shapes] inset corner radii are not flipped for vertical writing modes

When creating an inset shape, convert the radii to logical coordinates as well as the box.

BUG=352785

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169288 bbb929c8-8fbe-4397-9dbb-9b2b20218538
A thread-safe Media Query Parser

Implement a thread-safe Media Query parser based on eseidel's tokenizer (https://codereview.chromium.org/123053002/).

This CL is composed of 2 parts:
* Rendering the MediaQuery related classes thread-safe, by eliminating AtomicStrings and replacing them with hash comparisons.
* Replacing BisonCSSParser for a new parser for MQ parsing.

BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169289 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Update TestExpectations: gardening.

virtual/threaded/animations/transition-and-animation-2.html flaky on mac also.

BUG=248938
TEST=
TBR=zmo@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169290 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Layout Test svg/batik/text/textPosition2.svg is flaky

BUG=352856
TEST=
TBR=zmo@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169291 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Revert 169289 "A thread-safe Media Query Parser"

A bunch of builders failing compile step.

http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac%20Oilpan/builds/2093/steps/compile/logs/stdio

http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Oilpan/builds/2319/steps/compile/logs/stdio

> A thread-safe Media Query Parser
> 
> Implement a thread-safe Media Query parser based on eseidel's tokenizer (https://codereview.chromium.org/123053002/).
> 
> This CL is composed of 2 parts:
> * Rendering the MediaQuery related classes thread-safe, by eliminating AtomicStrings and replacing them with hash comparisons.
> * Replacing BisonCSSParser for a new parser for MQ parsing.
> 
> BUG=
> 
> Review URL: https://codereview.chromium.org/171383002

TBR=yoav@yoav.ws

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169292 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Replace Element::isMediaElement() with isHTMLMediaElement

This make HTMLMediaElement type casting more in line with the generated
code for most other HTML elements. Not incidentally, this simplifies the
code in several places.

BUG=none

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169293 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Improve the accuracy of V8 external allocation reporting in HTMLCanvasElement

This change modifies the existing V8 external allocation reporting in
HTMLCanvasElement to make it report the memory consumed by all
the image buffers that are held by the canvas element. This adresses
an issue where cyclic allocation of WebGL contexts was not putting
enough pressure on V8 to trigger garbage collections frequently
enough to prevent the browser from running out of GPU memory.

BUG=351871

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169294 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Revert node.contains to the original behavior.

The node.contains changed the behavior in
https://chromiumcodereview.appspot.com/21123005

We don't have any spec for that.
Until we have a wider consensus, it should be better to revert the change.
It is not wise to leave this change as is because it doesn't follow the semantics of node.contains defined in DOM living standard.

http://dom.spec.whatwg.org/#dom-node-contains

This is discussed on this W3 bug:
<https://www.w3.org/Bugs/Public/show_bug.cgi?id=22141>

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169296 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add a RELEASE_ASSERT() to invesigate some mysterious crashes

This change adds a RELEASE_ASSERT() which should replaces
a series of crashes we're seeing in the crash report.

BUG=352043
R=kouhei,japhet@chromium.org,eseidel

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169297 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Use invalidation sets for some pseudo elements.

Allow to use class invalidation sets for selectors containing ::before,
::after, or ::backdrop. These pseudo elements are updated correctly in
Element::recalcChildStyle for LocalStyleChange on their parents.

BUG=335247

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169298 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Follow up fix after r169252

The scrollTop should never go below zero after guardZone is applied.
Otherwise it will always force an update. A similar property
should hold for scrollBottom.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169299 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Encapsulate the checks for shouldRepaintOverflow into RenderObject.

Instead of explosing an IfNeeded method that returns a boolean on
RenderObject we should just add a method that does the work IfNeeded
like we do in the rest of blink.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169300 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Speedup window selection in heap allocations tracking.

Do not make extra call to updateVisibleNodes as another one
has already been performed in sortingChanged -> recursiveSortingLeave

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169301 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Fix showing thread's combobox in source panel without experiment

BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169302 bbb929c8-8fbe-4397-9dbb-9b2b20218538
TimelinePanel: disconnect header element from TimelineGrid and header element in TimelineView

BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169303 bbb929c8-8fbe-4397-9dbb-9b2b20218538
id of iframe incorrectly sets window name

As per the specification, the iframe's name should be an empty string if unset:
http://www.whatwg.org/specs/web-apps/current-work/multipage/the-iframe-element.html#attr-iframe-name

Instead, Chromium was using the iframe's id as window name if the name was
unset. This change makes us behave like Firefox 27 and IE 11, verified using:
http://jsfiddle.net/xf5H7/9/

This CL sets the name attribute iframe in a lot of layout tests so that
testRunner.dumpChildFramesAsText() keeps printing the same result.

R=arv@chromium.org, tkent@chromium.org, abarth, arv, tkent
BUG=347169
TEST=fast/frames/iframe-no-name.html

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

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

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169304 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove unused GraphicsContext::concat() return value.

Unused in Blink, going away in Skia.

R=schenney@chromium.org,reed@google.com

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

git-svn-id: svn://svn.chromium.org/blink/trunk@169305 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Oilpan: Make sure we can make pre-finalization hook with Visitor::registerWeakMembers.

BUG=none
TEST=automated

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

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