views: don't make device chooser's client view huge

views: don't make device chooser's client view huge

ChooserDialogView's client view does not need to be 402x320, since the
DeviceChooserContentView sizes itself. Worse, explicitly sizing the
ChooserDialogView's client view causes the dialog to double in height.

BUG=706337
Review-Url: https://codereview.chromium.org/2804303002
Cr-Commit-Position: refs/heads/master@{#463225}
Committed: https://chromium.googlesource.com/chromium/src/+/48fd59f0ef21b6daffc42147ac15bff3aefd4e67

[Elly Fong-Jones, 2017-04-07 17:01:40]

Description was changed from

==========
views: don't make device chooser's client view huge

ChooserDialogView's client view does not need to be 402x320, since the
DeviceChooserContentView sizes itself. Worse, explicitly sizing the
ChooserDialogView's client view causes the dialog to double in height.

BUG=706337
==========

to

==========
views: don't make device chooser's client view huge

ChooserDialogView's client view does not need to be 402x320, since the
DeviceChooserContentView sizes itself. Worse, explicitly sizing the
ChooserDialogView's client view causes the dialog to double in height.

BUG=706337
==========

[Elly Fong-Jones, 2017-04-07 17:01:40]

ellyjones@chromium.org changed reviewers:
+ benwells@chromium.org

[Elly Fong-Jones, 2017-04-07 17:01:46]

benwells: ptal? :)

[benwells, 2017-04-10 05:47:02]

lgtm

[Elly Fong-Jones, 2017-04-10 11:12:26]

The CQ bit was checked by ellyjones@chromium.org

[commit-bot: I haz the power, 2017-04-10 11:12:35]

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-04-10 11:55:50]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491822746890400, "parent_rev":
"6579200aa9189bce1cd61c05fb6eb4493f2302d8", "commit_rev":
"48fd59f0ef21b6daffc42147ac15bff3aefd4e67"}

[commit-bot: I haz the power, 2017-04-10 11:56:35]

Description was changed from

==========
views: don't make device chooser's client view huge

ChooserDialogView's client view does not need to be 402x320, since the
DeviceChooserContentView sizes itself. Worse, explicitly sizing the
ChooserDialogView's client view causes the dialog to double in height.

BUG=706337
==========

to

==========
views: don't make device chooser's client view huge

ChooserDialogView's client view does not need to be 402x320, since the
DeviceChooserContentView sizes itself. Worse, explicitly sizing the
ChooserDialogView's client view causes the dialog to double in height.

BUG=706337

Review-Url: https://codereview.chromium.org/2804303002
Cr-Commit-Position: refs/heads/master@{#463225}
Committed:
https://chromium.googlesource.com/chromium/src/+/48fd59f0ef21b6daffc42147ac15...
==========

[commit-bot: I haz the power, 2017-04-10 11:56:36]

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