[Offline pages] Fixing client ID from bookmark generation

[Offline pages] Fixing client ID from bookmark generation

Adding a check for a null bookmark ID and tests.

BUG=594896
TBR=nyquist@chromium.org

Committed: https://crrev.com/706a9dd9b8dae3f6f0c8b2f697a40c6a855e0653
Cr-Commit-Position: refs/heads/master@{#381597}

[fgorski, 2016-03-15 20:49:20]

fgorski@chromium.org changed reviewers:
+ bburns@chromium.org

[fgorski, 2016-03-15 20:49:21]

PTAL.

Note to self: add a Build.gn entry.

[fgorski, 2016-03-16 17:23:54]

Brenan, PTAL

[bburns, 2016-03-16 17:33:27]

lgtm

[fgorski, 2016-03-16 17:34:45]

The CQ bit was checked by fgorski@chromium.org

[commit-bot: I haz the power, 2016-03-16 17:35:07]

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

[commit-bot: I haz the power, 2016-03-16 17:45:44]

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

[commit-bot: I haz the power, 2016-03-16 17:45:45]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[fgorski, 2016-03-16 18:24:19]

fgorski@chromium.org changed reviewers:
+ nyquist@chromium.org

[fgorski, 2016-03-16 18:24:20]

Tommy, care too look at BUILD.gn? Thanks

[fgorski, 2016-03-16 23:28:23]

Description was changed from

==========
[Offline pages] Fixing client ID from bookmark generation

Adding a check for a null bookmark ID and tests.

BUG=594896
==========

to

==========
[Offline pages] Fixing client ID from bookmark generation

Adding a check for a null bookmark ID and tests.

BUG=594896
TBR=nyquist@chromium.org
==========

[fgorski, 2016-03-16 23:28:28]

The CQ bit was checked by fgorski@chromium.org

[commit-bot: I haz the power, 2016-03-16 23:28:52]

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

[commit-bot: I haz the power, 2016-03-17 00:11:46]

Description was changed from

==========
[Offline pages] Fixing client ID from bookmark generation

Adding a check for a null bookmark ID and tests.

BUG=594896
TBR=nyquist@chromium.org
==========

to

==========
[Offline pages] Fixing client ID from bookmark generation

Adding a check for a null bookmark ID and tests.

BUG=594896
TBR=nyquist@chromium.org
==========

[commit-bot: I haz the power, 2016-03-17 00:11:47]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-03-17 00:13:05]

Description was changed from

==========
[Offline pages] Fixing client ID from bookmark generation

Adding a check for a null bookmark ID and tests.

BUG=594896
TBR=nyquist@chromium.org
==========

to

==========
[Offline pages] Fixing client ID from bookmark generation

Adding a check for a null bookmark ID and tests.

BUG=594896
TBR=nyquist@chromium.org

Committed: https://crrev.com/706a9dd9b8dae3f6f0c8b2f697a40c6a855e0653
Cr-Commit-Position: refs/heads/master@{#381597}
==========

[commit-bot: I haz the power, 2016-03-17 00:13:06]

Patchset 4 (id:??) landed as
https://crrev.com/706a9dd9b8dae3f6f0c8b2f697a40c6a855e0653
Cr-Commit-Position: refs/heads/master@{#381597}