Fixing various problems with the new shelf layout in conjunction with the blue tray background on b…

Fixing various problems with the new shelf layout in conjunction with the blue tray background on bubble activation

There was a mismatch between the hover animation & the activation state. I cleaned up the activation code, removed the old override functions, and added comments on various places which had none yet.

BUG=274105
TEST=visual

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221597

[Mr4D (OOO till 08-26), 2013-09-04 21:55:41]

Please have a look!

[Harry McCleave, 2013-09-04 22:50:14]

On 2013/09/04 21:55:41, Mr4D wrote:
> Please have a look!

lgtm, ty for help /w this.

[jennyz, 2013-09-05 18:29:21]

lgtm with minor nit.

https://codereview.chromium.org/23531033/diff/1/ash/system/tray/tray_backgrou...
File ash/system/tray/tray_background_view.h (right):

https://codereview.chromium.org/23531033/diff/1/ash/system/tray/tray_backgrou...
ash/system/tray/tray_background_view.h:175: // background differently if set to
true.
nit: s/will store/stores? sounds more consistent with rest of the comment style.

[Mr4D (OOO till 08-26), 2013-09-05 18:46:32]

Addressed! Thanks!

https://codereview.chromium.org/23531033/diff/1/ash/system/tray/tray_backgrou...
File ash/system/tray/tray_background_view.h (right):

https://codereview.chromium.org/23531033/diff/1/ash/system/tray/tray_backgrou...
ash/system/tray/tray_background_view.h:175: // background differently if set to
true.
On 2013/09/05 18:29:22, jennyz wrote:
> nit: s/will store/stores? sounds more consistent with rest of the comment
style.

Done.

[commit-bot: I haz the power, 2013-09-05 18:47:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skuhne@chromium.org/23531033/7001

[commit-bot: I haz the power, 2013-09-06 06:09:23]

Change committed as 221597