ChromeOS: Fix "continue OOBE on restart" screen size calculation.

ChromeOS: Fix "continue OOBE on restart" screen size calculation.

This CL fixes sizes and styles of polymer-based OOBE after chrome restart.

BUG=604119
TEST=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/26ece4c3730b0800d6db484a421289fa03bc2288
Cr-Commit-Position: refs/heads/master@{#416150}

[Alexander Alekseev, 2016-08-31 12:09:39]

Description was changed from

==========
ChromeOS: Fix "continue OOBE on restart" screen size calculation.

This CL fixes sizes and styles of polymer-based OOBE after chrome restart.

BUG=604119
TEST=none
==========

to

==========
ChromeOS: Fix "continue OOBE on restart" screen size calculation.

This CL fixes sizes and styles of polymer-based OOBE after chrome restart.

BUG=604119
TEST=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Alexander Alekseev, 2016-08-31 12:09:53]

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

[commit-bot: I haz the power, 2016-08-31 12:10:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Alexander Alekseev, 2016-08-31 12:14:00]

alemate@chromium.org changed reviewers:
+ achuith@chromium.org

[Alexander Alekseev, 2016-08-31 12:14:01]

Before this CL flags to enable Material Design OOBE were set in reloadContent
OOBE method, which is not called when OOBE is restarted on the second and other
screens.

This CL moves enabling of new dialogs to screens itself and makes a few changes
to make size calculation consistent.

Please review.

[achuithb, 2016-09-01 22:47:20]

https://codereview.chromium.org/2302453002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/chromeos/login/oobe.js (right):

https://codereview.chromium.org/2302453002/diff/20001/chrome/browser/resource...
chrome/browser/resources/chromeos/login/oobe.js:311: // ---------- MD OOBE
screen
Do you still need this comment?

https://codereview.chromium.org/2302453002/diff/20001/chrome/browser/resource...
chrome/browser/resources/chromeos/login/oobe.js:332: respectMDMode_: function()
{
Better name? SetMDMode_? SwitchMDMode_?

https://codereview.chromium.org/2302453002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/chromeos/login/oobe_screen_network.js (right):

https://codereview.chromium.org/2302453002/diff/20001/chrome/browser/resource...
chrome/browser/resources/chromeos/login/oobe_screen_network.js:172:
respectMDMode_: function() {
Can we use a different name to avoid confusion during search?

[achuithb, 2016-09-01 22:47:26]

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

[achuithb, 2016-09-01 22:47:39]

lgtm

[commit-bot: I haz the power, 2016-09-01 22:48:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Alexander Alekseev, 2016-09-01 23:01:40]

https://codereview.chromium.org/2302453002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/chromeos/login/oobe.js (right):

https://codereview.chromium.org/2302453002/diff/20001/chrome/browser/resource...
chrome/browser/resources/chromeos/login/oobe.js:311: // ---------- MD OOBE
screen
On 2016/09/01 22:47:20, achuithb wrote:
> Do you still need this comment?

Done.

https://codereview.chromium.org/2302453002/diff/20001/chrome/browser/resource...
chrome/browser/resources/chromeos/login/oobe.js:332: respectMDMode_: function()
{
On 2016/09/01 22:47:20, achuithb wrote:
> Better name? SetMDMode_? SwitchMDMode_?

Done.

https://codereview.chromium.org/2302453002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/chromeos/login/oobe_screen_network.js (right):

https://codereview.chromium.org/2302453002/diff/20001/chrome/browser/resource...
chrome/browser/resources/chromeos/login/oobe_screen_network.js:172:
respectMDMode_: function() {
On 2016/09/01 22:47:20, achuithb wrote:
> Can we use a different name to avoid confusion during search?

Done.

[Alexander Alekseev, 2016-09-01 23:01:46]

The CQ bit was checked by alemate@chromium.org

[Alexander Alekseev, 2016-09-01 23:01:46]

The patchset sent to the CQ was uploaded after l-g-t-m from achuith@chromium.org
Link to the patchset: https://codereview.chromium.org/2302453002/#ps40001
(title: "Update after review")

[commit-bot: I haz the power, 2016-09-01 23:02:13]

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-09-02 01:07:46]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-02 01:09:33]

Description was changed from

==========
ChromeOS: Fix "continue OOBE on restart" screen size calculation.

This CL fixes sizes and styles of polymer-based OOBE after chrome restart.

BUG=604119
TEST=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
ChromeOS: Fix "continue OOBE on restart" screen size calculation.

This CL fixes sizes and styles of polymer-based OOBE after chrome restart.

BUG=604119
TEST=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/26ece4c3730b0800d6db484a421289fa03bc2288
Cr-Commit-Position: refs/heads/master@{#416150}
==========

[commit-bot: I haz the power, 2016-09-02 01:09:34]

Patchset 3 (id:??) landed as
https://crrev.com/26ece4c3730b0800d6db484a421289fa03bc2288
Cr-Commit-Position: refs/heads/master@{#416150}