Remove pre-MD code from SystemTrayBubble.

Remove pre-MD code from SystemTrayBubble.

Delete TrayPopupItemContainer class, which was only used for adding a
hover effect that we don't want/use in MD.

BUG=687802
Review-Url: https://codereview.chromium.org/2758433002
Cr-Commit-Position: refs/heads/master@{#457462}
Committed: https://chromium.googlesource.com/chromium/src/+/992fa8feb97734b7f1facce0cd5feffbb98d15f6

[Evan Stade, 2017-03-16 00:26:05]

The CQ bit was checked by estade@chromium.org to run a CQ dry run

[Evan Stade, 2017-03-16 00:26:29]

estade@chromium.org changed reviewers:
+ jamescook@chromium.org

[commit-bot: I haz the power, 2017-03-16 00:26:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-16 01:54:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-16 01:54:32]

Dry run: This issue passed the CQ dry run.

[James Cook, 2017-03-16 16:06:02]

Code LGTM but see email thread where we're considering adding back some kind of
hover effect.

I recommend adding a README file in ash/common/system/tray that mentions that we
used to have hover highlighting and includes the filename
tray_popup_item_container.h and this CL -- that will make it easier to find in
git logs if we want to revive this code.

[Evan Stade, 2017-03-16 16:57:36]

On 2017/03/16 16:06:02, James Cook wrote:
> Code LGTM but see email thread where we're considering adding back some kind
of
> hover effect.
> 
> I recommend adding a README file in ash/common/system/tray that mentions that
we
> used to have hover highlighting and includes the filename
> tray_popup_item_container.h and this CL -- that will make it easier to find in
> git logs if we want to revive this code.

I'm pretty sure we won't want to revive this code either way. If we re-added a
highlight it would be an InkDropHighlight, not a background paint.

[James Cook, 2017-03-16 17:02:14]

On 2017/03/16 16:57:36, Evan Stade wrote:
> On 2017/03/16 16:06:02, James Cook wrote:
> > Code LGTM but see email thread where we're considering adding back some kind
> of
> > hover effect.
> > 
> > I recommend adding a README file in ash/common/system/tray that mentions
that
> we
> > used to have hover highlighting and includes the filename
> > tray_popup_item_container.h and this CL -- that will make it easier to find
in
> > git logs if we want to revive this code.
> 
> I'm pretty sure we won't want to revive this code either way. If we re-added a
> highlight it would be an InkDropHighlight, not a background paint.

OK. README doesn't sound useful then. Still LGTM

[Evan Stade, 2017-03-16 17:02:36]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2017-03-16 17:03:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-16 17:09:13]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489683756917900, "parent_rev":
"ed9e4bf5f9558fe1f3d30a92606d263b4f9a2d1a", "commit_rev":
"992fa8feb97734b7f1facce0cd5feffbb98d15f6"}

[commit-bot: I haz the power, 2017-03-16 17:09:55]

Description was changed from

==========
Remove pre-MD code from SystemTrayBubble.

Delete TrayPopupItemContainer class, which was only used for adding a
hover effect that we don't want/use in MD.

BUG=687802
==========

to

==========
Remove pre-MD code from SystemTrayBubble.

Delete TrayPopupItemContainer class, which was only used for adding a
hover effect that we don't want/use in MD.

BUG=687802

Review-Url: https://codereview.chromium.org/2758433002
Cr-Commit-Position: refs/heads/master@{#457462}
Committed:
https://chromium.googlesource.com/chromium/src/+/992fa8feb97734b7f1facce0cd5f...
==========

[commit-bot: I haz the power, 2017-03-16 17:09:56]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/992fa8feb97734b7f1facce0cd5f...