Navigation transitions (web to native app): Pass data after starting provisional load (Chrome side)

content/browser/frame_host/navigator_impl.cc

Index: content/browser/frame_host/navigator_impl.cc
diff --git a/content/browser/frame_host/navigator_impl.cc b/content/browser/frame_host/navigator_impl.cc
index c5b67b40ad6c2b7f8f9df41354ecff7b2155831a..ad712a1749a4befc3b0fcefdba2a2a075da17a8f 100644
--- a/content/browser/frame_host/navigator_impl.cc
+++ b/content/browser/frame_host/navigator_impl.cc
@@ -255,8 +255,16 @@ void NavigatorImpl::DidStartProvisionalLoad(
         delegate_->NotifyChangedNavigationState(content::INVALIDATE_TYPE_URL);
     }
 
-    if (delegate_ && is_transition_navigation)
-      delegate_->DidStartNavigationTransition(render_frame_host);
+    if (delegate_ && is_transition_navigation) {
+      TransitionLayerData transition_data;
+      TransitionRequestManager::GetInstance()->HasPendingTransitionRequest(

[nasko, 2014/10/29 22:14:24]

This method name looks very weird. A Has* method should return a boolean and not
the actual data. Renaming to Get* fits more of how it is actually used here.

[Zhen Wang, 2014/10/29 23:11:05]

Done.

+          render_frame_host->GetProcess()->GetID(),
+          render_frame_host->GetRoutingID(),
+          validated_url,
+          &transition_data);
+      delegate_->DidStartNavigationTransition(render_frame_host,
+                                              transition_data);
+    }
   }
 
   if (delegate_) {