See the corresponding TODO in ash/common/shelf/shelf_widget.cc.

Consider that docked windows are going away so this part of work could become redundant.

Ben is this still started (m-58) or should we punt to 59 / remove milestone label?

This was originally filed under the assumption that the |opaque_layer_| was expensive, however it is a SOLID_COLOR which is not terribly expensive.

So I am downgrading this priority, if we choose to not proceed with the work a CL to remove the TODO's in the code should at least be submitted.

Is DockedBackgroundWidget still a thing or has it been removed?

It appears to be a thing still.

#11, will probably no longer be there with https://codereview.chromium.org/2700523004/

Re-assigning to afakhry@ re #11.

Sorry, miscommunication, I will take this one back.

I'm planning to mark this one as a WontFix because the |opaque_background_| layer is a SOLID_COLOR layer which is cheaper than originally thought.  The only reason I haven't closed the issue is because there are some TODO's in the code that should be cleaned up.

DockedBackgroundWidget has since been removed, updating title to reflect that.

Code TODO is being removed here: https://codereview.chromium.org/2887463003/

Will mark this as a WontFix once that lands...

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

commit ce7c55b8355104296e2c369333b591ad9e195331
Author: bruthig <bruthig@chromium.org>
Date: Mon May 15 21:47:21 2017

[ash-md] Removed unwanted TODO from ShelfWidget::DelegateView::opaque_background_

Removinga stale TODO after some learning the |opaque_background_| is a good
implementation.

BUG= 621551 

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

[modify] https://crrev.com/ce7c55b8355104296e2c369333b591ad9e195331/ash/shelf/shelf_widget.cc

I assume you meant Fixed?

wontfix seems appropriate given the title of the bug