Simplify "is external URL" in filter operations resolving

Simplify "is external URL" in filter operations resolving

We've already resolved the URL, so we might as well just use
equalIgnoringFragmentIdentifier on that rather than potentially
re-resolve and compare. No functional change.
Eventually we'll need to have the URL resolved before style resolution.

BUG=470608, 405315
Committed: https://crrev.com/1e10ae9fded23fccb138a01b662df295c4cf57d3
Cr-Commit-Position: refs/heads/master@{#406574}

[fs, 2016-07-20 14:24:19]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-20 14:24:43]

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

[fs, 2016-07-20 15:21:58]

fs@opera.com changed reviewers:
+ schenney@chromium.org

[Stephen Chennney, 2016-07-20 15:37:08]

Test? The bug reports seem to imply that a test can be constructed.

[fs, 2016-07-20 15:43:20]

On 2016/07/20 at 15:37:08, schenney wrote:
> Test? The bug reports seem to imply that a test can be constructed.

This doesn't fix anything (no functional change - added that to description.)
When I get to the actually fixing of the bug I'll add a test =).

[fs, 2016-07-20 15:43:32]

Description was changed from

==========
Simplify "is external URL" in filter operations resolving

We've already resolved the URL, so we might as well just use
equalIgnoringFragmentIdentifier on that rather than potentially
re-resolve and compare.
Eventually we'll need to have the URL resolved before style resolution.

BUG=470608, 405315
==========

to

==========
Simplify "is external URL" in filter operations resolving

We've already resolved the URL, so we might as well just use
equalIgnoringFragmentIdentifier on that rather than potentially
re-resolve and compare. No functional change.
Eventually we'll need to have the URL resolved before style resolution.

BUG=470608, 405315
==========

[Stephen Chennney, 2016-07-20 15:45:46]

On 2016/07/20 15:43:20, fs wrote:
> On 2016/07/20 at 15:37:08, schenney wrote:
> > Test? The bug reports seem to imply that a test can be constructed.
> 
> This doesn't fix anything (no functional change - added that to description.)
> When I get to the actually fixing of the bug I'll add a test =).

OK. LGTM.

[commit-bot: I haz the power, 2016-07-20 16:06:55]

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

[commit-bot: I haz the power, 2016-07-20 16:06:56]

Dry run: This issue passed the CQ dry run.

[fs, 2016-07-20 16:14:55]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-07-20 16:15:22]

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-07-20 16:20:43]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-20 16:21:04]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-20 16:22:46]

Description was changed from

==========
Simplify "is external URL" in filter operations resolving

We've already resolved the URL, so we might as well just use
equalIgnoringFragmentIdentifier on that rather than potentially
re-resolve and compare. No functional change.
Eventually we'll need to have the URL resolved before style resolution.

BUG=470608, 405315
==========

to

==========
Simplify "is external URL" in filter operations resolving

We've already resolved the URL, so we might as well just use
equalIgnoringFragmentIdentifier on that rather than potentially
re-resolve and compare. No functional change.
Eventually we'll need to have the URL resolved before style resolution.

BUG=470608, 405315

Committed: https://crrev.com/1e10ae9fded23fccb138a01b662df295c4cf57d3
Cr-Commit-Position: refs/heads/master@{#406574}
==========

[commit-bot: I haz the power, 2016-07-20 16:22:47]

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