Add 'Slow' to a test that timeouts after Skia roll.

BUG=259094
TBR=robertfillips@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@153936 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Mixed content iframe checks should use top

Prior to this CL, we were using the parent frame when checking iframes for
mixed content. Unfortunately, that breaks the iTunes download page because it
uses an HTTPS iframe to kick off the HTTP download.

This CL causes us to use the top-level frame for this check. That's reasonable
because if the top-level frame is HTTP, the user doesn't have any expectation
that they're being protected from network attackers

This change effectively reverts https://src.chromium.org/viewvc/blink?revision=150260&view=revision.  I tried partially reverting that revision by keeping the blocking behavior, but that broke a number of assumptions in Chromium about how didRunInsecureContent works.  It also broke the ReferrerPolicyTest browser tests in worrying ways.  I would like to get back to blocking mixed content iframes, but we'll need to do that without breaking the iTunes download page.

R=tsepez@chromium.org
BUG=244210

Review URL: https://chromiumcodereview.appspot.com/18104009

git-svn-id: svn://svn.chromium.org/blink/trunk@153937 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove IDBNotFoundError ExceptionCode

Instead of passing an ExceptionCode (int) around to signify
an exception, we now pass an ExceptionState object. This is
so that we can pass more data out than just a single int.

The ExceptionState object has methods for throwing an exception with an optional error message.

There are also temporary converters so that functions can
still use ExceptionCode until the entire code base has
transitioned. Once that is done the ExcetionState code can
be simplified further.

BUG=252233

Review URL: https://chromiumcodereview.appspot.com/18398002

git-svn-id: svn://svn.chromium.org/blink/trunk@153938 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move colorFromPrimitiveValue from StyleResolverState to TextLinkColors

Break dependencies on state for resolveColorFromPrimitiveValue and move
to TextLinkColors. Also renames it to colorFromPrimitiveValue as it
no longer modifies the state.

This change moves the flag preventing currentColor styles from being
cashed out out StyleResolverState and into RenderStyle where the other
similar flags are located. It also removes the dependency on Element
and RenderStyle by having the caller pass in the current color value.

BUG=259018
TEST=No new tests, covered by existing fast/css tests
R=dglazkov@chromium.org

Review URL: https://chromiumcodereview.appspot.com/18859010

git-svn-id: svn://svn.chromium.org/blink/trunk@153939 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Enable webaudio aac and mp3 tests by default on Android

BUG=258629

The decoders for aac and mp3 are always available on Android so we
don't need to test for their existence.  (Plus, we aren't using FFMPEG
on Android, so the test was incorrect anyway.)

Review URL: https://chromiumcodereview.appspot.com/18932004

git-svn-id: svn://svn.chromium.org/blink/trunk@153940 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add a unit test for Font code path selection.

This test is in preparation for optimizing the selection code. It
verifies the expected complex text ranges.

R=eae@chromium.org
BUG=251328

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

git-svn-id: svn://svn.chromium.org/blink/trunk@153941 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fix build on Windows after r153931.

MSVC doesn't seem to know that all possible enum values are in the
switch and erroneously warns that not all control paths return a
value.

BUG=none
R=dimich@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@153942 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Moving Canvas2DLayer* and DrawingBuffer tests from Source/WebKit/ to Source/core/

BUG=258542
TEST=webkit_unit_tests --gtest_filter="Canvas2DLayer*"

Review URL: https://chromiumcodereview.appspot.com/19005003

git-svn-id: svn://svn.chromium.org/blink/trunk@153943 bbb929c8-8fbe-4397-9dbb-9b2b20218538
VisibleUnits shouldn't use bloatedCharacters()

This CL converts this file over to using String::appendTo instead of
bloatedCharacters().

R=eseidel@chromium.org
BUG=254708

Review URL: https://chromiumcodereview.appspot.com/18970002

git-svn-id: svn://svn.chromium.org/blink/trunk@153944 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Split the giant applyProperty switch statement into a static function on StyleBuilder in preparation for moving it out of StyleResolver.cpp

The plan is eventually for the new (templated) StyleBuilder to hold all of
the css -> style type mappings.  Currently those type mappings are split
between 4 places:
1.  StyleResolver::applyProperty
2.  DeprecatedStyleBuilder::applyProperty
3.  StyleBuilder::applyProperty
4.  StyleResolver::applySVGProperty (in SVGCSSStyleSelector.cpp)

This moves the switch statement from StyleResolver::applyProperty
into a new static method StyleBuilder::applyOldProperty.

In the next patch, I'll move that whole function into StyleBuilderCustom.cpp
and then in the patch after I'll move all of StyleResolver::applySVGProperty
into StyleBuilder::applyProperty.

Then we'll only have 2 places to look for this code instead of 4.

BUG=237268
R=eae@chromium.org

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

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