Skip both paint and raster invalidation for LayoutSVGHiddenContainer subtree

Skip both paint and raster invalidation for LayoutSVGHiddenContainer subtree

BUG=722761
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2888253002
Cr-Commit-Position: refs/heads/master@{#473135}
Committed: https://chromium.googlesource.com/chromium/src/+/f89bf147022cf532a6f2456d5fed109301296d6f

[Xianzhu, 2017-05-18 00:41:17]

Description was changed from

==========
Skip both paint and raster invalidation for LayoutSVGHiddenContainer subtree

BUG=722761
==========

to

==========
Skip both paint and raster invalidation for LayoutSVGHiddenContainer subtree

BUG=722761
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Xianzhu, 2017-05-18 00:41:18]

The CQ bit was checked by wangxianzhu@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-18 00:43:13]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-18 02:59:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-18 02:59:04]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Xianzhu, 2017-05-18 03:07:00]

wangxianzhu@chromium.org changed reviewers:
+ chrishtr@chromium.org, fs@opera.com

[fs, 2017-05-18 08:52:42]

lgtm

https://codereview.chromium.org/2888253002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/paint/PaintInvalidator.h (right):

https://codereview.chromium.org/2888253002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/paint/PaintInvalidator.h:48: enum
ForcedSubtreeInvalidationFlag {
Nit: Drop the 'Forced' suffix here as well?

[fs, 2017-05-18 08:53:29]

https://codereview.chromium.org/2888253002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/paint/PaintInvalidator.h (right):

https://codereview.chromium.org/2888253002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/paint/PaintInvalidator.h:48: enum
ForcedSubtreeInvalidationFlag {
On 2017/05/18 at 08:52:41, fs wrote:
> Nit: Drop the 'Forced' suffix here as well?

Uhm, "prefix" is what I meant of course...

[chrishtr, 2017-05-18 15:27:17]

lgtm

[Xianzhu, 2017-05-18 15:42:54]

https://codereview.chromium.org/2888253002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/paint/PaintInvalidator.h (right):

https://codereview.chromium.org/2888253002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/paint/PaintInvalidator.h:48: enum
ForcedSubtreeInvalidationFlag {
On 2017/05/18 08:53:29, fs wrote:
> On 2017/05/18 at 08:52:41, fs wrote:
> > Nit: Drop the 'Forced' suffix here as well?
> 
> Uhm, "prefix" is what I meant of course...

Done.

[Xianzhu, 2017-05-18 15:43:40]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2017-05-18 15:43:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
chrishtr@chromium.org, fs@opera.com
Link to the patchset: https://codereview.chromium.org/2888253002/#ps20001
(title: "Rebaseline-cl etc.")

[commit-bot: I haz the power, 2017-05-18 15:44:18]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-18 17:34:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-18 17:34:34]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Xianzhu, 2017-05-19 03:35:33]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2017-05-19 03:35:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
chrishtr@chromium.org, fs@opera.com
Link to the patchset: https://codereview.chromium.org/2888253002/#ps40001
(title: "Rebaseline-cl")

[commit-bot: I haz the power, 2017-05-19 03:36:10]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-19 05:14:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-19 05:14:44]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Xianzhu, 2017-05-19 05:35:01]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2017-05-19 05:35:33]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-19 09:18:12]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1495172101402830,
"parent_rev": "19a83f8917e6a6df07d91cdaf3db4dc256f82390", "commit_rev":
"f89bf147022cf532a6f2456d5fed109301296d6f"}

[commit-bot: I haz the power, 2017-05-19 09:18:27]

Description was changed from

==========
Skip both paint and raster invalidation for LayoutSVGHiddenContainer subtree

BUG=722761
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Skip both paint and raster invalidation for LayoutSVGHiddenContainer subtree

BUG=722761
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2888253002
Cr-Commit-Position: refs/heads/master@{#473135}
Committed:
https://chromium.googlesource.com/chromium/src/+/f89bf147022cf532a6f2456d5fed...
==========

[commit-bot: I haz the power, 2017-05-19 09:18:31]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/f89bf147022cf532a6f2456d5fed...