Make ShelfLayoutManagerTest.ShelfLayoutInUnifiedDesktop font config independent

Make ShelfLayoutManagerTest.ShelfLayoutInUnifiedDesktop font config independent

BUG=600404
R=jamescook@chromium.org

Committed: https://crrev.com/e8c18394d174db7d105416a87115131a13e1bbb1
Cr-Commit-Position: refs/heads/master@{#385774}

[oshima, 2016-04-07 10:49:57]

The status area size is font config dependent. I guess you have different font
settings and that's probably why.

[James Cook, 2016-04-07 15:27:09]

LGTM

nit: Update the CL description to mention which test this is.

Thanks for fixing it! I have no idea why my fonts are different.

https://codereview.chromium.org/1869723002/diff/1/ash/shelf/shelf_layout_mana...
File ash/shelf/shelf_layout_manager_unittest.cc (right):

https://codereview.chromium.org/1869723002/diff/1/ash/shelf/shelf_layout_mana...
ash/shelf/shelf_layout_manager_unittest.cc:2328: UpdateDisplay("500x400,
500x400");
Hooray for different width and height!

[oshima, 2016-04-07 15:42:18]

Description was changed from

==========
Make the test font config independent

BUG=600404
R=jamescook@chromium.org
==========

to

==========
Make ShelfLayoutManagerTest.ShelfLayoutInUnifiedDesktop font config independent

BUG=600404
R=jamescook@chromium.org
==========

[oshima, 2016-04-07 15:42:34]

On 2016/04/07 15:27:09, James Cook wrote:
> LGTM
> 
> nit: Update the CL description to mention which test this is.
Done.
> 
> Thanks for fixing it! I have no idea why my fonts are different.
> 
>
https://codereview.chromium.org/1869723002/diff/1/ash/shelf/shelf_layout_mana...
> File ash/shelf/shelf_layout_manager_unittest.cc (right):
> 
>
https://codereview.chromium.org/1869723002/diff/1/ash/shelf/shelf_layout_mana...
> ash/shelf/shelf_layout_manager_unittest.cc:2328: UpdateDisplay("500x400,
> 500x400");
> Hooray for different width and height!

[oshima, 2016-04-07 15:42:43]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2016-04-07 15:42:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1869723002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1869723002/1

[oshima, 2016-04-07 16:13:09]

The CQ bit was unchecked by oshima@chromium.org

[commit-bot: I haz the power, 2016-04-07 16:13:12]

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

[oshima, 2016-04-07 16:13:16]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2016-04-07 16:13:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1869723002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1869723002/1

[commit-bot: I haz the power, 2016-04-07 16:13:58]

Description was changed from

==========
Make ShelfLayoutManagerTest.ShelfLayoutInUnifiedDesktop font config independent

BUG=600404
R=jamescook@chromium.org
==========

to

==========
Make ShelfLayoutManagerTest.ShelfLayoutInUnifiedDesktop font config independent

BUG=600404
R=jamescook@chromium.org
==========

[commit-bot: I haz the power, 2016-04-07 16:14:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-07 16:15:17]

Description was changed from

==========
Make ShelfLayoutManagerTest.ShelfLayoutInUnifiedDesktop font config independent

BUG=600404
R=jamescook@chromium.org
==========

to

==========
Make ShelfLayoutManagerTest.ShelfLayoutInUnifiedDesktop font config independent

BUG=600404
R=jamescook@chromium.org

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

[commit-bot: I haz the power, 2016-04-07 16:15:18]

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