[Offline pages] Disabling offline pages in incognito

[Offline pages] Disabling offline pages in incognito

This patch simply makes sure that offline page model is no longer avaialbe
in incognito mode.

BUG=603940, 606313
R=dewittj@chromium.org

Committed: https://crrev.com/4727ab67dde1088bdb5da8c63c51ee91e929e6e9
Cr-Commit-Position: refs/heads/master@{#392710}

[fgorski, 2016-05-10 18:12:57]

Description was changed from

==========
[Offline pages] Disabling offline pages in incognito

This patch simply makes sure that offline page model is no longer avaialbe
in incognito mode.

BUG=603940
==========

to

==========
[Offline pages] Disabling offline pages in incognito

This patch simply makes sure that offline page model is no longer avaialbe
in incognito mode.

BUG=603940,606313
R=dewittj@chromium.org
==========

[fgorski, 2016-05-10 18:12:57]

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

[fgorski, 2016-05-10 18:13:30]

PTAL

[dewittj, 2016-05-10 18:19:13]

lgtm

Can we add an integration test that checks this logic from the android side?  If
that's too much for right now, can you make a bug or TODO to track adding it?

[fgorski, 2016-05-10 20:38:28]

The CQ bit was checked by fgorski@chromium.org

[fgorski, 2016-05-10 20:38:30]

The patchset sent to the CQ was uploaded after l-g-t-m from dewittj@chromium.org
Link to the patchset: https://codereview.chromium.org/1966723002/#ps20001
(title: "Adding a TODO")

[commit-bot: I haz the power, 2016-05-10 20:39:48]

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

[commit-bot: I haz the power, 2016-05-10 20:54:43]

Description was changed from

==========
[Offline pages] Disabling offline pages in incognito

This patch simply makes sure that offline page model is no longer avaialbe
in incognito mode.

BUG=603940,606313
R=dewittj@chromium.org
==========

to

==========
[Offline pages] Disabling offline pages in incognito

This patch simply makes sure that offline page model is no longer avaialbe
in incognito mode.

BUG=603940,606313
R=dewittj@chromium.org
==========

[commit-bot: I haz the power, 2016-05-10 20:54:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-10 20:57:14]

Description was changed from

==========
[Offline pages] Disabling offline pages in incognito

This patch simply makes sure that offline page model is no longer avaialbe
in incognito mode.

BUG=603940,606313
R=dewittj@chromium.org
==========

to

==========
[Offline pages] Disabling offline pages in incognito

This patch simply makes sure that offline page model is no longer avaialbe
in incognito mode.

BUG=603940,606313
R=dewittj@chromium.org

Committed: https://crrev.com/4727ab67dde1088bdb5da8c63c51ee91e929e6e9
Cr-Commit-Position: refs/heads/master@{#392710}
==========

[commit-bot: I haz the power, 2016-05-10 20:57:15]

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