Navigation transitions (web to native app): Get names and rects of transition elements (Step 2)

Source/web/FrameLoaderClientImpl.cpp

Index: Source/web/FrameLoaderClientImpl.cpp
diff --git a/Source/web/FrameLoaderClientImpl.cpp b/Source/web/FrameLoaderClientImpl.cpp
index 107504056be14563f79a833c878ac0f713516dbe..0e1236b79ac1eb43b224afdf69436f629d0796ed 100644
--- a/Source/web/FrameLoaderClientImpl.cpp
+++ b/Source/web/FrameLoaderClientImpl.cpp
@@ -478,10 +478,15 @@ NavigationPolicy FrameLoaderClientImpl::decidePolicyForNavigation(const Resource
     return static_cast<NavigationPolicy>(webPolicy);
 }
 
-void FrameLoaderClientImpl::dispatchAddNavigationTransitionData(const String& allowedDestinationOrigin, const String& selector, const String& markup)
+void FrameLoaderClientImpl::dispatchAddNavigationTransitionData(
+    const String& allowedDestinationOrigin, const String& selector,
+    const String& markup, const Vector<String>& names,

[sof, 2014/10/24 15:17:14]

Ditto

[Zhen Wang, 2014/10/27 14:14:45]

Using struct now.

+    const Vector<IntRect>& rects)
 {
-    if (m_webFrame->client())
-        m_webFrame->client()->addNavigationTransitionData(allowedDestinationOrigin, selector, markup);
+    if (m_webFrame->client()) {
+        m_webFrame->client()->addNavigationTransitionData(
+            allowedDestinationOrigin, selector, markup, names, rects);

[sof, 2014/10/24 15:17:14]

Keep line length style as-was?

[Zhen Wang, 2014/10/27 14:14:45]

This will be changed into using struct in later CLs.

By the way, I see other places also using multiple line style. Are we guided to
use single line statements only?

+    }
 }
 
 void FrameLoaderClientImpl::dispatchWillRequestResource(FetchRequest* request)