Chromium Code Reviews| Index: third_party/WebKit/Source/core/timing/PerformanceNavigationTiming.cpp |
| diff --git a/third_party/WebKit/Source/core/timing/PerformanceNavigationTiming.cpp b/third_party/WebKit/Source/core/timing/PerformanceNavigationTiming.cpp |
| index 080836e01b62a151347b338d2609ebba07393c95..492bd87b424a99c19b88c1b77a917aa4292f175d 100644 |
| --- a/third_party/WebKit/Source/core/timing/PerformanceNavigationTiming.cpp |
| +++ b/third_party/WebKit/Source/core/timing/PerformanceNavigationTiming.cpp |
| @@ -121,16 +121,17 @@ double PerformanceNavigationTiming::loadEventEnd() const { |
| } |
| AtomicString PerformanceNavigationTiming::type() const { |
| - // TODO(sunjian) Right now NavigationType doesn't seem to have a Prerender |
| - // type yet, need to look into this crbug/663217 |
| switch (m_type) { |
| - case NavigationTypeReload: |
| + case NavigationType::Reload: |
| return "reload"; |
| - case NavigationTypeBackForward: |
| + case NavigationType::BackForward: |
| return "back_forward"; |
| - default: |
| + case NavigationType::Prerender: |
| + return "prerender"; |
| + case NavigationType::Navigate: |
| return "navigate"; |
| } |
| + NOTREACHED(); |
|
kinuko
2016/11/30 05:29:29
nit- still need to return something here to get it
sunjian
2016/11/30 19:15:47
Done.
|
| } |
| unsigned short PerformanceNavigationTiming::redirectCount() const { |