Add Navigation.MainFrameSchemeDifferentPage which excludes same-page navigations.

Add Navigation.MainFrameSchemeDifferentPage which excludes same-page navigations.

Same page navigations include things like location.assign with a fragment identifier change only, or path changes using history.pushState

BUG=None

Committed: https://crrev.com/794bee2a7d2f731148f0cd5765c550d140ea2000
Cr-Commit-Position: refs/heads/master@{#304498}

[cbentzel, 2014-11-12 16:42:47]

cbentzel@chromium.org changed reviewers:
+ davidben@chromium.org

[cbentzel, 2014-11-12 16:42:48]

I think I'm going to need to change some downstream callers for
RecordMainFrameNavigation

davidben: Please do review

[davidben, 2014-11-12 18:01:17]

lgtm

[cbentzel, 2014-11-13 16:55:43]

cbentzel@chromium.org changed reviewers:
+ avi@chromium.org, erikwright@chromium.org, isherman@chromium.org

[cbentzel, 2014-11-13 16:55:43]

cbentzel@chromium.org changed required reviewers:
+ avi@chromium.org, erikwright@chromium.org, isherman@chromium.org

[cbentzel, 2014-11-13 16:55:44]

avi: chrome/browser/tab_contents/navigation_metrics_recorder.cc
erikwright: components/navigation_metrics/* (perhaps this directory also needs
OWNERS?)
isherman: histograms.xml

[Avi (use Gerrit), 2014-11-13 18:59:04]

lgtm

navigation_metrics_recorder.cc is cool.

[Ilya Sherman, 2014-11-13 22:40:42]

histograms.xml lgtm

[erikwright (departed), 2014-11-17 15:56:18]

LGTM but yes an OWNERS file would be a good idea here.

[cbentzel, 2014-11-17 21:17:00]

The CQ bit was checked by cbentzel@chromium.org

[commit-bot: I haz the power, 2014-11-17 21:18:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/719733003/20001

[commit-bot: I haz the power, 2014-11-17 22:22:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-17 22:24:36]

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