Revert of Simplify MetricsWebContentsObserver::OnTimingUpdated

Revert of Simplify MetricsWebContentsObserver::OnTimingUpdated (patchset #1 id:1 of https://codereview.chromium.org/2795283003/ )

Reason for revert:
Address bug 708468

Original issue's description:
> Simplify MetricsWebContentsObserver::OnTimingUpdated
> 
> The check for whether a frame has navigated away is a bit more complex
> than needed and uses RenderViewHost, which has been deprecated for a
> while. This CL moves the child frame check first, which allows the code
> to perform a simpler check using the WebContents::GetMainFrame().
> 
> BUG=
> 
> Review-Url: https://codereview.chromium.org/2795283003
> Cr-Commit-Position: refs/heads/master@{#461875}
> Committed: https://chromium.googlesource.com/chromium/src/+/0735c4618fe9c05775cf55be073f7fcb4c2e75ac

TBR=csharrison@chromium.org,nasko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Review-Url: https://codereview.chromium.org/2802693002
Cr-Commit-Position: refs/heads/master@{#462048}
Committed: https://chromium.googlesource.com/chromium/src/+/253fe7d912b4e16fd56c6ce0a68906d0f1a376e2

[Bryan McQuade, 2017-04-05 13:02:26]

The CQ bit was checked by bmcquade@chromium.org

[Bryan McQuade, 2017-04-05 13:02:26]

Created Revert of Simplify MetricsWebContentsObserver::OnTimingUpdated

[commit-bot: I haz the power, 2017-04-05 13:02:33]

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-04-05 13:03:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491397346066530, "parent_rev":
"b146b4e388d8a0132a0a51cda5b8ebd8237b732e", "commit_rev":
"253fe7d912b4e16fd56c6ce0a68906d0f1a376e2"}

[commit-bot: I haz the power, 2017-04-05 13:03:54]

Description was changed from

==========
Revert of Simplify MetricsWebContentsObserver::OnTimingUpdated (patchset #1 id:1
of https://codereview.chromium.org/2795283003/ )

Reason for revert:
Address bug 708468

Original issue's description:
> Simplify MetricsWebContentsObserver::OnTimingUpdated
> 
> The check for whether a frame has navigated away is a bit more complex
> than needed and uses RenderViewHost, which has been deprecated for a
> while. This CL moves the child frame check first, which allows the code
> to perform a simpler check using the WebContents::GetMainFrame().
> 
> BUG=
> 
> Review-Url: https://codereview.chromium.org/2795283003
> Cr-Commit-Position: refs/heads/master@{#461875}
> Committed:
https://chromium.googlesource.com/chromium/src/+/0735c4618fe9c05775cf55be073f...

TBR=csharrison@chromium.org,nasko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
==========

to

==========
Revert of Simplify MetricsWebContentsObserver::OnTimingUpdated (patchset #1 id:1
of https://codereview.chromium.org/2795283003/ )

Reason for revert:
Address bug 708468

Original issue's description:
> Simplify MetricsWebContentsObserver::OnTimingUpdated
> 
> The check for whether a frame has navigated away is a bit more complex
> than needed and uses RenderViewHost, which has been deprecated for a
> while. This CL moves the child frame check first, which allows the code
> to perform a simpler check using the WebContents::GetMainFrame().
> 
> BUG=
> 
> Review-Url: https://codereview.chromium.org/2795283003
> Cr-Commit-Position: refs/heads/master@{#461875}
> Committed:
https://chromium.googlesource.com/chromium/src/+/0735c4618fe9c05775cf55be073f...

TBR=csharrison@chromium.org,nasko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Review-Url: https://codereview.chromium.org/2802693002
Cr-Commit-Position: refs/heads/master@{#462048}
Committed:
https://chromium.googlesource.com/chromium/src/+/253fe7d912b4e16fd56c6ce0a689...
==========

[commit-bot: I haz the power, 2017-04-05 13:03:55]

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

[Charlie Harrison, 2017-04-05 13:17:54]

LGTM