Title: [188903] trunk/Tools
- Revision
- 188903
- Author
- [email protected]
- Date
- 2015-08-24 20:25:53 -0700 (Mon, 24 Aug 2015)
Log Message
[Win] Unreviewed test gardening.
Fix another case where the logging string is constructed using the variable 'frame',
but the method argument is not given a name. This causes the global 'frame' value to
be used, producing incorrect output.
* DumpRenderTree/win/FrameLoadDelegate.cpp:
(FrameLoadDelegate::didReceiveServerRedirectForProvisionalLoadForFrame): Use 'frame'
argument.
Modified Paths
Diff
Modified: trunk/Tools/ChangeLog (188902 => 188903)
--- trunk/Tools/ChangeLog 2015-08-25 01:56:46 UTC (rev 188902)
+++ trunk/Tools/ChangeLog 2015-08-25 03:25:53 UTC (rev 188903)
@@ -1,3 +1,15 @@
+2015-08-24 Brent Fulgham <[email protected]>
+
+ [Win] Unreviewed test gardening.
+
+ Fix another case where the logging string is constructed using the variable 'frame',
+ but the method argument is not given a name. This causes the global 'frame' value to
+ be used, producing incorrect output.
+
+ * DumpRenderTree/win/FrameLoadDelegate.cpp:
+ (FrameLoadDelegate::didReceiveServerRedirectForProvisionalLoadForFrame): Use 'frame'
+ argument.
+
2015-08-24 Tim Horton <[email protected]>
(Finally...) get rid of WKViewIOS
Modified: trunk/Tools/DumpRenderTree/win/FrameLoadDelegate.cpp (188902 => 188903)
--- trunk/Tools/DumpRenderTree/win/FrameLoadDelegate.cpp 2015-08-25 01:56:46 UTC (rev 188902)
+++ trunk/Tools/DumpRenderTree/win/FrameLoadDelegate.cpp 2015-08-25 03:25:53 UTC (rev 188903)
@@ -133,7 +133,7 @@
return S_OK;
}
-HRESULT FrameLoadDelegate::didReceiveServerRedirectForProvisionalLoadForFrame(_In_opt_ IWebView*, _In_opt_ IWebFrame*)
+HRESULT FrameLoadDelegate::didReceiveServerRedirectForProvisionalLoadForFrame(_In_opt_ IWebView*, _In_opt_ IWebFrame* frame)
{
if (!done && gTestRunner->dumpFrameLoadCallbacks())
printf("%s - didReceiveServerRedirectForProvisionalLoadForFrame\n", descriptionSuitableForTestResult(frame).c_str());
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
