Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(400)

Issue 283573002: Place primary screen before secondary screens in picker (Closed)

Created:
6 years, 7 months ago by robert.bradford
Modified:
6 years, 7 months ago
Reviewers:
Sergey Ulanov
CC:
chromium-reviews, fischman+watch_chromium.org, feature-media-reviews_chromium.org, wjia+watch_chromium.org, mcasas+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@output-order-fix
Visibility:
Public.

Description

Place primary screen before secondary screens in picker The DesktopMediaPicker treats the contents of the DesktopMediaList as an ordered list of sources. This change ensures the primary display is always in the first position in the picker. BUG=372444 TEST=Under ChromeOS connect/disconnect/reconnect secondary screen and observe that the primary display is in the first position in the picker. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271669

Patch Set 1 #

Total comments: 1

Patch Set 2 : Address nit #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -1 line) Patch
M chrome/browser/media/desktop_media_list_ash.cc View 1 1 chunk +5 lines, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
robert.bradford
6 years, 7 months ago (2014-05-12 17:22:14 UTC) #1
robert.bradford
Hi Sergey, could you take a look at this change please.
6 years, 7 months ago (2014-05-19 11:34:22 UTC) #2
Sergey Ulanov
LGTM. Sorry about the delay - I was on vacation last week. https://codereview.chromium.org/283573002/diff/1/chrome/browser/media/desktop_media_list_ash.cc File chrome/browser/media/desktop_media_list_ash.cc ...
6 years, 7 months ago (2014-05-19 18:59:33 UTC) #3
robert.bradford
The CQ bit was checked by robert.bradford@intel.com
6 years, 7 months ago (2014-05-20 11:13:34 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/robert.bradford@intel.com/283573002/40001
6 years, 7 months ago (2014-05-20 11:13:53 UTC) #5
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). Please consider checking whether the failures are ...
6 years, 7 months ago (2014-05-20 12:45:48 UTC) #6
commit-bot: I haz the power
6 years, 7 months ago (2014-05-20 15:55:55 UTC) #7
Message was sent while issue was closed.
Change committed as 271669

Powered by Google App Engine
This is Rietveld 408576698