Remove repaint.js from svg tests that pass without it

This set of tests all attempt to do a "repaint" test but aren't about
painting at all.  If the display() call from repaint.js is removed,
these tests all continue to pass, indicating that their output pixels
are not affected by the WebTestProxy::displayRepaintMask that covers
part of the screen in grey to indicate what has changed.

Because of this, it also means that they are not a good candidate for a
true text-based repaint rect, as they are either spit out no repaints or
full-screen repaints, depending on when paint rect tracking begins.

My conclusion is that these tests are really trying to test some
incremental layout- or style-related change, and so merely forcing a
style/layout recalc, tickling some property, and outputting the final
test pixels is a sufficient test.

BUG=345027

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

git-svn-id: svn://svn.chromium.org/blink/trunk@170134 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Don't use stale compositing state in LocalFrame::nodeImage

This patch fixes a compositor chicken and egg bug in
LocalFrame::nodeImage where stale compositor state was read through
setBaseBackgroundColor. An updateLayoutAndStyleForPainting call has
been moved before setBaseBackgroundColor in nodeImage.

Additionally, two unnecessary protector RefPtrs have been removed
because the callers already ref'ed the objects. The Node RefPtr is
ref'ed by Clipboard, the nodeImage() caller. The LocalFrame RefPtr
is ref'ed by DragController::doSystemDrag.

As a cleanup, LocalFrame::nodeImage has been changed to
take a reference since the node cannot be null.

BUG=354373

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

git-svn-id: svn://svn.chromium.org/blink/trunk@170135 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Make the default testharness.js timeout longer than the LayoutTests timeout

With the previous defaults, tests could timeout before the LayoutTests
timeout, which would be reported as a failure. Making the timeout very
long should fix tests that are flaky with the 2 second timeout and will
make the SlowTests mechanism work for testharness.js tests, should the
need arise.

BUG=355489

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

git-svn-id: svn://svn.chromium.org/blink/trunk@170136 bbb929c8-8fbe-4397-9dbb-9b2b20218538
[webcrypto] Split exportKey.html test into smaller files.

Also adds additional tests for extractable=false.

BUG=245025

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

git-svn-id: svn://svn.chromium.org/blink/trunk@170137 bbb929c8-8fbe-4397-9dbb-9b2b20218538
[refactor] Split up the layout test sign-verify.html into smaller files.

Becomes:
 * hmac-sign-verify.html
 * sign-verify-badParameters.html

Motivation: I am splitting out per-algorithm tests so the tests are more readable.

Also changing how the error expectations are tested to simplify an upcoming change where the result is no longer just "null".

BUG=245025

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

git-svn-id: svn://svn.chromium.org/blink/trunk@170138 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add a missing null check in SliderThumbElement::stopDragging()

hostInput() is null checked in several other SliderThumbElement
functions and clearly can return null, so just handle it.

Add GCController.collect() to the test case to make the crash trigger
reliably, which was necessary in order to debug it.

BUG=356148

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

git-svn-id: svn://svn.chromium.org/blink/trunk@170139 bbb929c8-8fbe-4397-9dbb-9b2b20218538
CPUProfiler: convert Flame Chart into Icicle Chart

BUG=
R=pfeldman@chromium.org

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

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