Fix blank names issue for navigator.usb.requestDevice

Fix blank names issue for navigator.usb.requestDevice

Sometimes UsbDevice::product_string() returns empty string, and the
current WebUSB chooser uses it as the device name to be displayed.
This CL fixes the blank name issue by displaying, in decreasing order
of preference:
1. UsbDevice::product_string()
2. UsbIds::GetProductName()
3. "Unknown device from " + UsbIds::GetVendorName()
4. "Unknown device [$VID:$PID]"

BUG=665599
Committed: https://crrev.com/06198969e3c76b4999cbdf1a474bb4f94ee275cd
Cr-Commit-Position: refs/heads/master@{#435019}

[juncai, 2016-11-28 23:08:19]

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

[commit-bot: I haz the power, 2016-11-28 23:09:35]

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

[juncai, 2016-11-28 23:10:14]

juncai@chromium.org changed reviewers:
+ reillyg@chromium.org

[juncai, 2016-11-28 23:10:15]

Please take a look.

[commit-bot: I haz the power, 2016-11-29 01:59:18]

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

[commit-bot: I haz the power, 2016-11-29 01:59:19]

Dry run: This issue passed the CQ dry run.

[Reilly Grant (use Gerrit), 2016-11-29 16:57:56]

lgtm

[juncai, 2016-11-29 17:16:45]

The CQ bit was checked by juncai@chromium.org

[commit-bot: I haz the power, 2016-11-29 17:16:57]

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, 2016-11-29 17:23:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480439805502540, "parent_rev":
"124042190216dad479e12e2a0c2300b10c6a1a7d", "commit_rev":
"146abad721746625071a6a599a5bb3ce92bb2568"}

[commit-bot: I haz the power, 2016-11-29 17:24:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-29 17:25:58]

Description was changed from

==========
Fix blank names issue for navigator.usb.requestDevice

Sometimes UsbDevice::product_string() returns empty string, and the
current WebUSB chooser uses it as the device name to be displayed.
This CL fixes the blank name issue by displaying, in decreasing order
of preference:
1. UsbDevice::product_string()
2. UsbIds::GetProductName()
3. "Unknown device from " + UsbIds::GetVendorName()
4. "Unknown device [$VID:$PID]"

BUG=665599
==========

to

==========
Fix blank names issue for navigator.usb.requestDevice

Sometimes UsbDevice::product_string() returns empty string, and the
current WebUSB chooser uses it as the device name to be displayed.
This CL fixes the blank name issue by displaying, in decreasing order
of preference:
1. UsbDevice::product_string()
2. UsbIds::GetProductName()
3. "Unknown device from " + UsbIds::GetVendorName()
4. "Unknown device [$VID:$PID]"

BUG=665599

Committed: https://crrev.com/06198969e3c76b4999cbdf1a474bb4f94ee275cd
Cr-Commit-Position: refs/heads/master@{#435019}
==========

[commit-bot: I haz the power, 2016-11-29 17:25:59]

Patchset 1 (id:??) landed as
https://crrev.com/06198969e3c76b4999cbdf1a474bb4f94ee275cd
Cr-Commit-Position: refs/heads/master@{#435019}