Show device connection and paired status in chooser on non-Mac

Show device connection and paired status in chooser on non-Mac

This CL added code to show an icon for connected device in the
chooser, and added "- Paired" text after the device name if the
device is paired.

I uploaded some screenshots from ChromeOS in the issue page.

BUG=543466
Committed: https://crrev.com/1d833f2314aa7857d2ed53a53cf0bb59ef5b69f9
Cr-Commit-Position: refs/heads/master@{#418916}

[juncai, 2016-09-09 21:55:59]

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

[commit-bot: I haz the power, 2016-09-09 21:56:30]

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

[juncai, 2016-09-09 22:05:46]

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

[commit-bot: I haz the power, 2016-09-09 22:05:48]

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

[juncai, 2016-09-09 22:27:18]

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

[juncai, 2016-09-09 22:27:18]

Please take a look.

[msw, 2016-09-09 23:16:03]

lgtm with a nit

https://codereview.chromium.org/2330593002/diff/20001/chrome/browser/ui/views...
File chrome/browser/ui/views/chooser_content_view_unittest.cc (right):

https://codereview.chromium.org/2330593002/diff/20001/chrome/browser/ui/views...
chrome/browser/ui/views/chooser_content_view_unittest.cc:103:
EXPECT_EQ(l10n_util::GetStringFUTF16(
nit: make a local helper like base::string16 GetPairedText(std::string device)

[commit-bot: I haz the power, 2016-09-09 23:49:34]

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

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

Dry run: This issue passed the CQ dry run.

[juncai, 2016-09-14 23:14:36]

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

[commit-bot: I haz the power, 2016-09-14 23:15:14]

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

[juncai, 2016-09-15 00:57:14]

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

[commit-bot: I haz the power, 2016-09-15 00:58:08]

Dry run: 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-09-15 04:31:00]

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

[commit-bot: I haz the power, 2016-09-15 04:31:01]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[juncai, 2016-09-15 17:43:15]

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

[commit-bot: I haz the power, 2016-09-15 17:43:42]

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

[juncai, 2016-09-15 17:43:55]

https://codereview.chromium.org/2330593002/diff/20001/chrome/browser/ui/views...
File chrome/browser/ui/views/chooser_content_view_unittest.cc (right):

https://codereview.chromium.org/2330593002/diff/20001/chrome/browser/ui/views...
chrome/browser/ui/views/chooser_content_view_unittest.cc:103:
EXPECT_EQ(l10n_util::GetStringFUTF16(
On 2016/09/09 23:16:03, msw wrote:
> nit: make a local helper like base::string16 GetPairedText(std::string device)

Done.

[commit-bot: I haz the power, 2016-09-15 18:32:48]

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

[commit-bot: I haz the power, 2016-09-15 18:32:49]

Dry run: This issue passed the CQ dry run.

[juncai, 2016-09-15 18:38:45]

The CQ bit was checked by juncai@chromium.org

[juncai, 2016-09-15 18:38:45]

The patchset sent to the CQ was uploaded after l-g-t-m from msw@chromium.org
Link to the patchset: https://codereview.chromium.org/2330593002/#ps80001
(title: "address comments")

[commit-bot: I haz the power, 2016-09-15 18:39:27]

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-09-15 18:46:22]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-09-15 18:48:48]

Description was changed from

==========
Show device connection and paired status in chooser on non-Mac

This CL added code to show an icon for connected device in the
chooser, and added "- Paired" text after the device name if the
device is paired.

I uploaded some screenshots from ChromeOS in the issue page.

BUG=543466
==========

to

==========
Show device connection and paired status in chooser on non-Mac

This CL added code to show an icon for connected device in the
chooser, and added "- Paired" text after the device name if the
device is paired.

I uploaded some screenshots from ChromeOS in the issue page.

BUG=543466

Committed: https://crrev.com/1d833f2314aa7857d2ed53a53cf0bb59ef5b69f9
Cr-Commit-Position: refs/heads/master@{#418916}
==========

[commit-bot: I haz the power, 2016-09-15 18:48:49]

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