Small: Implement CheckResourceUrl

Small: Implement CheckResourceUrl

Very similar to: https://codereview.chromium.org/2450633003/

BUG=543161
Committed: https://crrev.com/4817cb8914c9130781c5b2bc44cb96f4284b9022
Cr-Commit-Position: refs/heads/master@{#428180}

[vakh (use Gerrit instead), 2016-10-27 01:10:27]

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

[vakh (use Gerrit instead), 2016-10-27 01:10:48]

vakh@chromium.org changed reviewers:
+ nparker@chromium.org, shess@chromium.org

[commit-bot: I haz the power, 2016-10-27 01:10:49]

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

[vakh (use Gerrit instead), 2016-10-27 01:10:49]

TODO: Add the entry in histograms.xml

[commit-bot: I haz the power, 2016-10-27 02:01:12]

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

[commit-bot: I haz the power, 2016-10-27 02:01:12]

Dry run: This issue passed the CQ dry run.

[vakh (use Gerrit instead), 2016-10-27 17:26:18]

git rebase

[vakh (use Gerrit instead), 2016-10-27 17:26:19]

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

[commit-bot: I haz the power, 2016-10-27 17:26:43]

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-10-27 18:23:23]

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

[commit-bot: I haz the power, 2016-10-27 18:23:24]

Dry run: This issue passed the CQ dry run.

[Nathan Parker, 2016-10-27 18:53:40]

We should have some test for these interface methods.  Is there existing Pver3
tests of these that we can convert?

[Scott Hess - ex-Googler, 2016-10-27 19:20:37]

lgtm

[vakh (use Gerrit instead), 2016-10-27 22:26:20]

On 2016/10/27 at 18:53:40, nparker wrote:
> We should have some test for these interface methods.  Is there existing Pver3
tests of these that we can convert?

I think it would be more useful to test these methods once we can start syncing
these stores.
At that time, we can stuff the database with mock values and test the
expectations better.

[Nathan Parker, 2016-10-27 22:27:02]

lgtm

(talked in person).  Yup, adding tests a bit later (~week) is more efficient.

[vakh (use Gerrit instead), 2016-10-27 22:46:49]

The CQ bit was checked by vakh@chromium.org

[commit-bot: I haz the power, 2016-10-27 22:47: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-10-27 22:55:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-27 22:58:19]

Description was changed from

==========
Small: Implement CheckResourceUrl

Very similar to: https://codereview.chromium.org/2450633003/

BUG=543161
==========

to

==========
Small: Implement CheckResourceUrl

Very similar to: https://codereview.chromium.org/2450633003/

BUG=543161

Committed: https://crrev.com/4817cb8914c9130781c5b2bc44cb96f4284b9022
Cr-Commit-Position: refs/heads/master@{#428180}
==========

[commit-bot: I haz the power, 2016-10-27 22:58:20]

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