Closed Bug 645956 Opened 14 years ago Closed 14 years ago

Remove NS_NewCSS*Rule functions

Categories

(Core :: CSS Parsing and Computation, defect)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla5

People

(Reporter: craig.topper, Assigned: craig.topper)

References

Details

Attachments

(5 files)

Part 1: Remove NS_NewCSSStyleRule
(deleted), patch
bzbarsky
: review+
Details | Diff | Splinter Review
Part 2: Remove NS_NewCSSImportRule
(deleted), patch
bzbarsky
: review+
Details | Diff | Splinter Review
Part 3: Remove NS_NewCSSNamespaceRule
(deleted), patch
bzbarsky
: review+
Details | Diff | Splinter Review
Part 4: Move CSSCharsetRule to header file and put in css namespace
(deleted), patch
bzbarsky
: review+
Details | Diff | Splinter Review
Part 5: Remove NS_NewCSSCharsetRule
(deleted), patch
bzbarsky
: review+
Details | Diff | Splinter Review
Now that all the CSS rule interface classes are gone we no longer need these creation functions and we can just allocate the classes directly.
Attachment #522596 - Flags: review?(bzbarsky)
Attachment #522597 - Flags: review?(bzbarsky)
Attachment #522599 - Flags: review?(bzbarsky)
Attachment #522600 - Flags: review?(bzbarsky)
Attachment #522601 - Flags: review?(bzbarsky)
Comment on attachment 522596 [details] [diff] [review] Part 1: Remove NS_NewCSSStyleRule r=me
Attachment #522596 - Flags: review?(bzbarsky) → review+
Comment on attachment 522597 [details] [diff] [review] Part 2: Remove NS_NewCSSImportRule r=me
Attachment #522597 - Flags: review?(bzbarsky) → review+
Comment on attachment 522599 [details] [diff] [review] Part 3: Remove NS_NewCSSNamespaceRule r=me
Attachment #522599 - Flags: review?(bzbarsky) → review+
Depends on: 577976
Comment on attachment 522600 [details] [diff] [review] Part 4: Move CSSCharsetRule to header file and put in css namespace r=me
Attachment #522600 - Flags: review?(bzbarsky) → review+
Comment on attachment 522601 [details] [diff] [review] Part 5: Remove NS_NewCSSCharsetRule r=me
Attachment #522601 - Flags: review?(bzbarsky) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: