Bail out of resetting a URL's engagement if the URL is not valid.

Bail out of resetting a URL's engagement if the URL is not valid.

This speculatively addresses a DCHECK failure when the history service
expires a URL on startup and notifies the site engagement service of it.
Occasionally, the URL sent to site engagement is invalid, triggering the
DCHECK.

BUG=612881
Committed: https://crrev.com/b30352f5c5bb51c5be8d3e147080729794f1d6a3
Cr-Commit-Position: refs/heads/master@{#394991}

[dominickn, 2016-05-20 02:21:06]

dominickn@chromium.org changed reviewers:
+ calamity@chromium.org

[dominickn, 2016-05-20 02:21:07]

PTAL, thanks

[calamity, 2016-05-20 02:35:21]

lgtm

[dominickn, 2016-05-20 02:35:56]

The CQ bit was checked by dominickn@chromium.org

[commit-bot: I haz the power, 2016-05-20 02:36:43]

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

[dominickn, 2016-05-20 02:41:33]

Description was changed from

==========
Bail out of resetting a URL's engagement if the URL is not valid.

This speculatively addresses a DCHECK failure when the history service
expires a URL on startup and notifies the site engagement service of it.

BUG=612881
==========

to

==========
Bail out of resetting a URL's engagement if the URL is not valid.

This speculatively addresses a DCHECK failure when the history service
expires a URL on startup and notifies the site engagement service of it.
Occasionally, the URL sent to site engagement is invalid, triggering the
DCHECK.

BUG=612881
==========

[commit-bot: I haz the power, 2016-05-20 03:55:33]

Description was changed from

==========
Bail out of resetting a URL's engagement if the URL is not valid.

This speculatively addresses a DCHECK failure when the history service
expires a URL on startup and notifies the site engagement service of it.
Occasionally, the URL sent to site engagement is invalid, triggering the
DCHECK.

BUG=612881
==========

to

==========
Bail out of resetting a URL's engagement if the URL is not valid.

This speculatively addresses a DCHECK failure when the history service
expires a URL on startup and notifies the site engagement service of it.
Occasionally, the URL sent to site engagement is invalid, triggering the
DCHECK.

BUG=612881
==========

[commit-bot: I haz the power, 2016-05-20 03:55:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-20 03:57:17]

Description was changed from

==========
Bail out of resetting a URL's engagement if the URL is not valid.

This speculatively addresses a DCHECK failure when the history service
expires a URL on startup and notifies the site engagement service of it.
Occasionally, the URL sent to site engagement is invalid, triggering the
DCHECK.

BUG=612881
==========

to

==========
Bail out of resetting a URL's engagement if the URL is not valid.

This speculatively addresses a DCHECK failure when the history service
expires a URL on startup and notifies the site engagement service of it.
Occasionally, the URL sent to site engagement is invalid, triggering the
DCHECK.

BUG=612881

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

[commit-bot: I haz the power, 2016-05-20 03:57:18]

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