Move getNavigationType logic from PerformanceBase to PerformanceNavigationTiming.

Move getNavigationType logic from PerformanceBase to PerformanceNavigationTiming.

Currently, the type() method in PerformanceNavigationTiming will call
PerformanceBase::getNavigationType(). Since PNT is the only caller of
this method. It makes more sense to merge this logic into PNT.

BUG=703409
Review-Url: https://codereview.chromium.org/2761113002
Cr-Commit-Position: refs/heads/master@{#460873}
Committed: https://chromium.googlesource.com/chromium/src/+/5d663f46bd1cef7b00e9a52b5148af16014ec652

[sunjian, 2017-03-20 23:46:21]

Description was changed from

==========
Move getNavigationType logic from PerformanceBase to
PerformanceNavigationTiming.

BUG=703409
==========

to

==========
Move getNavigationType logic from PerformanceBase to
PerformanceNavigationTiming.

Currently, the type() method in PerformanceNavigationTiming will call
PerformanceBase::getNavigationType(). Since PNT is the only caller of
this method. It makes more sense to merge this logic into PNT.

BUG=703409
==========

[sunjian, 2017-03-20 23:49:04]

The CQ bit was checked by sunjian@chromium.org

[commit-bot: I haz the power, 2017-03-20 23:49:57]

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-03-20 23:49:58]

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

[commit-bot: I haz the power, 2017-03-20 23:49:59]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[sunjian, 2017-03-28 21:55:46]

sunjian@chromium.org changed reviewers:
+ panicker@chromium.org

[panicker, 2017-03-29 17:37:30]

panicker@chromium.org changed reviewers:
+ haraken@chromium.org

[panicker, 2017-03-29 17:37:31]

LGTM.
Kentaro - can you review for owners (this was broken out of 
https://codereview.chromium.org/2647643004/ (where you'd suggested breaking into
smaller CLs.)

[haraken, 2017-03-29 23:55:47]

LGTM

[sunjian, 2017-03-30 00:02:31]

The CQ bit was checked by sunjian@chromium.org

[commit-bot: I haz the power, 2017-03-30 00:04:19]

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-03-30 02:07:03]

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

[commit-bot: I haz the power, 2017-03-30 02:07:04]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_TIMED_OUT, build hasn't
started yet, builder probably lacks capacity)

[sunjian, 2017-03-30 18:37:20]

The CQ bit was checked by sunjian@chromium.org

[sunjian, 2017-03-30 18:37:21]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, panicker@chromium.org
Link to the patchset: https://codereview.chromium.org/2761113002/#ps60001
(title: "sync to HEAD")

[commit-bot: I haz the power, 2017-03-30 18:38:21]

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-03-30 21:05:32]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490899040925250,
"parent_rev": "4582cd2a8064b47cacff25949beb7c4d2786d130", "commit_rev":
"5d663f46bd1cef7b00e9a52b5148af16014ec652"}

[commit-bot: I haz the power, 2017-03-30 21:06:20]

Description was changed from

==========
Move getNavigationType logic from PerformanceBase to
PerformanceNavigationTiming.

Currently, the type() method in PerformanceNavigationTiming will call
PerformanceBase::getNavigationType(). Since PNT is the only caller of
this method. It makes more sense to merge this logic into PNT.

BUG=703409
==========

to

==========
Move getNavigationType logic from PerformanceBase to
PerformanceNavigationTiming.

Currently, the type() method in PerformanceNavigationTiming will call
PerformanceBase::getNavigationType(). Since PNT is the only caller of
this method. It makes more sense to merge this logic into PNT.

BUG=703409

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

[commit-bot: I haz the power, 2017-03-30 21:06:21]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/5d663f46bd1cef7b00e9a52b5148...