Remove no longer used methods nextAncestorSibling() from NodeTraversal.h

These methods are no longer used and should be removed.  They were previously
used by the traverseNextTemplate methods that have since been merged into
wtf/TreeNode.h.

BUG=331880

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164639 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Revert "Perform layout when adding or removing custom scrollbars"

This reverts r164352 and r164548. The former tried to rebaseline
a reftest, which is not something that makes sense.

TBR=ccameron@chromium.org,adamk@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164640 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove HistoryController class members to track loads in progress.

They can easily be stack variables and only used as needed. This patch also
removes HistoryController::inSameDocumentLoad(), which shouldn't be necessary
anymore.

BUG=none

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164641 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Gracefully handle <meta http-equiv name content> elements.

If presented with a <meta> element having both a 'name' and
'http-equiv' attribute, try interpreting the 'name' attribute first
before _also_ considering 'http-equiv'. The specification requires
that at most one of these attributes to be used, but authors don't
always keep their side of the bargain.

This simply reinstates previous behavior, something that regressed in
https://codereview.chromium.org/25728006/ (it ignores 'http-equiv' if
'name' is supplied, even if the latter is set to some unknown value.)

R=
BUG=332090

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164642 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fix HTMLVideoElement aspect ratio on a failed poster load.

This fixes an aspect ratio change triggered by a failed poster URL
load. A failed load was causing the element aspect ratio to be
changed to 1x1 instead of keeping the default aspect ratio that would
be used if a poster URL load was never even attempted.

Added tests to verify aspect ratio behaviour for successful & failed
poster loads.

BUG=328865
TESTS=LayoutTests/media/video-poster.html

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164643 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Gardening: Rebaseline some picker tests for Windows XP.

TBR=wjmaclean@chromium.org
BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164644 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Avoid layout/full-repaint on view height change if possible

Previously we always layout/full-repaint of the whole view when the height
of the RenderView changes. In many cases this is unnecessary. Especially,
this causes unnecessary cost on Android when the top control shows or
hides.

Avoid repaint if the RenderView doesn't needs full layout.

BUG=258219
BUG=327815(about RenderBlock::percentHeightDescendants)

TEST=WebFrameTest.heightChangeRepaint
TEST=all layout tests still pass

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164645 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Make some TreeScope methods public on ShadowRoot instead of friending TreeScopeAdopter

setDocument and setParentTreeScope only make sense on ShadowRoot, not Document, so make
them protected in TreeScope and public in ShadowRoot. This allows the removal of the
friend declaration for TreeScopeAdopter.

Also use references instead of pointers where they make sense.

R=esprehn@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164646 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Speculative fix for windows hangs in run-webkit-tests.

It looks like we may be seeing cases where trying to kill a content_shell
process doesn't actually kill everything. It's possible that passing the
/t flag to taskkill in addition to /f will help.

TBR=cpu@chromium.org
BUG=309787

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164647 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Shape-inside should not be shared to descendants of non-blocks

Certain non-block elements may have render descendants, such as the
controls for a video element. In this case, the shape inside should
not be shared, and the descendant content should lay out as if the
shape were not present.

BUG=330750

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

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