Add trace event in NavigationControllerImpl::GoToIndex.

content/browser/frame_host/navigation_controller_impl.cc

Index: content/browser/frame_host/navigation_controller_impl.cc
diff --git a/content/browser/frame_host/navigation_controller_impl.cc b/content/browser/frame_host/navigation_controller_impl.cc
index 2ffeafa8135ebb3637f3e81aa6b0cb018bf0caaf..d18122274033a800093fc224d8cc3590441ace98 100644
--- a/content/browser/frame_host/navigation_controller_impl.cc
+++ b/content/browser/frame_host/navigation_controller_impl.cc
@@ -586,20 +586,18 @@ bool NavigationControllerImpl::CanGoToOffset(int offset) const {
 }
 
 void NavigationControllerImpl::GoBack() {
-  TRACE_EVENT0("browser,navigation",
-               "NavigationControllerImpl::GoBack");
   // Call GoToIndex rather than GoToOffset to get the NOTREACHED() check.
   GoToIndex(GetIndexForOffset(-1));
 }
 
 void NavigationControllerImpl::GoForward() {
-  TRACE_EVENT0("browser,navigation",
-               "NavigationControllerImpl::GoForward");
   // Call GoToIndex rather than GoToOffset to get the NOTREACHED() check.
   GoToIndex(GetIndexForOffset(1));
 }
 
 void NavigationControllerImpl::GoToIndex(int index) {
+  TRACE_EVENT0("browser,navigation,benchmark",
+               "NavigationControllerImpl::GoToIndex");

[nasko, 2017/03/17 22:14:58]

It would've been nice if this was done in the previous CL as commented on.

   if (index < 0 || index >= static_cast<int>(entries_.size())) {
     NOTREACHED();
     return;