TEST FAILED: test-page-mod.testPageModCssList (exception)

RESOLVED FIXED

Status

P1
major
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: KWierso, Assigned: zer0)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

7 years ago
error: TEST FAILED: test-page-mod.testPageModCssList (exception)
error: An exception occurred.
Traceback (most recent call last):
  File "resource://1461724c-bb7a-4614-b700-c8fb0a173d75-at-jetpack/addon-kit/lib/page-mod.js", line 283, in 
    exports.PageMod = function(options) PageMod(options)
  File "resource://1461724c-bb7a-4614-b700-c8fb0a173d75-at-jetpack/api-utils/lib/traits.js", line 110, in Trait
    return self.constructor.apply(self, arguments) || self._public;
  File "resource://1461724c-bb7a-4614-b700-c8fb0a173d75-at-jetpack/addon-kit/lib/page-mod.js", line 109, in PageMod
    let { contentStyle, contentStyleFile } = validate(options, validStyleOptions);
  File "resource://1461724c-bb7a-4614-b700-c8fb0a173d75-at-jetpack/api-utils/lib/api-utils.js", line 109, in validateOptions
    throw requirementError(key, req);
  File "resource://1461724c-bb7a-4614-b700-c8fb0a173d75-at-jetpack/api-utils/lib/api-utils.js", line 152, in requirementError
    return new Error(msg);
Error: The `contentStyleFile` option must be a local URL or an array of URLs




From the commit, it looks like this: 
+ contentStyleFile: [
+        // Highlight evaluation order in this list
+        "data:text/css,div { border: 1px solid black; }",
+        "data:text/css,div { border: 10px solid black; }",
+        // Highlight evaluation order between contentStylesheet & contentStylesheetFile
+        "data:text/css,div { height: 1000px; }",
+        // Highlight precedence between the author and user style sheet
+        "data:text/css,div { width: 200px; max-width: 640px!important}",
+      ],

doesn't count as an array of local URLs.
(Reporter)

Comment 1

7 years ago
A fix for this landed in https://github.com/mozilla/addon-sdk/commit/362667ee7568ffc79a13e0f1ca8fe3a33de3ed31 over in the original bug.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.