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

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect, P1)

Description

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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.

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

A fix for this landed in https://github.com/mozilla/addon-sdk/commit/362667ee7568ffc79a13e0f1ca8fe3a33de3ed31 over in the original bug.