Title: [239317] trunk/Source/WebKit
Revision
239317
Author
wenson_hs...@apple.com
Date
2018-12-17 20:39:35 -0800 (Mon, 17 Dec 2018)

Log Message

Unreviewed, fix the iOSMac engineering build

`generateRequestID()` is only invoked from code under `ENABLE(MEDIA_STREAM)`; move it under this guard to avoid
an unused function warning.

* UIProcess/UserMediaPermissionRequestManagerProxy.cpp:

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (239316 => 239317)


--- trunk/Source/WebKit/ChangeLog	2018-12-18 04:17:58 UTC (rev 239316)
+++ trunk/Source/WebKit/ChangeLog	2018-12-18 04:39:35 UTC (rev 239317)
@@ -1,3 +1,12 @@
+2018-12-17  Wenson Hsieh  <wenson_hs...@apple.com>
+
+        Unreviewed, fix the iOSMac engineering build
+
+        `generateRequestID()` is only invoked from code under `ENABLE(MEDIA_STREAM)`; move it under this guard to avoid
+        an unused function warning.
+
+        * UIProcess/UserMediaPermissionRequestManagerProxy.cpp:
+
 2018-12-17  David Kilzer  <ddkil...@apple.com>
 
         clang-tidy: Fix unnecessary object copy in CPUMonitor::setCPULimit()

Modified: trunk/Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.cpp (239316 => 239317)


--- trunk/Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.cpp	2018-12-18 04:17:58 UTC (rev 239316)
+++ trunk/Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.cpp	2018-12-18 04:39:35 UTC (rev 239317)
@@ -41,7 +41,6 @@
 
 #if ENABLE(MEDIA_STREAM)
 static const MediaProducer::MediaStateFlags activeCaptureMask = MediaProducer::HasActiveAudioCaptureDevice | MediaProducer::HasActiveVideoCaptureDevice;
-#endif
 
 static uint64_t generateRequestID()
 {
@@ -48,6 +47,7 @@
     static uint64_t uniqueRequestID = 1;
     return uniqueRequestID++;
 }
+#endif
 
 UserMediaPermissionRequestManagerProxy::UserMediaPermissionRequestManagerProxy(WebPageProxy& page)
     : m_page(page)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to