Settings Router Refactor: Remove one last stray subpage-back.

Settings Router Refactor: Remove one last stray subpage-back.

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

Committed: https://crrev.com/3ee2a94b5980556a55e93be8926a496c680cdd5c
Cr-Commit-Position: refs/heads/master@{#410872}

[tommycli, 2016-08-09 19:12:21]

Description was changed from

==========
Settings Router Refactor: Remove one last stray subpage-back.

BUG=632382
==========

to

==========
Settings Router Refactor: Remove one last stray subpage-back.

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

[tommycli, 2016-08-09 19:15:28]

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

[commit-bot: I haz the power, 2016-08-09 19:16:13]

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

[tommycli, 2016-08-09 19:17:01]

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

[tommycli, 2016-08-09 19:17:02]

dbeam: PTAL before someone notices it's broken thx ;)

[Dan Beam, 2016-08-09 19:18:46]

lgtm but can haz test or something?

[commit-bot: I haz the power, 2016-08-09 19:27:56]

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

[commit-bot: I haz the power, 2016-08-09 19:27:57]

Dry run: Try jobs failed on following builders:
  closure_compilation on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/closure_compila...)

[tommycli, 2016-08-09 20:20:24]

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

[commit-bot: I haz the power, 2016-08-09 20:21:16]

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

[tommycli, 2016-08-09 20:23:10]

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

[tommycli, 2016-08-09 20:23:37]

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

[commit-bot: I haz the power, 2016-08-09 20:23:57]

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

[tommycli, 2016-08-09 20:24:06]

On 2016/08/09 19:18:46, Dan Beam wrote:
> lgtm but can haz test or something?

Hi, I updated it to be an assert to fix closure compile error. (It was an assert
before too)

I also added a browsertest. Thanks!

[commit-bot: I haz the power, 2016-08-09 21:11:15]

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

[commit-bot: I haz the power, 2016-08-09 21:11:15]

Dry run: This issue passed the CQ dry run.

[Dan Beam, 2016-08-09 23:34:53]

lgtm

[tommycli, 2016-08-09 23:35:21]

On 2016/08/09 23:34:53, Dan Beam wrote:
> lgtm

thank you sir! submitting

[tommycli, 2016-08-09 23:35:26]

The CQ bit was checked by tommycli@chromium.org

[commit-bot: I haz the power, 2016-08-09 23:36:26]

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-08-09 23:42:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-09 23:45:46]

Description was changed from

==========
Settings Router Refactor: Remove one last stray subpage-back.

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

to

==========
Settings Router Refactor: Remove one last stray subpage-back.

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

Committed: https://crrev.com/3ee2a94b5980556a55e93be8926a496c680cdd5c
Cr-Commit-Position: refs/heads/master@{#410872}
==========

[commit-bot: I haz the power, 2016-08-09 23:45:50]

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