arc: Scale notification surface to fit

arc: Scale notification surface to fit

BUG=632720
BUG=b/30374625
TEST=Manual. Increase resolution and custom notification should always fill
    chrome notification space.

Committed: https://crrev.com/589b2256f24ac204c3fdc17a333aebeac3eb7c07
Cr-Commit-Position: refs/heads/master@{#408630}

[xiyuan, 2016-07-28 17:20:40]

xiyuan@chromium.org changed reviewers:
+ reveman@chromium.org, yoshiki@chromium.org

[reveman, 2016-07-28 18:50:53]

lgtm

https://codereview.chromium.org/2187363002/diff/40001/ui/arc/notification/arc...
File ui/arc/notification/arc_custom_notification_view.cc (right):

https://codereview.chromium.org/2187363002/diff/40001/ui/arc/notification/arc...
ui/arc/notification/arc_custom_notification_view.cc:106: surface_size !=
contents_size) {
nit: "surface_size != contents_size" check is not needed

[xiyuan, 2016-07-28 19:32:44]

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

[xiyuan, 2016-07-28 19:32:52]

https://codereview.chromium.org/2187363002/diff/40001/ui/arc/notification/arc...
File ui/arc/notification/arc_custom_notification_view.cc (right):

https://codereview.chromium.org/2187363002/diff/40001/ui/arc/notification/arc...
ui/arc/notification/arc_custom_notification_view.cc:106: surface_size !=
contents_size) {
On 2016/07/28 18:50:53, reveman wrote:
> nit: "surface_size != contents_size" check is not needed

Done.

[commit-bot: I haz the power, 2016-07-28 19:33:05]

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-07-28 20:15:56]

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

[commit-bot: I haz the power, 2016-07-28 20:15:57]

Dry run: This issue passed the CQ dry run.

[yoshiki, 2016-07-29 01:25:02]

lgtm

[xiyuan, 2016-07-29 13:13:59]

The CQ bit was checked by xiyuan@chromium.org

[xiyuan, 2016-07-29 13:13:59]

The patchset sent to the CQ was uploaded after l-g-t-m from reveman@chromium.org
Link to the patchset: https://codereview.chromium.org/2187363002/#ps80001
(title: "for nit in #3")

[xiyuan, 2016-07-29 13:14:01]

The CQ bit was unchecked by xiyuan@chromium.org

[xiyuan, 2016-07-29 13:37:43]

Description was changed from

==========
arc: Scale notification surface to fit

BUG=b/30374625
TEST=Manual. Increase resolution and custom notification should always fill
    chrome notification space.
==========

to

==========
arc: Scale notification surface to fit

BUG=632720
BUG=b/30374625
TEST=Manual. Increase resolution and custom notification should always fill
    chrome notification space.
==========

[xiyuan, 2016-07-29 13:38:03]

The CQ bit was checked by xiyuan@chromium.org

[commit-bot: I haz the power, 2016-07-29 13:38:14]

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-07-29 13:41:36]

Description was changed from

==========
arc: Scale notification surface to fit

BUG=632720
BUG=b/30374625
TEST=Manual. Increase resolution and custom notification should always fill
    chrome notification space.
==========

to

==========
arc: Scale notification surface to fit

BUG=632720
BUG=b/30374625
TEST=Manual. Increase resolution and custom notification should always fill
    chrome notification space.
==========

[commit-bot: I haz the power, 2016-07-29 13:41:37]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-07-29 13:43:57]

Description was changed from

==========
arc: Scale notification surface to fit

BUG=632720
BUG=b/30374625
TEST=Manual. Increase resolution and custom notification should always fill
    chrome notification space.
==========

to

==========
arc: Scale notification surface to fit

BUG=632720
BUG=b/30374625
TEST=Manual. Increase resolution and custom notification should always fill
    chrome notification space.

Committed: https://crrev.com/589b2256f24ac204c3fdc17a333aebeac3eb7c07
Cr-Commit-Position: refs/heads/master@{#408630}
==========

[commit-bot: I haz the power, 2016-07-29 13:43:58]

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