Justify the message center to the right of the screen (i.e. the same

Justify the message center to the right of the screen (i.e. the same
place as the system menu).

BUG=666625
Committed: https://crrev.com/cda46b04bb3184f699ad9dd44dcebe94fd6f5a59
Cr-Commit-Position: refs/heads/master@{#435733}

[Evan Stade, 2016-11-30 01:33:49]

estade@chromium.org changed reviewers:
+ tdanderson@chromium.org

[tdanderson, 2016-11-30 01:51:37]

https://codereview.chromium.org/2540053002/diff/1/ash/common/system/web_notif...
File ash/common/system/web_notification/web_notification_tray.cc (right):

https://codereview.chromium.org/2540053002/diff/1/ash/common/system/web_notif...
ash/common/system/web_notification/web_notification_tray.cc:382: if
(GetAnchorAlignment() == views::TrayBubbleView::ANCHOR_ALIGNMENT_BOTTOM) {
Don't you mean to check shelf alignment instead of anchor alignment here?

[Evan Stade, 2016-11-30 23:56:47]

https://codereview.chromium.org/2540053002/diff/1/ash/common/system/web_notif...
File ash/common/system/web_notification/web_notification_tray.cc (right):

https://codereview.chromium.org/2540053002/diff/1/ash/common/system/web_notif...
ash/common/system/web_notification/web_notification_tray.cc:382: if
(GetAnchorAlignment() == views::TrayBubbleView::ANCHOR_ALIGNMENT_BOTTOM) {
On 2016/11/30 01:51:37, tdanderson wrote:
> Don't you mean to check shelf alignment instead of anchor alignment here?

sure. It's functionally equivalent but I guess checking the shelf alignment is
slightly more correct and concise.

[Evan Stade, 2016-11-30 23:57:18]

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

[commit-bot: I haz the power, 2016-11-30 23:57:54]

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-12-01 00:26:38]

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

[commit-bot: I haz the power, 2016-12-01 00:26:39]

Dry run: This issue passed the CQ dry run.

[tdanderson, 2016-12-01 20:58:23]

LGTM

[Evan Stade, 2016-12-01 21:59:36]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2016-12-01 21:59: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-12-01 22:06:21]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480629576492560,
"parent_rev": "cb06ce19b3df563fd1cd490efb7eea88463cd0e4", "commit_rev":
"18a8641e22bf394398e91fe486a330b7ce90c174"}

[commit-bot: I haz the power, 2016-12-01 22:06:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-01 22:08:54]

Description was changed from

==========
Justify the message center to the right of the screen (i.e. the same
place as the system menu).

BUG=666625
==========

to

==========
Justify the message center to the right of the screen (i.e. the same
place as the system menu).

BUG=666625

Committed: https://crrev.com/cda46b04bb3184f699ad9dd44dcebe94fd6f5a59
Cr-Commit-Position: refs/heads/master@{#435733}
==========

[commit-bot: I haz the power, 2016-12-01 22:08:55]

Patchset 2 (id:??) landed as
https://crrev.com/cda46b04bb3184f699ad9dd44dcebe94fd6f5a59
Cr-Commit-Position: refs/heads/master@{#435733}