Only setup distillability mojo if needed

Only setup distillability mojo if needed

Prevent extra calls to set up mojo for Reader Mode's distillability
logic. Currently the distillability service is set up for any
RenderFrameHost that changes (this is not necessarily the main frame)
and even if the old service is still alive. This change starts
watching for when the service is killed and avoids service setup on
non-main frame hosts.

BUG=623979
Committed: https://crrev.com/70bb9fd22cc806498fc63a5749fdaaaea7354564
Cr-Commit-Position: refs/heads/master@{#404678}

[mdjones, 2016-07-08 17:31:02]

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

[mdjones, 2016-07-08 17:31:03]

ptal

[wychen, 2016-07-08 23:10:20]

lgtm

https://codereview.chromium.org/2131343002/diff/1/components/dom_distiller/co...
File components/dom_distiller/content/browser/distillability_driver.cc (right):

https://codereview.chromium.org/2131343002/diff/1/components/dom_distiller/co...
components/dom_distiller/content/browser/distillability_driver.cc:80: // This
method is invoked if a if any of the active RenderFrameHosts is
This line need some fixes.

[mdjones, 2016-07-11 15:50:44]

https://codereview.chromium.org/2131343002/diff/1/components/dom_distiller/co...
File components/dom_distiller/content/browser/distillability_driver.cc (right):

https://codereview.chromium.org/2131343002/diff/1/components/dom_distiller/co...
components/dom_distiller/content/browser/distillability_driver.cc:80: // This
method is invoked if a if any of the active RenderFrameHosts is
On 2016/07/08 23:10:20, wychen wrote:
> This line need some fixes.

Done.

[mdjones, 2016-07-11 15:50:53]

The CQ bit was checked by mdjones@chromium.org

[mdjones, 2016-07-11 15:50:53]

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/2131343002/#ps20001
(title: "fix comment")

[commit-bot: I haz the power, 2016-07-11 15:51:15]

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-07-11 16:58:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-11 16:58:56]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-11 17:00:37]

Description was changed from

==========
Only setup distillability mojo if needed

Prevent extra calls to set up mojo for Reader Mode's distillability
logic. Currently the distillability service is set up for any
RenderFrameHost that changes (this is not necessarily the main frame)
and even if the old service is still alive. This change starts
watching for when the service is killed and avoids service setup on
non-main frame hosts.

BUG=623979
==========

to

==========
Only setup distillability mojo if needed

Prevent extra calls to set up mojo for Reader Mode's distillability
logic. Currently the distillability service is set up for any
RenderFrameHost that changes (this is not necessarily the main frame)
and even if the old service is still alive. This change starts
watching for when the service is killed and avoids service setup on
non-main frame hosts.

BUG=623979

Committed: https://crrev.com/70bb9fd22cc806498fc63a5749fdaaaea7354564
Cr-Commit-Position: refs/heads/master@{#404678}
==========

[commit-bot: I haz the power, 2016-07-11 17:00:38]

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