Bug 1867509 - Fix build with disabled WebRTC support r=pehrsons
authorJan Grulich <jgrulich@redhat.com>
Thu, 30 Nov 2023 14:16:01 +0000 (19 months ago)
changeset 687806 1b92bede1095b7e662ced99328c2d1db23d93d3e
parent 687805 1d1fb9d5a4974d94e8c317f226c962fd351eae27
child 687807 385880f4db2bb9a045b9b4e9997ae3150305639e
push id195981
push userpehrsons@gmail.com
push dateThu, 30 Nov 2023 14:18:44 +0000 (19 months ago)
treeherderautoland@1b92bede1095 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspehrsons
bugs1867509
milestone122.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1867509 - Fix build with disabled WebRTC support r=pehrsons We changed resolve value type for camera access requests and this was not updated in the non-WebRTC build. Differential Revision: https://phabricator.services.mozilla.com/D195102
ipc/glue/BackgroundParentImpl.cpp
--- a/ipc/glue/BackgroundParentImpl.cpp
+++ b/ipc/glue/BackgroundParentImpl.cpp
@@ -1369,17 +1369,17 @@ mozilla::ipc::IPCResult BackgroundParent
                      CameraAccessRequestPromise::ResolveOrRejectValue& aValue) {
                if (aValue.IsResolve()) {
                  resolver(aValue.ResolveValue());
                } else {
                  resolver(CamerasAccessStatus::Error);
                }
              });
 #else
-  aResolver(NS_ERROR_NOT_IMPLEMENTED);
+  aResolver(CamerasAccessStatus::Error);
 #endif
   return IPC_OK();
 }
 
 bool BackgroundParentImpl::DeallocPEndpointForReportParent(
     PEndpointForReportParent* aActor) {
   RefPtr<dom::EndpointForReportParent> actor =
       dont_AddRef(static_cast<dom::EndpointForReportParent*>(aActor));