Allow DomDistillerService to have no store.

Allow DomDistillerService to have no store.

We do not want storage or sync of distilled versions of page as we store
them in files with ReadingListItems.
Do not instantiate the store in ReadingList for iOS.

BUG=686402
Review-Url: https://codereview.chromium.org/2667533003
Cr-Commit-Position: refs/heads/master@{#447551}
Committed: https://chromium.googlesource.com/chromium/src/+/b227a4e9a84a3faae02a125085a49906e1b4c54a

[Olivier, 2017-01-30 08:52:01]

olivierrobin@chromium.org changed reviewers:
+ noyau@chromium.org, wychen@chromium.org

[Olivier, 2017-02-01 06:32:30]

Ping

[Olivier, 2017-02-01 06:32:56]

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

[commit-bot: I haz the power, 2017-02-01 06:33:14]

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

[wychen, 2017-02-01 07:32:02]

The concept looks good. I haven't built and tested this CL though.

[Olivier, 2017-02-01 08:02:54]

On 2017/02/01 07:32:02, wychen (APAC time zone) wrote:
> The concept looks good. I haven't built and tested this CL though.

I tested on iOS, it seems to work.

[wychen, 2017-02-01 08:04:30]

lgtm

[commit-bot: I haz the power, 2017-02-01 08:10:11]

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

[commit-bot: I haz the power, 2017-02-01 08:10:12]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Olivier, 2017-02-01 08:10:46]

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

[commit-bot: I haz the power, 2017-02-01 08:11:03]

Dry run: 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, 2017-02-01 08:51:33]

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

[commit-bot: I haz the power, 2017-02-01 08:51:34]

Dry run: This issue passed the CQ dry run.

[noyau (Ping after 24h), 2017-02-01 17:38:18]

lgtm

[Olivier, 2017-02-01 17:38:43]

The CQ bit was checked by olivierrobin@chromium.org

[commit-bot: I haz the power, 2017-02-01 17:39:03]

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, 2017-02-01 17:43:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485970723644000, "parent_rev":
"701d8eaabbed864ec14ba539c7ec7ff9db5d94ee", "commit_rev":
"b227a4e9a84a3faae02a125085a49906e1b4c54a"}

[commit-bot: I haz the power, 2017-02-01 17:44:50]

Description was changed from

==========
Allow DomDistillerService to have no store.

We do not want storage or sync of distilled versions of page as we store
them in files with ReadingListItems.
Do not instantiate the store in ReadingList for iOS.

BUG=686402
==========

to

==========
Allow DomDistillerService to have no store.

We do not want storage or sync of distilled versions of page as we store
them in files with ReadingListItems.
Do not instantiate the store in ReadingList for iOS.

BUG=686402

Review-Url: https://codereview.chromium.org/2667533003
Cr-Commit-Position: refs/heads/master@{#447551}
Committed:
https://chromium.googlesource.com/chromium/src/+/b227a4e9a84a3faae02a125085a4...
==========

[commit-bot: I haz the power, 2017-02-01 17:44:51]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b227a4e9a84a3faae02a125085a4...