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

Issue 719846 link

Starred by 0 users

Issue metadata

Status: WontFix
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Suggest rename cc::DirectRenderer to cc::BaseRenderer

Project Member Reported by xing...@intel.com, May 9 2017

Issue description

direct_renderer.h comments that "Direct" refers to the fact that it does 
not delegate rendering to another compositor.(see historical 
DelegatingRenderer for reference). So DirectRenderer is in contrast with
DelegatingRenderer. 

Now that DelegatingRenderer is gone, there is no direct or delegating. 
Maybe BaseRenderer or Renderer is a more straightforward name.
 

Comment 1 by xing...@intel.com, May 9 2017

Description: Show this description

Comment 2 by xing...@intel.com, May 9 2017

Components: Internals>Compositing
Status: Assigned (was: Untriaged)
You're right about the history but I don't see that BaseRenderer is providing more value as a name. IMO not worth it to change this right now.

Comment 4 by xing...@intel.com, May 18 2017

Status: WontFix (was: Assigned)
OK, will close it.

Sign in to add a comment