fast/media/color-does-not-include-alpha.html passes now

BUG=243593
TBR=jamesr@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160761 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add a comment explaining what RoundedRect is

...and why it uses integer units.

R=eae@chromium.org
BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160762 bbb929c8-8fbe-4397-9dbb-9b2b20218538
[SVG] invalidateRelativeLengthClients() should also clear resource caches

After r160730 we are no longer clearing resource container caches on
layout. Instead, we're relying on the layout invalidation path to handle
this task - and that includes SVGElement::invalidateRelativeLengthClients().

(This problem is only triggered under the mask/clip caching CL).

R=pdr@chromium.org,schenney@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160763 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Don't allow style sharing of customized styles

There were many users of customStyleForRenderer() that were creating a
custom style and then not marking it as unique. This normally wasn't
an issue since the elements don't tend to be siblings of each other or
at the same level of the tree, but could manifest in future form control
types or the PasswordGeneratorButtonElement if we moved the calls to
addToStyleSharingList (like future patches might).

This patch adds all the missing setUnique() calls, but we should probably
consider some more general solution in the future or get rid of the
customStyleForRenderer() hook which is being used in bad ways all over
the UA shadow elements.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160764 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove unused ShadowBlur

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160765 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move BreakingContext out of LineBreaker

This patch just moves out BreakingContext from the scope of LineBreaker. The goal is to separate BreakingContext
into its own file (https://chromiumcodereview.appspot.com/25054004#msg2). I want to do this step by step. This change
moves out BC and leaves it in place. I'll make a separate patch to add the new file for it.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160766 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move part of the accelerated scrolling code

This change moves half of the accelerated scrolling functions
to RenderLayerScrollableArea. This removes all functions from
ScrollableArea on RenderLayer (though there are more to move).

While touching ScrollingCoordinatorChromiumTest, changed the
testing to be RenderBox based so as to minimize future changes.

BUG=260899

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160767 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Range should not use confusingAndOftenMisusedAttached()

Range is really checking inActiveDocument() and is confused.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160768 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Have ContainerNodeAlgorithms deal with references instead of pointers

Have ContainerNodeAlgorithms deal with references instead of pointers
since those can never be null. This makes the code look safer.

R=adamk

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160769 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Update transitions/cubic-bezier-overflow-svg-length.html to use stroke-dashoffset

This test attempts to test that transitions with cubic-bezier timing functions
are able to produce values for SVG length properties that are outside the range
of the 'from' and 'to' keyframes. It uses the stroke-width property to do this,
but this property should not permit negative values. This is enforced by the
Web Animations implementation, but not the legacy implementation, which
complicates the test.

This change switches the test to use the stroke-dashoffset property, which
permits negative values.

Note that clamping of stroke-width values is tested in
animations/interpolation/svg-stroke-width-interpolation.html.

R=timloh

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160770 bbb929c8-8fbe-4397-9dbb-9b2b20218538
accessKey should be compared case-insensitively.

This fixes a regression from r153757. I overlooked CaseFoldingHash specified to
HashMap<> included in the previous implementation, which caused capital
accessKeys to be totally ignored.

BUG=303204
R=jamesr@chromium.org

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

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