EQT: Record the max queueing time from navigation start to navigation away

content/renderer/render_frame_impl.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/render_frame_impl.h"
 
 #include <map>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 3803 matching lines @@
     if (render_thread_impl) {  // Can be NULL in tests.
       render_thread_impl->histogram_customizer()->
           RenderViewNavigatedToHost(GURL(GetLoadingUrl()).host(),
                                     RenderView::GetRenderViewCount());
       // The scheduler isn't interested in history inert commits unless they
       // are reloads.
       if (commit_type != blink::kWebHistoryInertCommit ||
           PageTransitionCoreTypeIs(navigation_state->GetTransitionType(),
                                    ui::PAGE_TRANSITION_RELOAD)) {
         render_thread_impl->GetRendererScheduler()->OnNavigationStarted();
+        render_thread_impl->GetRendererScheduler()->OnCommitProvisionalLoad();

[Liquan (Max) Gu, 2017/05/25 16:34:40]

Do you think we should combine the part in OnNavigationStarted() into
OnCommitProvisionalLoad(), so that here only needs OnCommitProvisionalLoad(). 

[tdresser, 2017/05/25 19:43:38]

I propose we leave it for now, and as a follow up (potentially done by scheduler
folks), we move remove OnNavigationStarted completely, and do all associated
work in OnCommitProvisionalLoad.

[Liquan (Max) Gu, 2017/05/29 23:44:10]

Acknowledged.

       }
     }
   }
 
   // Remember that we've already processed this request, so we don't update
   // the session history again.  We do this regardless of whether this is
   // a session history navigation, because if we attempted a session history
   // navigation without valid HistoryItem state, WebCore will think it is a
   // new navigation.
   navigation_state->set_request_committed(true);
@@ skipping 3221 matching lines @@
       policy(info.default_policy),
       replaces_current_history_item(info.replaces_current_history_item),
       history_navigation_in_new_child_frame(
           info.is_history_navigation_in_new_child_frame),
       client_redirect(info.is_client_redirect),
       cache_disabled(info.is_cache_disabled),
       form(info.form),
       source_location(info.source_location) {}
 
 }  // namespace content