Closed
Bug 1345698
Opened 8 years ago
Closed 8 years ago
stylo: Implement access to CSSNamespaceRule
Categories
(Core :: CSS Parsing and Computation, enhancement, P2)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla55
| Tracking | Status | |
|---|---|---|
| firefox55 | --- | fixed |
People
(Reporter: xidorn, Assigned: xidorn)
References
Details
Attachments
(5 files)
|
(deleted),
text/x-review-board-request
|
heycam
:
review+
|
Details |
|
(deleted),
text/x-review-board-request
|
heycam
:
review+
|
Details |
|
(deleted),
text/x-review-board-request
|
heycam
:
review+
|
Details |
|
(deleted),
text/x-review-board-request
|
heycam
:
review+
|
Details |
|
(deleted),
text/x-review-board-request
|
heycam
:
review+
|
Details |
No description provided.
|
Assignee | |
Comment 1•8 years ago
|
||
I'll work on this after bug 1315601. Since this is a trivial one to implement, I'd like to use this bug to do some code simplification as well.
Assignee: nobody → xidorn+moz
Depends on: 1315601
|
||
Updated•8 years ago
|
Priority: -- → P2
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
|
||
Comment 7•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8847031 [details]
Bug 1345698 part 1 - Unify common code for implementing a rule type.
https://reviewboard.mozilla.org/r/120074/#review122284
Attachment #8847031 -
Flags: review?(cam) → review+
|
|
||
Comment 8•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8847032 [details]
Bug 1345698 part 2 - Add bindings for namespace rule.
https://reviewboard.mozilla.org/r/120076/#review122290
Attachment #8847032 -
Flags: review?(cam) → review+
|
|
||
Comment 9•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8847033 [details]
Bug 1345698 part 3 - Fix a build failure for adding source.
https://reviewboard.mozilla.org/r/120078/#review122292
Attachment #8847033 -
Flags: review?(cam) → review+
|
|
||
Comment 10•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8847034 [details]
Bug 1345698 part 4 - Create a new CSSNamespaceRule as a base class.
https://reviewboard.mozilla.org/r/120080/#review122298
Attachment #8847034 -
Flags: review?(cam) → review+
|
|
||
Comment 11•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8847035 [details]
Bug 1345698 part 5 - Add support of CSSNamespaceRule for stylo.
https://reviewboard.mozilla.org/r/120082/#review122304
Attachment #8847035 -
Flags: review?(cam) → review+
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
|
||
Comment 17•8 years ago
|
||
Pushed by xquan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/52ba7a56ee83
part 1 - Unify common code for implementing a rule type. r=heycam
https://hg.mozilla.org/integration/autoland/rev/7caba57b2d1b
part 2 - Add bindings for namespace rule. r=heycam
https://hg.mozilla.org/integration/autoland/rev/fb56873eb923
part 3 - Fix a build failure for adding source. r=heycam
https://hg.mozilla.org/integration/autoland/rev/0ee93f46f578
part 4 - Create a new CSSNamespaceRule as a base class. r=heycam
https://hg.mozilla.org/integration/autoland/rev/3123f116d6dc
part 5 - Add support of CSSNamespaceRule for stylo. r=heycam
|
||
Comment 18•8 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/52ba7a56ee83
https://hg.mozilla.org/mozilla-central/rev/7caba57b2d1b
https://hg.mozilla.org/mozilla-central/rev/fb56873eb923
https://hg.mozilla.org/mozilla-central/rev/0ee93f46f578
https://hg.mozilla.org/mozilla-central/rev/3123f116d6dc
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•