Fill |model_name| and |vendor_name| fields of StorageInfo for MTP devices.

Fill |model_name| and |vendor_name| fields of StorageInfo for MTP devices.

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

[kinaba, 2014-05-13 05:20:44]

PTAL.

[Lei Zhang, 2014-05-13 06:19:51]

lgtm

https://codereview.chromium.org/287613002/diff/1/components/storage_monitor/m...
File
components/storage_monitor/media_transfer_protocol_device_observer_linux_unittest.cc
(right):

https://codereview.chromium.org/287613002/diff/1/components/storage_monitor/m...
components/storage_monitor/media_transfer_protocol_device_observer_linux_unittest.cc:56:
vendor_name->clear();
maybe add a fake vendor / product name and check for them in the
BasicAttachDetach test?

[kinaba, 2014-05-13 06:29:31]

https://codereview.chromium.org/287613002/diff/1/components/storage_monitor/m...
File
components/storage_monitor/media_transfer_protocol_device_observer_linux_unittest.cc
(right):

https://codereview.chromium.org/287613002/diff/1/components/storage_monitor/m...
components/storage_monitor/media_transfer_protocol_device_observer_linux_unittest.cc:56:
vendor_name->clear();
On 2014/05/13 06:19:51, Lei Zhang wrote:
> maybe add a fake vendor / product name and check for them in the
> BasicAttachDetach test?

Sounds good. Done.

[kinaba, 2014-05-13 06:39:20]

The CQ bit was checked by kinaba@chromium.org

[commit-bot: I haz the power, 2014-05-13 06:39:27]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/kinaba@chromium.org/287613002/20001

[commit-bot: I haz the power, 2014-05-13 08:37:56]

Change committed as 270053