Remove non-MD code from AudioDetailedView

Remove non-MD code from AudioDetailedView

BUG=687798
TEST=none

Review-Url: https://codereview.chromium.org/2724963002
Cr-Commit-Position: refs/heads/master@{#454602}
Committed: https://chromium.googlesource.com/chromium/src/+/93695f6919948c77e7961914429523485af7e5de

[mohsen, 2017-03-01 22:49:01]

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

[commit-bot: I haz the power, 2017-03-01 22:49:38]

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

[mohsen, 2017-03-01 23:01:45]

mohsen@chromium.org changed reviewers:
+ tdanderson@chromium.org

[mohsen, 2017-03-01 23:01:45]

Please take a look...

[tdanderson, 2017-03-02 20:48:50]

LGTM with one request

https://codereview.chromium.org/2724963002/diff/20001/ash/common/system/tray/...
File ash/common/system/tray/hover_highlight_view.cc (left):

https://codereview.chromium.org/2724963002/diff/20001/ash/common/system/tray/...
ash/common/system/tray/hover_highlight_view.cc:257: views::Label*
HoverHighlightView::AddCheckableLabel(const base::string16& text,
Thanks for removing this.

https://codereview.chromium.org/2724963002/diff/20001/ash/common/system/tray/...
ash/common/system/tray/hover_highlight_view.cc:266:
rb.GetImageNamed(IDR_MENU_CHECK).ToImageSkia();
Looks like this is the only place still using IDR_MENU_CHECK, can you please
remove it from the .grd and also delete the corresponding PNG(s)?

[mohsen, 2017-03-03 06:57:32]

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

[commit-bot: I haz the power, 2017-03-03 06:57:47]

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

[mohsen, 2017-03-03 06:59:18]

https://codereview.chromium.org/2724963002/diff/20001/ash/common/system/tray/...
File ash/common/system/tray/hover_highlight_view.cc (left):

https://codereview.chromium.org/2724963002/diff/20001/ash/common/system/tray/...
ash/common/system/tray/hover_highlight_view.cc:266:
rb.GetImageNamed(IDR_MENU_CHECK).ToImageSkia();
On 2017/03/02 at 20:48:49, tdanderson wrote:
> Looks like this is the only place still using IDR_MENU_CHECK, can you please
remove it from the .grd and also delete the corresponding PNG(s)?

Thanks for catching this. Done.

[commit-bot: I haz the power, 2017-03-03 07:00:05]

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

[commit-bot: I haz the power, 2017-03-03 07:00:06]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[mohsen, 2017-03-03 07:03:32]

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

[commit-bot: I haz the power, 2017-03-03 07:03:47]

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-03 07:52:17]

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

[commit-bot: I haz the power, 2017-03-03 07:52:17]

Dry run: This issue passed the CQ dry run.

[mohsen, 2017-03-03 16:57:39]

mohsen@chromium.org changed reviewers:
+ sadrul@chromium.org

[mohsen, 2017-03-03 16:57:40]

+sadrul@: for changes in ui/views/resources/.

[sadrul, 2017-03-03 17:00:59]

lgtm

[mohsen, 2017-03-03 17:05:53]

The CQ bit was checked by mohsen@chromium.org

[mohsen, 2017-03-03 17:05:53]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdanderson@chromium.org
Link to the patchset: https://codereview.chromium.org/2724963002/#ps60001
(title: "Rebased")

[commit-bot: I haz the power, 2017-03-03 17:06:08]

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-03 17:10:18]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1488560753211110,
"parent_rev": "c83e0b72cd34675858ed368cf3801184cfeec57f", "commit_rev":
"93695f6919948c77e7961914429523485af7e5de"}

[commit-bot: I haz the power, 2017-03-03 17:10:52]

Description was changed from

==========
Remove non-MD code from AudioDetailedView

BUG=687798
TEST=none
==========

to

==========
Remove non-MD code from AudioDetailedView

BUG=687798
TEST=none

Review-Url: https://codereview.chromium.org/2724963002
Cr-Commit-Position: refs/heads/master@{#454602}
Committed:
https://chromium.googlesource.com/chromium/src/+/93695f6919948c77e79619144295...
==========

[commit-bot: I haz the power, 2017-03-03 17:10:53]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/93695f6919948c77e79619144295...