MD Settings: Add Bluetooth device type icons in device lists.

MD Settings: Add Bluetooth device type icons in device lists.

The mapping from device types to icons is consistent with the Bluetooth list in
the system menu.
Newly added device icons are copyed from
https://github.com/PolymerElements/iron-icons/blob/master/hardware-icons.html
with the same id.

BUG=681784
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2637053002
Cr-Commit-Position: refs/heads/master@{#444225}
Committed: https://chromium.googlesource.com/chromium/src/+/dcd83b32c7de902ee531d61a03762fbde0bc5ac8

[fukino, 2017-01-17 11:07:15]

Description was changed from

==========
MD Settings: Add Bluetooth device type icons in device lists.

The mapping from device types to icons is consistent with the Bluetooth list in
the system menu.
Newly added device icons are copyed from
https://github.com/PolymerElements/iron-icons/blob/master/hardware-icons.html
with the same id.

BUG=681784
==========

to

==========
MD Settings: Add Bluetooth device type icons in device lists.

The mapping from device types to icons is consistent with the Bluetooth list in
the system menu.
Newly added device icons are copyed from
https://github.com/PolymerElements/iron-icons/blob/master/hardware-icons.html
with the same id.

BUG=681784
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[fukino, 2017-01-17 11:24:31]

fukino@chromium.org changed reviewers:
+ stevenjb@chromium.org

[fukino, 2017-01-17 11:24:31]

Steven, could you take a look?

https://codereview.chromium.org/2637053002/diff/20001/chrome/browser/resource...
File
chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.html
(right):

https://codereview.chromium.org/2637053002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.html:35:
<iron-icon id="check" icon="cr:check" hidden$="[[!device.connected]]">
This checkmark icon will be replaced with "Connected" and "Not connected" label
in a separete CL.

[stevenjb, 2017-01-17 18:34:44]

lgtm w/ comment fix

https://codereview.chromium.org/2637053002/diff/20001/chrome/browser/resource...
File
chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.js
(right):

https://codereview.chromium.org/2637053002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.js:86:
* @param {!chrome.bluetooth.Device} device
@return {string}

[fukino, 2017-01-18 00:07:30]

Thanks!

https://codereview.chromium.org/2637053002/diff/20001/chrome/browser/resource...
File
chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.js
(right):

https://codereview.chromium.org/2637053002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/bluetooth_page/bluetooth_device_list_item.js:86:
* @param {!chrome.bluetooth.Device} device
On 2017/01/17 18:34:44, stevenjb wrote:
> @return {string}

Done.

[fukino, 2017-01-18 00:07:34]

The CQ bit was checked by fukino@chromium.org

[fukino, 2017-01-18 00:07:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
stevenjb@chromium.org
Link to the patchset: https://codereview.chromium.org/2637053002/#ps40001
(title: "Fix an annotation.")

[commit-bot: I haz the power, 2017-01-18 00:08:01]

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-01-18 01:24:20]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484698054312860,
"parent_rev": "d94c7b072f3b496815122c3c310456320d592c3c", "commit_rev":
"dcd83b32c7de902ee531d61a03762fbde0bc5ac8"}

[commit-bot: I haz the power, 2017-01-18 01:25:38]

Description was changed from

==========
MD Settings: Add Bluetooth device type icons in device lists.

The mapping from device types to icons is consistent with the Bluetooth list in
the system menu.
Newly added device icons are copyed from
https://github.com/PolymerElements/iron-icons/blob/master/hardware-icons.html
with the same id.

BUG=681784
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Add Bluetooth device type icons in device lists.

The mapping from device types to icons is consistent with the Bluetooth list in
the system menu.
Newly added device icons are copyed from
https://github.com/PolymerElements/iron-icons/blob/master/hardware-icons.html
with the same id.

BUG=681784
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2637053002
Cr-Commit-Position: refs/heads/master@{#444225}
Committed:
https://chromium.googlesource.com/chromium/src/+/dcd83b32c7de902ee531d61a0376...
==========

[commit-bot: I haz the power, 2017-01-18 01:25:39]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/dcd83b32c7de902ee531d61a0376...