Title: [141491] trunk/Tools
Revision
141491
Author
psola...@apple.com
Date
2013-01-31 14:56:47 -0800 (Thu, 31 Jan 2013)

Log Message

DumpRenderTree should put NSSound calls inside USE(APPKIT)
https://bugs.webkit.org/show_bug.cgi?id=108499

Reviewed by Andy Estes.

* DumpRenderTree/mac/DumpRenderTree.mm:
(dumpRenderTree):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (141490 => 141491)


--- trunk/Tools/ChangeLog	2013-01-31 22:33:55 UTC (rev 141490)
+++ trunk/Tools/ChangeLog	2013-01-31 22:56:47 UTC (rev 141491)
@@ -1,3 +1,13 @@
+2013-01-31  Pratik Solanki  <psola...@apple.com>
+
+        DumpRenderTree should put NSSound calls inside USE(APPKIT)
+        https://bugs.webkit.org/show_bug.cgi?id=108499
+
+        Reviewed by Andy Estes.
+
+        * DumpRenderTree/mac/DumpRenderTree.mm:
+        (dumpRenderTree):
+
 2013-01-30  Brian Weinstein  <bweinst...@apple.com>
 
         Add a call to the page UI client to determine if a plug-in should load

Modified: trunk/Tools/DumpRenderTree/mac/DumpRenderTree.mm (141490 => 141491)


--- trunk/Tools/DumpRenderTree/mac/DumpRenderTree.mm	2013-01-31 22:33:55 UTC (rev 141490)
+++ trunk/Tools/DumpRenderTree/mac/DumpRenderTree.mm	2013-01-31 22:56:47 UTC (rev 141491)
@@ -110,9 +110,11 @@
 +(void)setAllowsAnyHTTPSCertificate:(BOOL)allow forHost:(NSString *)host;
 @end
 
+#if USE(APPKIT)
 @interface NSSound (Details)
 + (void)_setAlertType:(NSUInteger)alertType;
 @end
+#endif
 
 static void runTest(const string& testPathOrURL);
 
@@ -860,7 +862,9 @@
     if (forceComplexText)
         [WebView _setAlwaysUsesComplexTextCodePath:YES];
 
+#if USE(APPKIT)
     [NSSound _setAlertType:0];
+#endif
 
     WebView *webView = createWebViewAndOffscreenWindow();
     mainFrame = [webView mainFrame];
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to