ExternalNavigationHandler: Avoid else-after-return.

ExternalNavigationHandler: Avoid else-after-return.

This makes the code much more readable and complies with the Chromium
style guide.

Committed: https://crrev.com/f463b2126f450e792e5d8fb6171d1b4292ba3579
Cr-Commit-Position: refs/heads/master@{#419979}

[Matt Giuca, 2016-09-21 00:39:52]

mgiuca@chromium.org changed reviewers:
+ mariakhomenko@chromium.org

[Matt Giuca, 2016-09-21 00:39:53]

Hi Maria,

I found this code a bit too nested so I removed some "else" clauses. Looks big
but it's actually just un-indenting mostly.

[Matt Giuca, 2016-09-21 00:41:06]

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

[commit-bot: I haz the power, 2016-09-21 00:42:23]

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, 2016-09-21 00:45:24]

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

[commit-bot: I haz the power, 2016-09-21 00:45:25]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Matt Giuca, 2016-09-21 02:58:30]

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

[commit-bot: I haz the power, 2016-09-21 02:58:45]

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

[Maria, 2016-09-21 03:59:28]

lgtm

[commit-bot: I haz the power, 2016-09-21 05:03:23]

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

[commit-bot: I haz the power, 2016-09-21 05:03:24]

Dry run: This issue passed the CQ dry run.

[Matt Giuca, 2016-09-21 05:30:14]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2016-09-21 05:30:34]

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-21 05:35:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-21 05:37:26]

Description was changed from

==========
ExternalNavigationHandler: Avoid else-after-return.

This makes the code much more readable and complies with the Chromium
style guide.
==========

to

==========
ExternalNavigationHandler: Avoid else-after-return.

This makes the code much more readable and complies with the Chromium
style guide.

Committed: https://crrev.com/f463b2126f450e792e5d8fb6171d1b4292ba3579
Cr-Commit-Position: refs/heads/master@{#419979}
==========

[commit-bot: I haz the power, 2016-09-21 05:37:27]

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