Convert dom_distiller::WebContentsMainFrameObserver to use the new navigation callbacks.

Convert dom_distiller::WebContentsMainFrameObserver to use the new navigation callbacks.

BUG=682002
Review-Url: https://codereview.chromium.org/2663333004
Cr-Commit-Position: refs/heads/master@{#447579}
Committed: https://chromium.googlesource.com/chromium/src/+/5a063aecb2a8724e483a3ff1c682622e97e0426e

[jam, 2017-02-01 00:10:27]

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

[jam, 2017-02-01 00:11:11]

Patchset #1 (id:1) has been deleted

[jam, 2017-02-01 00:11:21]

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

[jam, 2017-02-01 00:11:29]

Description was changed from

==========
Convert dom_distiller::WebContentsMainFrameObserver to use the new navigation
callbacks.

BUG=682002
==========

to

==========
Convert dom_distiller::WebContentsMainFrameObserver to use the new navigation
callbacks.

BUG=682002
==========

[jam, 2017-02-01 00:11:29]

jam@chromium.org changed reviewers:
+ nyquist@chromium.org

[commit-bot: I haz the power, 2017-02-01 00:11:49]

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 01:54:59]

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

[commit-bot: I haz the power, 2017-02-01 01:55:00]

Dry run: This issue passed the CQ dry run.

[nyquist, 2017-02-01 07:10:04]

https://codereview.chromium.org/2663333004/diff/20001/content/test/web_conten...
File content/test/web_contents_observer_sanity_checker.cc (right):

https://codereview.chromium.org/2663333004/diff/20001/content/test/web_conten...
content/test/web_contents_observer_sanity_checker.cc:131:
//AssertRenderFrameExists(new_host->GetParent());
It looks from the comments in the test code that the change to this file should
be scrapped now?

[jam, 2017-02-01 15:29:48]

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

[commit-bot: I haz the power, 2017-02-01 15:29:58]

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

[jam, 2017-02-01 15:31:04]

https://codereview.chromium.org/2663333004/diff/20001/content/test/web_conten...
File content/test/web_contents_observer_sanity_checker.cc (right):

https://codereview.chromium.org/2663333004/diff/20001/content/test/web_conten...
content/test/web_contents_observer_sanity_checker.cc:131:
//AssertRenderFrameExists(new_host->GetParent());
On 2017/02/01 07:10:04, nyquist wrote:
> It looks from the comments in the test code that the change to this file
should
> be scrapped now?

Ooops, thanks fixed.

[commit-bot: I haz the power, 2017-02-01 16:29:56]

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

[commit-bot: I haz the power, 2017-02-01 16:29:57]

Dry run: This issue passed the CQ dry run.

[nyquist, 2017-02-01 17:45:35]

lgtm

[jam, 2017-02-01 19:30:04]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2017-02-01 19:30:26]

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 19:35:31]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485977404004670,
"parent_rev": "a86e8e115054ad167458d0379980db5473cb20de", "commit_rev":
"5a063aecb2a8724e483a3ff1c682622e97e0426e"}

[commit-bot: I haz the power, 2017-02-01 19:36:03]

Description was changed from

==========
Convert dom_distiller::WebContentsMainFrameObserver to use the new navigation
callbacks.

BUG=682002
==========

to

==========
Convert dom_distiller::WebContentsMainFrameObserver to use the new navigation
callbacks.

BUG=682002

Review-Url: https://codereview.chromium.org/2663333004
Cr-Commit-Position: refs/heads/master@{#447579}
Committed:
https://chromium.googlesource.com/chromium/src/+/5a063aecb2a8724e483a3ff1c682...
==========

[commit-bot: I haz the power, 2017-02-01 19:36:04]

Committed patchset #2 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/5a063aecb2a8724e483a3ff1c682...