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

Issue 679842 link

Starred by 1 user
Status: Fixed
Owner:
ajuma@chromium.org
Closed: Jan 2017
Cc:
weiliangc@chromium.org
ajuma@chromium.org
█ jaydasika@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Feature

Blocking:
issue 611883



Sign in to add a comment

Remove calls to owning_layer_->id() in RenderSurfaceImpl

Project Member Reported by ajuma@chromium.org, Jan 10 2017 
We can use RenderSurfaceImpl::stable_effect_id_ instead.

Comment 1 by ajuma@chromium.org, Jan 14 2017

Owner: ajuma@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 16 2017 

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

commit 847b8a76e4db2b92fef928dedd6afd2bd935add2
Author: ajuma <ajuma@chromium.org>
Date: Mon Jan 16 18:12:16 2017

cc: Remove calls to owning_layer_->id() in RenderSurfaceImpl

This replaces calls to owning_layer_->id() with calls to id(), which
currently happens to have the same value so there's no behavior change.
This is a step towards removing RenderSurfaceImpl's dependency on an
owning LayerImpl.

BUG= 679842 
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2635623002
Cr-Commit-Position: refs/heads/master@{#443922}

[modify] https://crrev.com/847b8a76e4db2b92fef928dedd6afd2bd935add2/cc/layers/render_surface_impl.cc

Comment 3 by ajuma@chromium.org, Jan 16 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment