72302 - Default namespace in <html:style> is HTML by default [NAMESPACE]

Status: VERIFIED FIXED

(Core :: CSS Parsing and Computation, defect, P3)

Description

[Hixie (not reading bugmail)]

This broke out of bug 71647.

STEPS TO REPRODUCE
   Build a stylesheet that defines some namespaces, but no default namespace,
   for example:
      @namespace html url(http://www.w3.org/1999/xhtml);
      @namespace test url(http://www.example.org/);
      test { color: green; }

ACTUAL RESULTS
   Our CSS parser generates the following type element selector:
      mTag          "testA"
      mNameSpace    0x00000003  (kNameSpaceID_HTML)
   ...so in fact the sheet above is actually stored as:
      @namespace html url(http://www.w3.org/1999/xhtml);
      @namespace test url(http://www.example.org/);
      html|testA  { color: green; }

EXPECTED RESULTS
   Without a default namespace declaration in CSS, "test" should be interpreted 
   as "*|test".

See: http://www.hixie.ch/tests/adhoc/css/selectors/namespace/002.xml

Comment 1

[Hixie (not reading bugmail)]

Well well. Good catch Daniel.

As can be seen from:
   http://www.hixie.ch/tests/adhoc/css/selectors/namespace/003.xml
...the problem is actually a little different. We seem to be saying that the
<html:style> element sets a default namespace! This is wrong.

Comment 2

[rubydoo123]

mozilla0.9/P3

Comment 3

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Attachment: proposed fix

Comment 4

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Proposing fix for this bug. Ian's test goes green with it.

(s)reviews needed, in particular for possible (?) side effects of the proposed
modifications.

Ian, can you please torture it a little bit ? thanks.

Comment 5

[Marc Attinasi]

[s]r=attinasi on the patch.  Seems like it was a bit presumptuous to assume the
HTML namespace in the first place, so any regressions should be fixed at the
regression site rather than by backing this out.

Comment 6

[Peter Van der Beken [:peterv]]

r=peterv.

Comment 7

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

checked in ; Ian's tests about namespaces become green.

http://www.hixie.ch/tests/adhoc/css/selectors/namespace/

Comment 8

[Hixie (not reading bugmail)]

verified per 003 and 002.