[Useless] Hooking up storage manager in page model.

Combined with 2015793002.

[Offline Pages] Hooking up storage manager in page model.

Start using ClearPagesIfNeeded() call from storage manager. Also adding
a method allowing consumers of the model can call clear cache more
explicitly.

BUG=609357

[romax, 2016-05-19 02:27:42]

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

[romax, 2016-05-19 02:27:43]

Filip PTAL, this is how I'm thinking to call ClearPagesIfNeeded in page model.
If this is fine I'm going to add tests.

I'm having the routing API wanting to give bridge some ways to call clear cache
forcingly. Not sure if this is a good idea since it would not have more strict
rules than what we have now.

[fgorski, 2016-05-19 04:16:24]

For now we don't need to expose the API to the bridge. Don't worry about that.

https://codereview.chromium.org/1997543002/diff/1/components/offline_pages/of...
File components/offline_pages/offline_page_model.h (right):

https://codereview.chromium.org/1997543002/diff/1/components/offline_pages/of...
components/offline_pages/offline_page_model.h:219: void ExpirePages(const
OfflinePageStorageManager::ClearPageCallback&);
Who is calling this one?

[romax, 2016-05-20 00:47:50]

I removed ExpirePages but I'm adding PostTask/PostDelayTask in OnLoadDone and
InformSavePageDone.
I don't really have an idea how to test the changes reasonably, please let me
know if you have a thought.

https://codereview.chromium.org/1997543002/diff/1/components/offline_pages/of...
File components/offline_pages/offline_page_model.h (right):

https://codereview.chromium.org/1997543002/diff/1/components/offline_pages/of...
components/offline_pages/offline_page_model.h:219: void ExpirePages(const
OfflinePageStorageManager::ClearPageCallback&);
On 2016/05/19 04:16:23, fgorski wrote:
> Who is calling this one?

No one is calling this one as of now. I was assuming the bridge would call this,
I'll remove it.

[romax, 2016-05-27 01:16:02]

Description was changed from

==========
[Offline Pages] Hooking up storage manager in page model.

Start using ClearPagesIfNeeded() call from storage manager. Also adding
a method allowing consumers of the model can call clear cache more
explicitly.

BUG=609357
==========

to

==========
Combined with 2015793002.

[Offline Pages] Hooking up storage manager in page model.

Start using ClearPagesIfNeeded() call from storage manager. Also adding
a method allowing consumers of the model can call clear cache more
explicitly.

BUG=609357
==========