Bug 1314520 - part 1 Don't trigger a ProcessingError when we fail to send a PAPZConstructor. r=rhunt a=IanN DONTBUILD CLOSED TREE SEAMONKEY_2_49_ESR_RELBRANCH
authorDavid Anderson <danderson@mozilla.com>
Mon, 14 Nov 2016 14:58:18 -0800
branchSEAMONKEY_2_49_ESR_RELBRANCH
changeset 357568 75494cf725fead68d00bc15d189259d707ac9ca8
parent 357567 4939882bc713553dc564fae3d5b67a8b896496ca
child 357569 e0e23e1fd9e3bf023397ebf0dd412ccc6d6954d4
push id7836
push userfrgrahl@gmx.net
push dateMon, 15 Jul 2019 16:17:02 +0000
treeherdermozilla-esr52@3a4afe7ecac5 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrhunt, IanN
bugs1314520
milestone52.9.1
Bug 1314520 - part 1 Don't trigger a ProcessingError when we fail to send a PAPZConstructor. r=rhunt a=IanN DONTBUILD CLOSED TREE SeaMonkey ESR52 release branch only.
dom/ipc/ContentChild.cpp
--- a/dom/ipc/ContentChild.cpp
+++ b/dom/ipc/ContentChild.cpp
@@ -1458,18 +1458,21 @@ ContentChild::RecvSetProcessSandbox(cons
 
 bool
 ContentChild::RecvNotifyLayerAllocated(const dom::TabId& aTabId, const uint64_t& aLayersId)
 {
   if (!CompositorBridgeChild::Get()->IPCOpen()) {
     return true;
   }
 
+  // Note: sending the constructor could fail, but we do not propagate the
+  // error back since the GPU process is fallible.
   APZChild* apz = ContentProcessController::Create(aTabId);
-  return CompositorBridgeChild::Get()->SendPAPZConstructor(apz, aLayersId);
+  CompositorBridgeChild::Get()->SendPAPZConstructor(apz, aLayersId);
+  return true;
 }
 
 bool
 ContentChild::RecvSpeakerManagerNotify()
 {
 #ifdef MOZ_WIDGET_GONK
   // Only notify the process which has the SpeakerManager instance.
   RefPtr<SpeakerManagerService> service =