[CrOS] Enumerate existing mount points and check for media devices.

[CrOS] Enumerate existing mount points and check for media devices.

BUG=none
TEST=none


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

[kmadhusu, 2012-08-03 06:16:24]

thestig@: When you get a chance, Can you please test this code on CrOS, just to
make sure we enumerate all the existing media devices? Sorry for troubling you.

Thanks.

[Lei Zhang, 2012-08-03 06:24:19]

lgtm

https://chromiumcodereview.appspot.com/10843061/diff/1/chrome/browser/media_g...
File chrome/browser/media_gallery/media_device_notifications_chromeos.h (right):

https://chromiumcodereview.appspot.com/10843061/diff/1/chrome/browser/media_g...
chrome/browser/media_gallery/media_device_notifications_chromeos.h:51: void
CheckExistingMountPoints();
How about:
// Checks existing mount points map for media devices. For each mount point,
call CheckMountedPathOnFileThread() below.
void CheckExistingMountPointsOnUIThread();

and add a DCHECK() in the impl.

[kmadhusu, 2012-08-03 06:37:02]

Addressed comment. Thanks.

https://chromiumcodereview.appspot.com/10843061/diff/1/chrome/browser/media_g...
File chrome/browser/media_gallery/media_device_notifications_chromeos.h (right):

https://chromiumcodereview.appspot.com/10843061/diff/1/chrome/browser/media_g...
chrome/browser/media_gallery/media_device_notifications_chromeos.h:51: void
CheckExistingMountPoints();
On 2012/08/03 06:24:19, Lei Zhang wrote:
> How about:
> // Checks existing mount points map for media devices. For each mount point,
> call CheckMountedPathOnFileThread() below.
> void CheckExistingMountPointsOnUIThread();
> 
> and add a DCHECK() in the impl.

Done.

[commit-bot: I haz the power, 2012-08-03 06:37:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kmadhusu@chromium.org/10843061/5001

[commit-bot: I haz the power, 2012-08-03 08:07:16]

Change committed as 149812