Cleanup: Misc fixes to MediaDeviceNotifications.

Cleanup: Misc fixes to MediaDeviceNotifications.

BUG=none
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=137502

[vandebo (ex-Chrome), 2012-05-16 17:00:16]

LGTM

https://chromiumcodereview.appspot.com/10389161/diff/4002/chrome/browser/medi...
File
chrome/browser/media_gallery/media_device_notifications_window_win_unittest.cc
(right):

https://chromiumcodereview.appspot.com/10389161/diff/4002/chrome/browser/medi...
chrome/browser/media_gallery/media_device_notifications_window_win_unittest.cc:15:
#include "chrome/browser/media_gallery/media_device_notifications_window_win.h"
nit: it looks like most files in base have this file first for unit tests.  The
style guide doesn't call it out one way or another.  Do you have specific
knowledge about this style item?

[Lei Zhang, 2012-05-16 19:02:55]

https://chromiumcodereview.appspot.com/10389161/diff/4002/chrome/browser/medi...
File
chrome/browser/media_gallery/media_device_notifications_window_win_unittest.cc
(right):

https://chromiumcodereview.appspot.com/10389161/diff/4002/chrome/browser/medi...
chrome/browser/media_gallery/media_device_notifications_window_win_unittest.cc:15:
#include "chrome/browser/media_gallery/media_device_notifications_window_win.h"
On 2012/05/16 17:00:16, vandebo wrote:
> nit: it looks like most files in base have this file first for unit tests. 
The
> style guide doesn't call it out one way or another.  Do you have specific
> knowledge about this style item?

I remembered wrong. Reverted.

[commit-bot: I haz the power, 2012-05-16 19:04:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/10389161/5003

[commit-bot: I haz the power, 2012-05-16 20:43:52]

Change committed as 137502