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

Issue 568903002: Only have drawsContent() be true for graphics layers if not paintsIntoCompositedAncestor. (Closed)

Created:
6 years, 3 months ago by chrishtr
Modified:
6 years, 3 months ago
Reviewers:
ajuma
CC:
abarth-chromium, blink-reviews, blink-reviews-rendering, eae+blinkwatch, jchaffraix+rendering, leviw+renderwatch, pdr., rune+blink, zoltan1
Project:
blink
Visibility:
Public.

Description

Only have drawsContent() be true for graphics layers if not paintsIntoCompositedAncestor. BUG=409593 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181943

Patch Set 1 #

Patch Set 2 : Fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -7 lines) Patch
M Source/core/rendering/compositing/CompositedLayerMapping.cpp View 1 8 chunks +8 lines, -7 lines 0 comments Download

Messages

Total messages: 12 (5 generated)
chrishtr
These layers are also invalidated whenever you invalidate anything for m_graphicsLayer, so could be causing ...
6 years, 3 months ago (2014-09-12 21:22:09 UTC) #3
ajuma
On 2014/09/12 21:22:09, chrishtr wrote: > These layers are also invalidated whenever you invalidate anything ...
6 years, 3 months ago (2014-09-12 21:51:07 UTC) #4
chrishtr
On 2014/09/12 at 21:51:07, ajuma wrote: > On 2014/09/12 21:22:09, chrishtr wrote: > > These ...
6 years, 3 months ago (2014-09-12 21:53:45 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/568903002/20001
6 years, 3 months ago (2014-09-12 21:55:05 UTC) #7
commit-bot: I haz the power
Try jobs failed on following builders: android_chromium_gn_compile_rel on tryserver.blink (http://build.chromium.org/p/tryserver.blink/builders/android_chromium_gn_compile_rel/builds/10462)
6 years, 3 months ago (2014-09-12 22:09:46 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/568903002/20001
6 years, 3 months ago (2014-09-12 22:46:09 UTC) #11
commit-bot: I haz the power
6 years, 3 months ago (2014-09-12 23:01:22 UTC) #12
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as 181943

Powered by Google App Engine
This is Rietveld 408576698