Fix bug in specialized handler logic.

Fix bug in specialized handler logic.

It was too demanding requiring both authority and path rather than just
one. More info in attached bug.
Also begins adding unit tests for this code.

BUG=591280
Committed: https://crrev.com/2bf85a70160c1408738b63c850647dd6e594290a
Cr-Commit-Position: refs/heads/master@{#380970}

[Yaron, 2016-03-11 22:06:59]

yfriedman@chromium.org changed reviewers:
+ tedchoc@chromium.org

[Yaron, 2016-03-11 22:06:59]

I suspect current tests could be robolectric but this test suite if done fully
probably couldn't so I just did it as InstrumentationTestCase. I'll change it if
you feel strongly

[Yaron, 2016-03-11 22:07:31]

https://codereview.chromium.org/1786813003/diff/20001/chrome/android/javatest...
File
chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java
(right):

https://codereview.chromium.org/1786813003/diff/20001/chrome/android/javatest...
chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationDelegateImplTest.java:1:
// Copyright 2015 The Chromium Authors. All rights reserved.
note to self: it's now 2016

[Ted C, 2016-03-11 23:07:41]

lgtm

[Yaron, 2016-03-14 13:31:08]

The CQ bit was checked by yfriedman@chromium.org

[commit-bot: I haz the power, 2016-03-14 13:31:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1786813003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1786813003/20001

[Yaron, 2016-03-14 13:35:56]

The CQ bit was unchecked by yfriedman@chromium.org

[Yaron, 2016-03-14 13:36:43]

The CQ bit was checked by yfriedman@chromium.org

[Yaron, 2016-03-14 13:36:44]

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

[commit-bot: I haz the power, 2016-03-14 13:36:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1786813003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1786813003/40001

[commit-bot: I haz the power, 2016-03-14 14:31:52]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-14 14:32:51]

Description was changed from

==========
Fix bug in specialized handler logic.

It was too demanding requiring both authority and path rather than just
one. More info in attached bug.
Also begins adding unit tests for this code.

BUG=591280
==========

to

==========
Fix bug in specialized handler logic.

It was too demanding requiring both authority and path rather than just
one. More info in attached bug.
Also begins adding unit tests for this code.

BUG=591280

Committed: https://crrev.com/2bf85a70160c1408738b63c850647dd6e594290a
Cr-Commit-Position: refs/heads/master@{#380970}
==========

[commit-bot: I haz the power, 2016-03-14 14:32:52]

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