Bug 1429425 - contentScripts.register runAt option should be set to document_idle when missing. r=mixedpuppy,zombie

MozReview-Commit-ID: 1D1J8zmVWqC

--HG--
extra : rebase_source : f17dbf6ec0cab40edee472933fd901c5b44e683f

toolkit/components/extensions/ext-contentScripts.js

@@ -68,7 +68,7 @@ class ContentScriptParent {
       exclude_globs: details.excludeGlobs,
       all_frames: details.allFrames,
       match_about_blank: details.matchAboutBlank,
-      run_at: details.runAt,
+      run_at: details.runAt || "document_idle",
       js: [],
       css: [],
     };

toolkit/components/extensions/test/xpcshell/test_ext_contentScripts_register.js

@@ -308,7 +308,6 @@ add_task(async function test_contentscripts_register_js() {
         js: [{code: `(${textScriptCodeIdle})()`}],
         runAt: "document_idle",
       },
-
       // Extension URLs.
       {
         matches: ["http://localhost/*/file_sample.html"],
@@ -328,7 +327,7 @@ add_task(async function test_contentscripts_register_js() {
       {
         matches: ["http://localhost/*/file_sample.html"],
         js: [{file: "content_script.js"}],
-        runAt: "document_idle",
+        // "runAt" is not specified here to ensure that it defaults to document_idle when missing.
       },
     ];