Apply additional feedback to weak member change.

R=ager@chromium.org, erik.corry@gmail.com, haraken@chromium.org, vegorov@chromium.org, zerny@chromium.org
BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164880 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Improve a search logic to find a SVGFontFaceSrcElement

In SVGFontFaceElement::rebuildFontFace(), it tries to find a SVGFontFaceSrcElement. However, current search logic looks inefficient. The search logic starts to find SVGFontFaceSrcElement from a first child node, then it replaces a discovered element whenever finding next SVGFontFaceSrcElement. Thus, it would be good if we find SVGFontFaceSrcElement from a last node. Then, we go out of the searching loop as soon as we find a first SVGFontFaceSrcElement. It will help to reduce iteration number in the search loop.

Additionally the search loop is moved to the place which uses a SVGFontFaceSrcElement actually.

BUG=N/A

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164881 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Include the logical bottom margin in the layout overflow rect. In the most common use
case this ensures we can scroll down to the margin that sits after any element that has it.
The only cases we need to avoid are where the margin is present but doesn't contribute height.
These are (1) where the element has a margin quirk (i.e. the margin is
defined with quirky em values ('__qems')) and so is disregarded in margin collapsing and (2) when
the element is self-collapsing so the entire element adds no height of its own.

This fix revealed a separate bug where collapsed margins from the content of a table caption
were not contributing to the height of the caption and its parent. This is tested in
fast/forms/form-hides-table.html.

BUG=320743

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

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

See also webkit.org/b/113415

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164882 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove position argument from VTTParser::collectTimeStamp

The |position| argument is no longer used by any of the callers.
(They all pass a pointer to a dummy local set to zero.)

BUG=305317

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

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