Bug 1529762 - PageStyle child actors should only instantiate for web pages. r=Felipe
authorMike Conley <mconley@mozilla.com>
Fri, 08 Mar 2019 20:22:19 +0000
changeset 463220 cd0a4ead024efdaa27e4ec82e80f3aee7f632ec3
parent 463219 6d58a14e2ceff0829263dcabb07f5c2db07adf0d
child 463221 92aa678ac36b4460a434de28e768c22cffea6242
push id80061
push usermconley@mozilla.com
push dateFri, 08 Mar 2019 20:26:05 +0000
treeherderautoland@8ac8ed5ac772 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersFelipe
bugs1529762
milestone67.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 1529762 - PageStyle child actors should only instantiate for web pages. r=Felipe This avoids instantiating them for internal pages, like about:home. Differential Revision: https://phabricator.services.mozilla.com/D22520
browser/base/content/test/performance/browser_startup_content.js
browser/components/BrowserGlue.jsm
--- a/browser/base/content/test/performance/browser_startup_content.js
+++ b/browser/base/content/test/performance/browser_startup_content.js
@@ -41,17 +41,16 @@ const whitelist = {
     "resource:///modules/sessionstore/ContentSessionStore.jsm",
     "resource://gre/modules/sessionstore/SessionHistory.jsm",
 
     // Browser front-end
     "resource:///actors/AboutReaderChild.jsm",
     "resource:///actors/BrowserTabChild.jsm",
     "resource:///modules/ContentMetaHandler.jsm",
     "resource:///actors/LinkHandlerChild.jsm",
-    "resource:///actors/PageStyleChild.jsm",
     "resource:///actors/SearchTelemetryChild.jsm",
     "resource://gre/modules/ActorChild.jsm",
     "resource://gre/modules/ActorManagerChild.jsm",
     "resource://gre/modules/E10SUtils.jsm",
     "resource://gre/modules/Readerable.jsm",
     "resource://gre/modules/WebProgressChild.jsm",
 
     // Telemetry
--- a/browser/components/BrowserGlue.jsm
+++ b/browser/components/BrowserGlue.jsm
@@ -210,16 +210,19 @@ let ACTORS = {
       group: "browsers",
       events: {
         "pageshow": {},
       },
       messages: [
         "PageStyle:Switch",
         "PageStyle:Disable",
       ],
+      // Only matching web pages, as opposed to internal about:, chrome: or
+      // resource: pages. See https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/Match_patterns
+      matches: ["*://*/*"],
     },
   },
 
   Plugin: {
     child: {
       module: "resource:///actors/PluginChild.jsm",
       events: {
         "PluginBindingAttached": {capture: true, wantUntrusted: true},