[Offline Pages] Adds storage for kInvalidOfflineId.

[Offline Pages] Adds storage for kInvalidOfflineId.

Since kInvalidOfflineId is a static data member, it needs explicitly
defined storage as soon as it becomes 'odr-used'.  Replace a temporary
variable in RecentTabHelper with storage in offline_page_model.cc both
for clarity and to allow further references to kInvalidOfflineId to be
taken.

BUG=

Committed: https://crrev.com/07cab6870842811b1b4bad01a69ead1a95acac33
Cr-Commit-Position: refs/heads/master@{#436116}

[dewittj, 2016-12-02 23:49:11]

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

[commit-bot: I haz the power, 2016-12-02 23:50:21]

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

[dewittj, 2016-12-02 23:50:35]

Description was changed from

==========
[Offline Pages] Adds storage for kInvalidOfflineId.

Since kInvalidOfflineId is a static data member, it needs explicitly
defined storage as soon as it becomes 'odr-used'.  Replace a temporary
variable in RecentTabHelper with storage in offline_page_model.cc both
for clarity and to allow further references to kInvalidOfflineId to be
taken.

BUG=
==========

to

==========
[Offline Pages] Adds storage for kInvalidOfflineId.

Since kInvalidOfflineId is a static data member, it needs explicitly
defined storage as soon as it becomes 'odr-used'.  Replace a temporary
variable in RecentTabHelper with storage in offline_page_model.cc both
for clarity and to allow further references to kInvalidOfflineId to be
taken.

BUG=
==========

[dewittj, 2016-12-02 23:50:35]

dewittj@chromium.org changed reviewers:
+ carlosk@chromium.org

[commit-bot: I haz the power, 2016-12-03 00:38:57]

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

[commit-bot: I haz the power, 2016-12-03 00:38:58]

Dry run: This issue passed the CQ dry run.

[carlosk, 2016-12-03 01:05:18]

LGTM and thanks for fixing this.

[dewittj, 2016-12-03 01:05:36]

The CQ bit was checked by dewittj@chromium.org

[commit-bot: I haz the power, 2016-12-03 01:06:11]

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-12-03 01:15:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480727136465900, "parent_rev":
"c30f7dd3a18180652fb08ba40703bd7048875e3d", "commit_rev":
"9eca2ad6c58b3a57fcf61e1cafddabbbd6d4713a"}

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

Description was changed from

==========
[Offline Pages] Adds storage for kInvalidOfflineId.

Since kInvalidOfflineId is a static data member, it needs explicitly
defined storage as soon as it becomes 'odr-used'.  Replace a temporary
variable in RecentTabHelper with storage in offline_page_model.cc both
for clarity and to allow further references to kInvalidOfflineId to be
taken.

BUG=
==========

to

==========
[Offline Pages] Adds storage for kInvalidOfflineId.

Since kInvalidOfflineId is a static data member, it needs explicitly
defined storage as soon as it becomes 'odr-used'.  Replace a temporary
variable in RecentTabHelper with storage in offline_page_model.cc both
for clarity and to allow further references to kInvalidOfflineId to be
taken.

BUG=
==========

[commit-bot: I haz the power, 2016-12-03 01:16:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-03 01:20:54]

Description was changed from

==========
[Offline Pages] Adds storage for kInvalidOfflineId.

Since kInvalidOfflineId is a static data member, it needs explicitly
defined storage as soon as it becomes 'odr-used'.  Replace a temporary
variable in RecentTabHelper with storage in offline_page_model.cc both
for clarity and to allow further references to kInvalidOfflineId to be
taken.

BUG=
==========

to

==========
[Offline Pages] Adds storage for kInvalidOfflineId.

Since kInvalidOfflineId is a static data member, it needs explicitly
defined storage as soon as it becomes 'odr-used'.  Replace a temporary
variable in RecentTabHelper with storage in offline_page_model.cc both
for clarity and to allow further references to kInvalidOfflineId to be
taken.

BUG=

Committed: https://crrev.com/07cab6870842811b1b4bad01a69ead1a95acac33
Cr-Commit-Position: refs/heads/master@{#436116}
==========

[commit-bot: I haz the power, 2016-12-03 01:20:55]

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