MD system tray: Avoid adding separator as the first item of Bluetooth device list.

MD system tray: Avoid adding separator as the first item of Bluetooth device list.

In pre-MD device list, TrayDetailsView::AddScrollSeparator() checked if the added
separator was the first item or not.
https://codesearch.chromium.org/chromium/src/ash/common/system/tray/tray_details_view.cc?q=TrayDetailsView::AddScrollSeparator&sq=package:chromium&l=430&dr=CSs
We should do the same check in MD version of Bluetooth device list, too.

BUG=675566
Review-Url: https://codereview.chromium.org/2582293003
Cr-Commit-Position: refs/heads/master@{#442473}
Committed: https://chromium.googlesource.com/chromium/src/+/b613df596c63e3e69b4addaaa676e54877b8f261

[fukino, 2016-12-19 11:04:26]

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

[fukino, 2016-12-19 11:04:26]

Hi Jenny, could you take a look?

[fukino, 2016-12-22 05:35:38]

Friendly ping.

[jennyz, 2017-01-06 21:43:12]

lgtm

[tdanderson, 2017-01-09 20:39:10]

On 2017/01/06 21:43:12, jennyz wrote:
> lgtm

fukino@, friendly ping - is this ready to land?

[fukino, 2017-01-10 01:08:19]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2017-01-10 01:09:42]

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

[fukino, 2017-01-10 01:10:44]

On 2017/01/09 20:39:10, tdanderson wrote:
> On 2017/01/06 21:43:12, jennyz wrote:
> > lgtm
> 
> fukino@, friendly ping - is this ready to land?

I'm sorry!
I'm landing it, and I'll merge the patch asap.

[commit-bot: I haz the power, 2017-01-10 03:11:06]

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

[commit-bot: I haz the power, 2017-01-10 03:11:11]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)

[fukino, 2017-01-10 03:55:33]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2017-01-10 03:56:20]

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-10 04:41:03]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484020533712830, "parent_rev":
"dcb4b63e4b07bf55e5ad9c20e9346e42c7eeba84", "commit_rev":
"b613df596c63e3e69b4addaaa676e54877b8f261"}

[commit-bot: I haz the power, 2017-01-10 04:41:29]

Description was changed from

==========
MD system tray: Avoid adding separator as the first item of Bluetooth device
list.

In pre-MD device list, TrayDetailsView::AddScrollSeparator() checked if the
added
separator was the first item or not.
https://codesearch.chromium.org/chromium/src/ash/common/system/tray/tray_deta...
We should do the same check in MD version of Bluetooth device list, too.

BUG=675566
==========

to

==========
MD system tray: Avoid adding separator as the first item of Bluetooth device
list.

In pre-MD device list, TrayDetailsView::AddScrollSeparator() checked if the
added
separator was the first item or not.
https://codesearch.chromium.org/chromium/src/ash/common/system/tray/tray_deta...
We should do the same check in MD version of Bluetooth device list, too.

BUG=675566

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

[commit-bot: I haz the power, 2017-01-10 04:41:30]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b613df596c63e3e69b4addaaa676...