Bug 17082 – Acid3 bug: CSS objects in Webkit are not "live"

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Eric Seidel (no email)]

WebKit fails Acid3's modification of style blocks text node test

This will require some reduction.  It might be a bug in the test.

Test 73: Undefined value

    function () {
      // test 73: dynamic modification of <style> blocks' text nodes, from Jonas Sicking and Garret Smith
      var doc = kungFuDeathGrip.childNodes[3].contentDocument;
      assert(doc, "missing document for test");
      assert(doc.images[0], "prerequisite failed: no image");
      assertEquals(doc.images[0].height, 10, "prerequisite failed: style didn't affect image");
      assertEquals(doc.styleSheets[0].href, null, "internal stylesheet had a URI");
      doc.styleSheets[0].ownerNode.firstChild.data = "img { height: 20px; }";
      assertEquals(doc.images[0].height, 20, "change failed to take effect");
      doc.styleSheets[0].ownerNode.appendChild(doc.createTextNode("img { height: 30px; }"));
      assertEquals(doc.images[0].height, 30, "append failed to take effect");
      var rules = doc.styleSheets.cssRules; // "All CSS objects in the DOM are "live"" says section 2.1, Overview of the DOM Level 2 CSS Interfaces
      doc.styleSheets[0].insertRule("img { height: 40px; }", 2);
      assertEquals(doc.images[0].height, 40, "insertRule failed to take effect");
      assertEquals(doc.styleSheets.cssRules.length, rules.length, "cssRules isn't live");
      return 5;
    },

[Eric Seidel (no email)]

We'll need to build this into a nice stand-alone test case.

[Robert Blaut]

Created attachment 18905 [details]
minimal test case

[Robert Blaut]

The acid3 test 73 was changed a little:

 function () {
      // test 73: dynamic modification of <style> blocks' text nodes, from Jonas Sicking and Garret Smith
      var doc = kungFuDeathGrip.childNodes[3].contentDocument;
      assert(doc, "missing document for test");
      assert(doc.images[0], "prerequisite failed: no image");
      assertEquals(doc.images[0].height, 10, "prerequisite failed: style didn't affect image");
      doc.styleSheets[0].ownerNode.firstChild.data = "img { height: 20px; }";
      assertEquals(doc.images[0].height, 20, "change failed to take effect");
      doc.styleSheets[0].ownerNode.appendChild(doc.createTextNode("img { height: 30px; }"));
      assertEquals(doc.images[0].height, 30, "append failed to take effect");
      var rules = doc.styleSheets[0].cssRules; // "All CSS objects in the DOM are "live"" says section 2.1, Overview of the DOM Level 2 CSS Interfaces
      doc.styleSheets[0].insertRule("img { height: 40px; }", 2);
      assertEquals(doc.images[0].height, 40, "insertRule failed to take effect");
      assertEquals(doc.styleSheets[0].cssRules.length, 3, "count of rules is wrong");
      assertEquals(rules.length, 3, "cssRules isn't live");
      // while we're at it, check some other things on doc.styleSheets:
      assert(doc.styleSheets[0].href === null, "internal stylesheet had a URI: " + doc.styleSheets[0].href);
      assert(document.styleSheets[0].href === null, "internal acid3 stylesheet had a URI: " + document.styleSheets[0].href);
      return 5;
    },

[Dave Hyatt]

I'm working on this one.  Nearly have a fix ready.

[Dave Hyatt]

Created attachment 19034 [details]
Make the rules list live for stylesheets.

This code is just a huge mess.  This is the minimal surgery that I had to do to get this working (and should perform better).  The old code copied all of the rules into a separate list.   The new code will not do this if a StyleList is available, unless the IE rules() extension is being used (and charset rules have to be omitted).

In the longer term, this class could be fixed to just always reference StyleLists and to never have its own internal list.  It would have to operate a bit more like a collection in that case (knowing to skip charset rules if created as an IE rules() extension list).

[Oliver Hunt]

Comment on attachment 19034 [details]
Make the rules list live for stylesheets.

Testcases amd changelogs are your friend :p

[Dave Hyatt]

Fixed in r30130.