GitLab

Per request of haraken:
https://codereview.chromium.org/212593008/#msg4

TBR=haraken

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

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

When calling RenderLayerClipper::backgroundClipRect on the
RenderLayerClipper object corresponding to the RenderLayer
which is equal to clipRectsContext.rootLayer, it first calls
parentClipRects on the clipper of that RenderLayer's parent.
In turn, this starts a recursive walk up the RenderLayer
tree in updateClipRects. This is incorrect, since a
RenderLayerClipper object is unable to clip above rootLayer
in the render tree. In particular, the recursion condition
in updateClipRects
depends on the recursion starting at a layer which is below
or equal to rootLayer.

BUG=336563

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

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

A bunch of code was needlessly null-checking enclosingLayer. That function can
only return zero for RenderObjects that are outside of the render tree.

R=esprehn@chromium.org

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

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

In Skia r13323, I changed Skia's image filter clipping behaviour to
expand clip bounds to accommodate filter margins. However, the legacy
Blink DAG already makes such accommodation, which means we're creating
an offscreen with redundant pixels. The fix is to specify the computed
bounds directly in saveLayer(), which overrides (well, actually
intersects) the clip, giving the correct bounds. This does give
different results for large blur sigmas, but I believe the new results
are spec-correct: the intermediate buffers used by filters should
never exceed the size of the filter region.

Covered by marked svg tests.

BUG=
R=schenney@chromium.org

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

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

BUG=357004
TBR=vsevik
NOTRY=true

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

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

Sample of the new output:
    InlineFlowBox       0x397ffa898130          RenderInline 0x397ffa840170 {pos=30,13 size=212.271,15} baseline=24/20


R=ojan@chromium.org
BUG=

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

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

Revert of Add runtime feature and property names for unprefixed CSS Transforms (https://codereview.chromium.org/211033002/)

Reason for revert:
Breaks perf test: src/tools/perf/tough_animation_cases/transform_transition_js_block.html

BUG=356859

Original issue's description:
> Add runtime feature and property names for unprefixed CSS Transforms
> 
> Adds a runtime flag (enabled only for testing) and defines property names
> for each property defined in CSS Transforms[1]. Future patches will make
> these properties functional.
> 
> [1] http://www.w3.org/TR/css3-transforms/
> 
> BUG=154772
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170005
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170149

TBR=eseidel@chromium.org,esprehn@chromium.org,dstockwell@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=154772

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

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

Reason for revert:
Breaks perf test: src/tools/perf/tough_animation_cases/transform_transition_js_block.html

BUG=356859

Original issue's description:
> CSS Transforms: Implement transform-style
> 
> Implements transform-style as defined in CSS Transforms[1]. Behavior is
> identical to -webkit-transform-style.
> 
> Note, this property is guarded by the CSSTransformsUnprefixed runtime feature
> (enabled only for testing).
> 
> [1] http://www.w3.org/TR/css3-transforms/
> 
> BUG=154772
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170013
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170166

TBR=eseidel@chromium.org,dstockwell@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=154772

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

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

Apparently, runOnOpenPopup1() function was being called again after invoking
iframe.contentWindow.location.reload(), right between finishing the execution of
runOnIFrameLoad() and finishJSTest(), in the Mac platform.

This patch avoids the issue by using one global flag to early return in
that function when the iframe has already been reloaded.

BUG=357176
R=tkent@chromium.org,eae@chromium.org

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

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

R=tkent@chromium.org
BUG=58719

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

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

Having both ::compare() and ::diff() was really confusing, this patch
renames them to stylePropagationDiff() and visualInvalidationDiff()
making their purpose more clear.

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

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

We were missing the xcode_settings OTHER_CFLAGS.

R=oilpan-reviews@chromium.org, tkent@chromium.org, zerny@chromium.org
BUG=

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

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

When the background of the table row is changed we repaint the entire table.
This CL adds a layout test showing the issue.

BUG=357291

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

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

Code to update the blending status in the RenderLayer hierarchy when a child is added or removed current invokes paintsWithBlendMode which calls compositingState() when it should not. This patch removes the compositing state check in this case, which means we may do more work.

All existing tests pass.

R=senorblanco@chromium.org
BUG=343759

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

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

In certain rare cases, history.{push,replace}State are called when during
plugin detach. In this case, it's possible for the state modification to be
requested on a frame that is half-way detached. The Document is still live,
but the DocumentLoader is gone. Exit early in this case.

BUG=356424

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

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

Revert of Make start/stop loading notifications per-frame (https://codereview.chromium.org/183793002/)

Reason for revert:
Possibly causing content_browsertests EndToEnd to fail.

See: http://build.chromium.org/p/chromium.webkit/builders/Linux%20Tests%20%28dbg%29/builds/2095/steps/content_browsertests/logs/stdio

Output:
[20917:20917:0327/122353:3268961829:INFO:CONSOLE(0)] "Uncaught TypeError: Cannot convert undefined or null to object", source: http://127.0.0.1:34379/files/web_ui_mojo.html (0)
[20917:20917:0327/122353:3269022437:INFO:CONSOLE(28)] "Uncaught TypeError: undefined is not a function", source: http://127.0.0.1:34379/files/web_ui_mojo.js (28)
BrowserTestBase signal handler received SIGTERM. Backtrace:
#0 0x000000a593be base::debug::StackTrace::StackTrace()
#1 0x000002bfc17b content::(anonymous namespace)::DumpStackTraceSignalHandler()
#2 0x7f9c3e4ab4a0 <unknown>
#3 0x7f9c3e55da43 __poll
#4 0x7f9c42aabff6 <unknown>
#5 0x7f9c42aac124 g_main_context_iteration
#6 0x000000aaaf5f base::MessagePumpGlib::RunWithDispatcher()
#7 0x000000a8e802 base::RunLoop::Run()
#8 0x00000058256e content::(anonymous namespace)::WebUIMojoTest_EndToEnd_Test::RunTestOnMainThread()

Original issue's description:
> Make start/stop loading notifications per-frame
> 
> Also, make ProgressTracker a frame-level concept rather than a page-level concept and merge it with FrameLoader's FrameProgressTracker helper. Send per-frame progress change notifications accordingly.
> 
> This depends on https://codereview.chromium.org/180113003/ landing in chromium.
> 
> BUG=347643
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170192

TBR=abarth@chromium.org,japhet@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=347643

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

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

It's possible that MemoryCache::replace will try to evict a resource that has
already been evicted. If there is another resource present in its place, it
won't get evicted, but will get partially overwritten. In that partial
overwrite, we leave references to it in the LRU lists, leading to later
crashes.

BUG=357174
TEST=MemoryCacheTest.RemoveDuringRevalidation

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

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

Use didNotifySubtreeInsertionsToDocument instead.

BUG=356095
TEST=automated
R=morrita@chromium.org

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

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

BUG=345027

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

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

BUG=352282

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

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

Traverse the selectorList() like we do for :host to find features for
invalidation sets. Relevance for 351613 is that we need to support this in
order to implement efficient invalidation sets for :hover with
childrenAffectedByHover due to UA stylesheet rules.

R=esprehn
BUG=356125,351613

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

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

Add Traversal<*Element>::firstAncestor() API to traverse elements of a specific
type. It is a common pattern in our code base to traverse the ancestors of a
Node until we find an Element of a specific type. This new firstAncestor() API
makes the code much simpler. This CL already makes use of the new API in
several places.

R=esprehn, eseidel
BUG=346733

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

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

This is a revert of https://src.chromium.org/viewvc/blink?view=rev&revision=168627.

BUG=254020

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

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

Chromium-side changes to parse and use optional WebM TrackEntry
DefaultDuration, derive simpleblock frame durations from subsequent
frame duration, or estimate missing frame duration as the maximum
duration seen so far cause changes in mediasource buffered range tests.

This change updates the tests relative to the new WebM frame duration
parsing/derivation/defaulting.

Also includes TestExpectations update to allow these tests to fail until Chromium side lands.

BUG=351166
TBR=acolwell@chromium.org
TEST=All media layout tests pass locally on Linux (with the Chromium prereq CLs applied), and failure is allowed by TestExpectations (without the prereq CLs applied)

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

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

This is a commit to make https://codereview.chromium.org/205263004/
apply cleanly.

TBR=jamesr@chromium.org
BUG=345027

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

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

BUG=356839

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

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

This patch changes SetPropertyTextAction to use setStyleText method
instead of setPropertyText for the undo operation. This
makes it possible to successfully undo multiple property insertions.

BUG=341506

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

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

This is a follow-up to r170203 to remove even more carriage returns.

TBR=jamesr@chromium.org
BUG=345027

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

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

This is a pre-commit for https://codereview.chromium.org/205263004/ to
change the line-endings this file in that patch so that the patch can be
applied cleanly in rietveld.

TBR=jamesr@chromium.org
BUG=345027

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

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

The previous behavior was to select all on mouse down and clear on mouse up. This matched the Cocoa NSSearchField behavior.
On Android the button may be so small users couldn't tap it reliably and only mousedown would fire.

BUG=329326

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

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

wrap-unwrap.html becomes these files:

 * wrapKey-unextractable.html
 * wrapKey-lacks-usage.html
 * wrapKey-badParameters.html
 * unwrapKey-lacks-usage.html
 * unwrapKey-badParameters.html

BUG=245025

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

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

When firing null-events stop DeviceSensorEventController only after the event has propagated to window.

When no sensors are available chrome should fire an all-null event
into blink. However at the time of firing the active DOM objects
can be suspended or stopped, which can prohibit propagation of the
event to window. Currently the controller will be unregistered in
any case because the event is a null-event. This results in the
potential loss of null-events and the javascript callback is never
executed. To avoid this issue the firing of a null-event should
happen until it effectively propagates to window.

BUG=356750
TEST=http://jsbin.com/pivimoga/1/ (manual)
TEST=https://codereview.chromium.org/214763002/ (browsertest)

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

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

R=wibling@chromium.org,haraken@chromium.org
BUG=340522

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

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

Gauge if removing these compatibility properties is feasible.

R=jochen@chromium.org
BUG=357112

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

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

This is a follow up that adds a simple layout test to test
https://codereview.chromium.org/201673004/

This test creates an audio context at 48 kHz and a 3 kHz sine wave in
an AudioBuffer that is played through the context. The test verifies
that the output is sufficiently close to the expected sine wave output.

BUG=344375

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

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

This is split out from 206533004, as it's largely unrelated to
the remaining changes there.

BUG=

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

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

allows to have less function paramenters. This patch also
streamlines a bit the code in calculateUnscaledKernelSize.

BUG=356329

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

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

Revert of Avoid unnecessary descents into nested boxes with floats. (https://codereview.chromium.org/195363007/)

Reason for revert:
Caused a lot of float crashes in ClusterFuzz
https://code.google.com/p/chromium/issues/detail?id=357242

Original issue's description:
> Avoid unnecessary descents into nested boxes with floats.
> 
> Steal a bit from RenderBlock to avoid descending through the same
> stack of nested boxes multiple times while preparing to lay them out.
> 
> 
> This CL creates a 4% improvement in the 
> Performance/Layout/floats_*_nested.html tests.
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170060

TBR=ojan@chromium.org,leviw@chromium.org,bjonesbe@adobe.com,jchaffraix@chromium.org
NOTREECHECKS=true
NOTRY=true

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

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

beyond the end of the color table.  Instead, draw these pixels as black.

This matches Firefox and IE.

BUG=176678
TEST=none
R=abarth@chromium.org, hclam@chromium.org

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

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

Also, make ProgressTracker a frame-level concept rather than a page-level concept and merge it with FrameLoader's FrameProgressTracker helper. Send per-frame progress change notifications accordingly.

This depends on https://codereview.chromium.org/180113003/ landing in chromium.

BUG=347643

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

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