Convert MobileSetupUI to use the new navigation callbacks.

Convert MobileSetupUI to use the new navigation callbacks.

BUG=682002
Review-Url: https://codereview.chromium.org/2659753002
Cr-Commit-Position: refs/heads/master@{#446593}
Committed: https://chromium.googlesource.com/chromium/src/+/e56024e9bcb988df909d4914aa48ea762b5cd0d3

[jam, 2017-01-26 22:04:37]

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

[commit-bot: I haz the power, 2017-01-26 22:05:01]

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

[jam, 2017-01-26 22:05:18]

Description was changed from

==========
Convert MobileSetupUI to use the new navigation callbacks.

BUG=682002
==========

to

==========
Convert MobileSetupUI to use the new navigation callbacks.

BUG=682002
==========

[jam, 2017-01-26 22:05:19]

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

[jam, 2017-01-26 22:10:23]

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

[commit-bot: I haz the power, 2017-01-26 22:10:37]

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, 2017-01-26 23:13:15]

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

[commit-bot: I haz the power, 2017-01-26 23:13:16]

Dry run: This issue passed the CQ dry run.

[achuithb, 2017-01-27 01:14:59]

lgtm with optional nit

https://codereview.chromium.org/2659753002/diff/20001/chrome/browser/ui/webui...
File chrome/browser/ui/webui/chromeos/mobile_setup_ui.cc (right):

https://codereview.chromium.org/2659753002/diff/20001/chrome/browser/ui/webui...
chrome/browser/ui/webui/chromeos/mobile_setup_ui.cc:643: result_value);
Early return on error seems preferable instead of else?

[jam, 2017-01-27 04:06:24]

https://codereview.chromium.org/2659753002/diff/20001/chrome/browser/ui/webui...
File chrome/browser/ui/webui/chromeos/mobile_setup_ui.cc (right):

https://codereview.chromium.org/2659753002/diff/20001/chrome/browser/ui/webui...
chrome/browser/ui/webui/chromeos/mobile_setup_ui.cc:643: result_value);
On 2017/01/27 01:14:59, achuithb wrote:
> Early return on error seems preferable instead of else?

Done.

[jam, 2017-01-27 04:06:50]

The CQ bit was checked by jam@chromium.org

[jam, 2017-01-27 04:06:50]

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/2659753002/#ps40001
(title: "nit")

[commit-bot: I haz the power, 2017-01-27 04:07:23]

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-01-27 04:39:42]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485490010150890,
"parent_rev": "7cb7db5ae23a21d9267ce2e707a7eb6a1389e66e", "commit_rev":
"e56024e9bcb988df909d4914aa48ea762b5cd0d3"}

[commit-bot: I haz the power, 2017-01-27 04:40:14]

Description was changed from

==========
Convert MobileSetupUI to use the new navigation callbacks.

BUG=682002
==========

to

==========
Convert MobileSetupUI to use the new navigation callbacks.

BUG=682002

Review-Url: https://codereview.chromium.org/2659753002
Cr-Commit-Position: refs/heads/master@{#446593}
Committed:
https://chromium.googlesource.com/chromium/src/+/e56024e9bcb988df909d4914aa48...
==========

[commit-bot: I haz the power, 2017-01-27 04:40:15]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/e56024e9bcb988df909d4914aa48...