New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 740678 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Remove PropertyTreeState iterator and PropertyTreeState::GetInnermostNode

Project Member Reported by pdr@chromium.org, Jul 10 2017

Issue description

Labels: BugSource-Team PaintTeamTriaged-20170711
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f34f89d41a332a0bb31b34f0e51d9e494b37f912

commit f34f89d41a332a0bb31b34f0e51d9e494b37f912
Author: Philip Rogers <pdr@chromium.org>
Date: Sat Jul 29 00:42:55 2017

Remove PropertyTreeState::GetInnermostNode

This code was replaced by a new approach in [1] and is no longer used. The
README.md has been updated to refer to the new description found in
ConversionContext::Convert.

[1] https://chromium.googlesource.com/chromium/src/+/f62169afa7f911143f50059c52ac0d9442284e96

Bug:  740678 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: If26626568df7cb4cf342a55366b8b86d07a794f2
Reviewed-on: https://chromium-review.googlesource.com/592155
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#490559}
[modify] https://crrev.com/f34f89d41a332a0bb31b34f0e51d9e494b37f912/third_party/WebKit/Source/platform/graphics/compositing/README.md
[modify] https://crrev.com/f34f89d41a332a0bb31b34f0e51d9e494b37f912/third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.cpp
[modify] https://crrev.com/f34f89d41a332a0bb31b34f0e51d9e494b37f912/third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.h
[modify] https://crrev.com/f34f89d41a332a0bb31b34f0e51d9e494b37f912/third_party/WebKit/Source/platform/graphics/paint/PropertyTreeStateTest.cpp

Comment 3 by pdr@chromium.org, Jul 29 2017

Status: Fixed (was: Available)

Sign in to add a comment