Bug 1147793 - Remove unnecessary intr semantic in ipdl file. r=bent
authorJerryShih <hshih@mozilla.com>
Thu, 26 Mar 2015 03:04:00 -0400
changeset 264979 f95cac3a56942f284faddc835c49d0a8960b4da4
parent 264978 7e4aab7d129f8c122885c6e7ba729c56b2245811
child 264980 501bd236aee30acc3df8a671f7eb575b96f5424b
push id4718
push userraliiev@mozilla.com
push dateMon, 11 May 2015 18:39:53 +0000
treeherdermozilla-beta@c20c4ef55f08 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
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 1147793 - Remove unnecessary intr semantic in ipdl file. r=bent
--- a/dom/ipc/PContent.ipdl
+++ b/dom/ipc/PContent.ipdl
@@ -605,18 +605,17 @@ parent:
                             TabId openerTabId)
         returns (ContentParentId cpId, bool isForApp, bool isForBrowser, TabId tabId);
     sync BridgeToChildProcess(ContentParentId cpId);
      * This call connects the content process to a plugin process. While this
      * call runs, a new PluginModuleParent will be created in the ContentChild
      * via bridging. The corresponding PluginModuleChild will live in the plugin
-     * process. We use intr semantics here to ensure that the PluginModuleParent
-     * allocation message is dispatched before LoadPlugin returns.
+     * process.
     sync LoadPlugin(uint32_t pluginId) returns (nsresult rv);
      * This call is used by asynchronous plugin instantiation to notify the
      * content parent that it is now safe to initiate the plugin bridge for
      * the specified plugin id. When this call returns, the requested bridge
      * connection has been made.
--- a/gfx/layers/ipc/PCompositor.ipdl
+++ b/gfx/layers/ipc/PCompositor.ipdl
@@ -24,19 +24,17 @@ namespace mozilla {
 namespace layers {
  * The PCompositor protocol is used to manage communication between
  * the main thread and the compositor thread context. It's primary
  * purpose is to manage the PLayerTransaction sub protocol.
-// This should really be 'sync', but we're using 'rpc' as a workaround
-// for Bug 716631.
-intr protocol PCompositor
+sync protocol PCompositor
   // A Compositor manages a single Layer Manager (PLayerTransaction)
   manages PLayerTransaction;
   // The child should invalidate everything so that the whole window is redrawn.
   async InvalidateAll();
--- a/gfx/layers/ipc/PImageBridge.ipdl
+++ b/gfx/layers/ipc/PImageBridge.ipdl
@@ -18,17 +18,17 @@ using mozilla::layers::TextureFlags from
 namespace mozilla {
 namespace layers {
  * The PImageBridge protocol is used to allow isolated threads or processes to push
  * frames directly to the compositor thread/process without relying on the main thread
  * which might be too busy dealing with content script.
-intr protocol PImageBridge
+sync protocol PImageBridge
   manages PCompositable;
   manages PTexture;
   async ParentAsyncMessages(AsyncParentMessageData[] aMessages);