As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 765590 - CSSNameSpaceRule.type should be 10, not 0
: CSSNameSpaceRule.type should be 10, not 0
[good first bug][mentor=Ms2ger][lang=...
: dev-doc-complete
Product: Core
Classification: Components
Component: DOM: CSS Object Model (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla16
Assigned To: Jignesh Kakadiya [:jhk]
: Andrew Overholt [:overholt]
Depends on:
  Show dependency treegraph
Reported: 2012-06-17 10:16 PDT by :Ms2ger (⌚ UTC+1/+2)
Modified: 2012-08-13 07:31 PDT (History)
5 users (show)
dholbert: in‑testsuite+
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch(v1) (1.87 KB, patch)
2012-06-21 03:19 PDT, Jignesh Kakadiya [:jhk]
Ms2ger: feedback+
Details | Diff | Splinter Review
Patch(v2) (3.38 KB, patch)
2012-06-21 05:55 PDT, Jignesh Kakadiya [:jhk]
dbaron: review+
Ms2ger: feedback+
Details | Diff | Splinter Review

Comment 1 User image David Baron :dbaron: ⌚️UTC-8 2012-06-17 10:35:29 PDT
Yeah, I don't think there was a value assigned at the time we implemented this.

Are you interested in posting a patch?
Comment 2 User image Kang-Hao (Kenny) Lu [:kennyluck] 2012-06-17 10:37:32 PDT
We could probably make this a "good first bug" too, although I am not sure this is really a good one...
Comment 3 User image :Ms2ger (⌚ UTC+1/+2) 2012-06-17 11:08:06 PDT
To fix this bug, a new constant NAMESPACE_RULE = 10; should be added to <>, and the implementation at <> should be updated to use it instead of the UNKNOWN_RULE it has now.
Comment 4 User image David Baron :dbaron: ⌚️UTC-8 2012-06-17 12:51:56 PDT
and also see
Comment 5 User image Jignesh Kakadiya [:jhk] 2012-06-21 03:19:18 PDT
Created attachment 635234 [details] [diff] [review]
Comment 6 User image :Ms2ger (⌚ UTC+1/+2) 2012-06-21 03:32:24 PDT
Comment on attachment 635234 [details] [diff] [review]

Review of attachment 635234 [details] [diff] [review]:

Looks good to me, code-wise; a test would be good, though.
Comment 7 User image Jignesh Kakadiya [:jhk] 2012-06-21 05:55:30 PDT
Created attachment 635276 [details] [diff] [review]

Patch+test. I tested it locally and it doesn't break any other tests.
Comment 8 User image :Ms2ger (⌚ UTC+1/+2) 2012-06-21 06:21:03 PDT
Comment on attachment 635276 [details] [diff] [review]

Review of attachment 635276 [details] [diff] [review]:

Looks good to me.
Comment 9 User image David Baron :dbaron: ⌚️UTC-8 2012-06-21 11:29:12 PDT
Comment on attachment 635276 [details] [diff] [review]

Comment 10 User image Daniel Holbert [:dholbert] 2012-06-21 13:00:45 PDT
Comment 11 User image Ed Morley [:emorley] 2012-06-22 03:45:27 PDT
Comment 12 User image Jean-Yves Perrier [:teoli] 2012-08-13 07:31:26 PDT
I've added a line in :

Updated: which was quite outdated.

Created: as it didn't existed at all!

Note that I still to add a link from the doc for @namespace to the CSSOM interface, but as this at-rule is not yet documented, I'll wait a bit. (Anyway I have to make sure that every combination of at-rule/CSSOM interface are correctly linked and documented.

Note You need to log in before you can comment on or make changes to this bug.