Reader Mode distillability uses Mojo

Reader Mode distillability uses Mojo

This change converts the distillability code from using the
deprecated Chromium IPC to using Mojo.

BUG=603349
Committed: https://crrev.com/8d71187700cce73cfd8b2a9b2fc859a051564dd8
Cr-Commit-Position: refs/heads/master@{#387644}

[mdjones, 2016-04-14 01:15:15]

mdjones@chromium.org changed reviewers:
+ wychen@chromium.org

[mdjones, 2016-04-14 01:15:16]

ptal

[wychen, 2016-04-14 20:33:07]

lgtm

[mdjones, 2016-04-15 00:25:11]

The CQ bit was checked by mdjones@chromium.org

[commit-bot: I haz the power, 2016-04-15 00:25:45]

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

[commit-bot: I haz the power, 2016-04-15 01:10:23]

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

[commit-bot: I haz the power, 2016-04-15 01:10:24]

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

[wychen, 2016-04-15 06:11:23]

https://codereview.chromium.org/1891643002/diff/60001/components/dom_distille...
File components/dom_distiller/content/browser/distillability_driver.cc (right):

https://codereview.chromium.org/1891643002/diff/60001/components/dom_distille...
components/dom_distiller/content/browser/distillability_driver.cc:76:
web_contents()->GetMainFrame()->GetServiceRegistry()
I guess " if (web_contents()) return;" is needed.

[mdjones, 2016-04-15 15:50:58]

https://codereview.chromium.org/1891643002/diff/60001/components/dom_distille...
File components/dom_distiller/content/browser/distillability_driver.cc (right):

https://codereview.chromium.org/1891643002/diff/60001/components/dom_distille...
components/dom_distiller/content/browser/distillability_driver.cc:76:
web_contents()->GetMainFrame()->GetServiceRegistry()
On 2016/04/15 06:11:22, wychen wrote:
> I guess " if (web_contents()) return;" is needed.

Done.

[mdjones, 2016-04-15 15:51:05]

The CQ bit was checked by mdjones@chromium.org

[mdjones, 2016-04-15 15:51:06]

The patchset sent to the CQ was uploaded after l-g-t-m from wychen@chromium.org
Link to the patchset: https://codereview.chromium.org/1891643002/#ps80001
(title: "add web contents check")

[commit-bot: I haz the power, 2016-04-15 15:51:25]

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

[commit-bot: I haz the power, 2016-04-15 18:11:59]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-04-15 18:13:11]

Description was changed from

==========
Reader Mode distillability uses Mojo

This change converts the distillability code from using the
deprecated Chromium IPC to using Mojo.

BUG=603349
==========

to

==========
Reader Mode distillability uses Mojo

This change converts the distillability code from using the
deprecated Chromium IPC to using Mojo.

BUG=603349

Committed: https://crrev.com/8d71187700cce73cfd8b2a9b2fc859a051564dd8
Cr-Commit-Position: refs/heads/master@{#387644}
==========

[commit-bot: I haz the power, 2016-04-15 18:13:11]

Patchset 5 (id:??) landed as
https://crrev.com/8d71187700cce73cfd8b2a9b2fc859a051564dd8
Cr-Commit-Position: refs/heads/master@{#387644}