Bug 1311048 - Surface process id in application sharing for completeness (we never share own process). r=jesup
authorJan-Ivar Bruaroey <jib@mozilla.com>
Fri, 04 Nov 2016 00:59:47 -0400
changeset 348006 b8f2d6428e38806fb1ce5898ed2a16d3b7e77d3c
parent 348005 8da036418f24571d18ad32a5b7870571e8f141a8
child 348007 dff36aad2d6ab88c1c2afd3419e4d614fda95058
push id10298
push userraliiev@mozilla.com
push dateMon, 14 Nov 2016 12:33:03 +0000
treeherdermozilla-aurora@7e29173b1641 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjesup
bugs1311048
milestone52.0a1
Bug 1311048 - Surface process id in application sharing for completeness (we never share own process). r=jesup MozReview-Commit-ID: HShFSBMO0yx
media/webrtc/trunk/webrtc/video_engine/desktop_capture_impl.cc
--- a/media/webrtc/trunk/webrtc/video_engine/desktop_capture_impl.cc
+++ b/media/webrtc/trunk/webrtc/video_engine/desktop_capture_impl.cc
@@ -182,16 +182,19 @@ int32_t AppDeviceInfoImpl::GetDeviceName
 
     const char *deviceUniqueId = desktopApplication.getUniqueIdName();
     len = deviceUniqueId ? strlen(deviceUniqueId) : 0;
     if (len && deviceUniqueIdUTF8 && len <= deviceUniqueIdUTF8Length) {
       memcpy(deviceUniqueIdUTF8,
              deviceUniqueId,
              len);
     }
+    if (pid) {
+      *pid = desktopApplication.getProcessId();
+    }
   }
   return 0;
 }
 
 int32_t AppDeviceInfoImpl::DisplayCaptureSettingsDialogBox(const char* deviceUniqueIdUTF8,
                                                            const char* dialogTitleUTF8,
                                                            void* parentWindow,
                                                            uint32_t positionX,