MD Settings: Fix endless recursion when exiting CBD dialog.

MD Settings: Fix endless recursion when exiting CBD dialog.

BUG=703992
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2763423002
Cr-Commit-Position: refs/heads/master@{#458871}
Committed: https://chromium.googlesource.com/chromium/src/+/bd09b2036a499ad0be4d31a51898c33d20252878

[dpapad, 2017-03-22 18:02:56]

Description was changed from

==========
Candidate fix for endless recursion.

BUG=
==========

to

==========
Candidate fix for endless recursion.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2017-03-22 18:55:31]

Description was changed from

==========
Candidate fix for endless recursion.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Fix endless recursion when exiting CBD dialog.

BUG=703992
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2017-03-22 18:55:57]

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

[commit-bot: I haz the power, 2017-03-22 18:57:08]

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

[dpapad, 2017-03-22 19:00:20]

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

[dpapad, 2017-03-22 19:00:34]

Patchset #3 (id:40001) has been deleted

[commit-bot: I haz the power, 2017-03-22 19:00:37]

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

[dpapad, 2017-03-22 20:18:11]

dpapad@chromium.org changed reviewers:
+ tommycli@chromium.org

[dpapad, 2017-03-22 20:18:12]

https://codereview.chromium.org/2763423002/diff/60001/chrome/test/data/webui/...
File chrome/test/data/webui/settings/route_tests.js (right):

https://codereview.chromium.org/2763423002/diff/60001/chrome/test/data/webui/...
chrome/test/data/webui/settings/route_tests.js:109: settings.Route.BASIC);
I had to update this test too, since it assumed that after going back user
should be on chrome://md-settings/advanced. Let me know if you think this
behavior will cause any problems.

[tommycli, 2017-03-22 20:25:50]

perfect lgtm

[commit-bot: I haz the power, 2017-03-22 20:35:54]

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

[commit-bot: I haz the power, 2017-03-22 20:35:56]

Dry run: This issue passed the CQ dry run.

[dpapad, 2017-03-22 20:47:38]

The CQ bit was checked by dpapad@chromium.org

[commit-bot: I haz the power, 2017-03-22 20:48: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, 2017-03-22 20:55:17]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490215658440730,
"parent_rev": "6a8f2bd33c4e705dfec6750a47240d4060ddf534", "commit_rev":
"bd09b2036a499ad0be4d31a51898c33d20252878"}

[commit-bot: I haz the power, 2017-03-22 20:56:02]

Description was changed from

==========
MD Settings: Fix endless recursion when exiting CBD dialog.

BUG=703992
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Fix endless recursion when exiting CBD dialog.

BUG=703992
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[commit-bot: I haz the power, 2017-03-22 20:56:04]

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