Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(110)

Issue 75303002: [Blending and compositing] Implement the isolation CSS property in software (Closed)

Created:
7 years, 1 month ago by mitica
Modified:
7 years, 1 month ago
Reviewers:
Stephen Chennney
CC:
blink-reviews, zoltan1, bemjb+rendering_chromium.org, eae+blinkwatch, leviw+renderwatch, dglazkov+blink, apavlov+blink_chromium.org, jchaffraix+rendering, darktears
Visibility:
Public.

Description

[Blending and compositing] Implement the isolation CSS property in software Spec: http://dev.w3.org/fxtf/compositing-1/#isolation Implement "isolation:isolate" for HTML elements under the compositing runtime flag. BUG=229615 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=162219

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+107 lines, -2 lines) Patch
A LayoutTests/css3/compositing/isolation-isolate-blended-child.html View 1 chunk +33 lines, -0 lines 0 comments Download
A LayoutTests/css3/compositing/isolation-isolate-blended-child-expected.html View 1 chunk +34 lines, -0 lines 0 comments Download
A LayoutTests/css3/compositing/isolation-isolate-simple.html View 1 chunk +22 lines, -0 lines 0 comments Download
A + LayoutTests/css3/compositing/isolation-isolate-simple-expected.txt View 1 chunk +1 line, -1 line 0 comments Download
M Source/core/css/resolver/StyleAdjuster.cpp View 1 chunk +1 line, -0 lines 0 comments Download
M Source/core/rendering/RenderObject.h View 2 chunks +3 lines, -1 line 0 comments Download
M Source/core/rendering/RenderObject.cpp View 1 chunk +5 lines, -0 lines 0 comments Download
M Source/core/rendering/style/RenderStyle.h View 1 chunk +1 line, -0 lines 0 comments Download
M Source/core/rendering/style/RenderStyle.cpp View 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
mitica
Hi Stephen, can you please review this change? Thanks!
7 years, 1 month ago (2013-11-18 13:38:03 UTC) #1
Stephen Chennney
LGTM
7 years, 1 month ago (2013-11-18 15:09:27 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mitica@adobe.com/75303002/1
7 years, 1 month ago (2013-11-18 17:39:33 UTC) #3
commit-bot: I haz the power
7 years, 1 month ago (2013-11-18 18:44:54 UTC) #4
Message was sent while issue was closed.
Change committed as 162219

Powered by Google App Engine
This is Rietveld 408576698