Title: [188812] trunk/Tools
Revision
188812
Author
[email protected]
Date
2015-08-21 18:26:50 -0700 (Fri, 21 Aug 2015)

Log Message

[Win] Unreviewed test correction after r188709.

I removed a function argument that happened to be mirrored by a global
value with the same name. This prevented a compile error, but testing
showed that the wrong value was being used in the function.

* DumpRenderTree/win/FrameLoadDelegate.cpp:
(FrameLoadDelegate::willPerformClientRedirectToURL): Put function argument
back

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (188811 => 188812)


--- trunk/Tools/ChangeLog	2015-08-22 00:56:55 UTC (rev 188811)
+++ trunk/Tools/ChangeLog	2015-08-22 01:26:50 UTC (rev 188812)
@@ -1,3 +1,15 @@
+2015-08-21  Brent Fulgham  <[email protected]>
+
+        [Win] Unreviewed test correction after r188709.
+
+        I removed a function argument that happened to be mirrored by a global
+        value with the same name. This prevented a compile error, but testing
+        showed that the wrong value was being used in the function.
+
+        * DumpRenderTree/win/FrameLoadDelegate.cpp:
+        (FrameLoadDelegate::willPerformClientRedirectToURL): Put function argument
+        back
+
 2015-08-21  Timothy Horton  <[email protected]>
 
         WebKitTestRunner should use WKWebView on OS X and iOS

Modified: trunk/Tools/DumpRenderTree/win/FrameLoadDelegate.cpp (188811 => 188812)


--- trunk/Tools/DumpRenderTree/win/FrameLoadDelegate.cpp	2015-08-22 00:56:55 UTC (rev 188811)
+++ trunk/Tools/DumpRenderTree/win/FrameLoadDelegate.cpp	2015-08-22 01:26:50 UTC (rev 188812)
@@ -268,7 +268,7 @@
     return S_OK;
 }
 
-HRESULT FrameLoadDelegate::willPerformClientRedirectToURL(_In_opt_ IWebView*, _In_ BSTR url, double /*delaySeconds*/, DATE /*fireDate*/, _In_opt_ IWebFrame*)
+HRESULT FrameLoadDelegate::willPerformClientRedirectToURL(_In_opt_ IWebView*, _In_ BSTR url, double /*delaySeconds*/, DATE /*fireDate*/, _In_opt_ IWebFrame* frame)
 {
     if (!done && gTestRunner->dumpFrameLoadCallbacks())
         printf("%s - willPerformClientRedirectToURL: %S \n", descriptionSuitableForTestResult(frame).c_str(),
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to