Status

()

defect
P4
minor
RESOLVED FIXED
9 years ago
8 years ago

People

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

Tracking

(Blocks 1 bug)

Trunk
mozilla5
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(6 attachments, 20 obsolete attachments)

12.70 KB, patch
bzbarsky
: review+
Details | Diff | Splinter Review
8.38 KB, patch
bzbarsky
: review+
Details | Diff | Splinter Review
3.62 KB, patch
bzbarsky
: review+
Details | Diff | Splinter Review
85.22 KB, patch
bzbarsky
: review+
Details | Diff | Splinter Review
5.22 KB, patch
bzbarsky
: review+
Details | Diff | Splinter Review
6.33 KB, patch
bzbarsky
: review+
Details | Diff | Splinter Review
(Assignee)

Description

9 years ago
This interface is only implemented by one class so there is no reason for it to exist.
(Assignee)

Comment 1

9 years ago
Attachment #455923 - Flags: review?(dbaron)
(Assignee)

Updated

9 years ago
Blocks: deCOM
(Assignee)

Comment 2

9 years ago
Attachment #455924 - Flags: review?(dbaron)
(Assignee)

Comment 3

9 years ago
Attachment #455925 - Flags: review?(dbaron)
(Assignee)

Comment 4

9 years ago
Attachment #455926 - Flags: review?(dbaron)
(Assignee)

Comment 5

9 years ago
Attachment #455928 - Flags: review?(dbaron)
(Assignee)

Updated

9 years ago
Attachment #455928 - Attachment is patch: true
Attachment #455928 - Attachment mime type: application/octet-stream → text/plain
(Assignee)

Comment 6

9 years ago
Attachment #455957 - Flags: review?(dbaron)
(Assignee)

Comment 7

9 years ago
Patches depend on patches in bug 575901 and bug 571946
Depends on: 571946, 575901
(Assignee)

Updated

9 years ago
Blocks: 577976
(Assignee)

Updated

9 years ago
Attachment #455923 - Flags: review?(dbaron) → review?(bzbarsky)
(Assignee)

Updated

9 years ago
Attachment #455924 - Flags: review?(dbaron) → review?(bzbarsky)
(Assignee)

Updated

9 years ago
Attachment #455925 - Flags: review?(dbaron) → review?(bzbarsky)
(Assignee)

Updated

9 years ago
Attachment #455926 - Flags: review?(dbaron) → review?(bzbarsky)
(Assignee)

Updated

9 years ago
Attachment #455928 - Flags: review?(dbaron) → review?(bzbarsky)
(Assignee)

Updated

9 years ago
Attachment #455957 - Flags: review?(dbaron) → review?(bzbarsky)
Comment on attachment 455923 [details] [diff] [review]
Part 1: Rename CSSStyleRuleImpl

r=bzbarsky
Attachment #455923 - Flags: review?(bzbarsky) → review+
Comment on attachment 455924 [details] [diff] [review]
Part 2: rename DOMCSSStyleRuleImpl

r=bzbarsky
Attachment #455924 - Flags: review?(bzbarsky) → review+
Though it's worth double-checking with dbaron on these renames.  Maybe we want to just drop both the "ns" prefix and the "Impl" suffix and stick these in mozilla::css?
Comment on attachment 455925 [details] [diff] [review]
Part 3: Rename CSSImportantRule

r=bzbarsky, with the caveat above.
Attachment #455925 - Flags: review?(bzbarsky) → review+
Comment on attachment 455928 [details] [diff] [review]
Part 5: Remove virtual from methods that came from nsICSSStyleRule

r=bzbarsky
Attachment #455928 - Flags: review?(bzbarsky) → review+
Comment on attachment 455957 [details] [diff] [review]
Part 6: DeCOMtaminate the methods that came from nsICSSStyleSheet

r=bzbarsky
Attachment #455957 - Flags: review?(bzbarsky) → review+
(In reply to comment #10)
> Though it's worth double-checking with dbaron on these renames.  Maybe we want
> to just drop both the "ns" prefix and the "Impl" suffix and stick these in
> mozilla::css?

We probably do, but I don't think it needs to happen here.  This is an improvement over the current state.
(Assignee)

Updated

9 years ago
Attachment #455926 - Flags: review?(bzbarsky)
(Assignee)

Comment 15

9 years ago
Old patch didn't apply anymore so I went ahead and added namespace.
Attachment #461798 - Flags: review?(bzbarsky)
(Assignee)

Updated

9 years ago
Attachment #455923 - Attachment is obsolete: true
(Assignee)

Updated

9 years ago
Attachment #461798 - Attachment description: 455923: Part 1: Rename CSSStyleRuleImpl updated to use namespace → Part 1: Rename CSSStyleRuleImpl updated to use namespace
(Assignee)

Comment 16

9 years ago
Attachment #455924 - Attachment is obsolete: true
Attachment #461799 - Flags: review?(bzbarsky)
(Assignee)

Comment 17

9 years ago
Attachment #461799 - Attachment is obsolete: true
Attachment #461800 - Flags: review?(bzbarsky)
Attachment #461799 - Flags: review?(bzbarsky)
(Assignee)

Updated

9 years ago
Attachment #461800 - Attachment description: 461799: Part 2: rename DOMCSSStyleRuleImpl updated to use namespace. → Part 2: rename DOMCSSStyleRuleImpl updated to use namespace.
(Assignee)

Comment 18

9 years ago
Attachment #461801 - Flags: review?(bzbarsky)
(Assignee)

Updated

9 years ago
Attachment #455925 - Attachment is obsolete: true
(Assignee)

Updated

9 years ago
Attachment #461801 - Attachment is obsolete: true
Attachment #461801 - Flags: review?(bzbarsky)
(Assignee)

Comment 19

9 years ago
Attachment #461803 - Flags: review?(bzbarsky)
(Assignee)

Comment 20

9 years ago
Attachment #455926 - Attachment is obsolete: true
Attachment #461804 - Flags: review?(bzbarsky)
(Assignee)

Comment 21

9 years ago
Attachment #455928 - Attachment is obsolete: true
Attachment #461805 - Flags: review?(bzbarsky)
(Assignee)

Comment 22

9 years ago
Attachment #455957 - Attachment is obsolete: true
Attachment #461807 - Flags: review?(bzbarsky)
(Assignee)

Comment 23

9 years ago
Updating for bitrot
Attachment #461798 - Attachment is obsolete: true
Attachment #468220 - Flags: review?(bzbarsky)
Attachment #461798 - Flags: review?(bzbarsky)
(Assignee)

Comment 24

9 years ago
Attachment #461800 - Attachment is obsolete: true
Attachment #468221 - Flags: review?(bzbarsky)
Attachment #461800 - Flags: review?(bzbarsky)
(Assignee)

Comment 25

9 years ago
Attachment #461803 - Attachment is obsolete: true
Attachment #468222 - Flags: review?(bzbarsky)
Attachment #461803 - Flags: review?(bzbarsky)
(Assignee)

Comment 26

9 years ago
Attachment #461804 - Attachment is obsolete: true
Attachment #461804 - Flags: review?(bzbarsky)
Attachment #468223 - Flags: review?(bzbarsky)
(Assignee)

Comment 27

9 years ago
Attachment #461805 - Attachment is obsolete: true
Attachment #468224 - Flags: review?(bzbarsky)
Attachment #461805 - Flags: review?(bzbarsky)
(Assignee)

Comment 28

9 years ago
Attachment #461807 - Attachment is obsolete: true
Attachment #468225 - Flags: review?(bzbarsky)
Attachment #461807 - Flags: review?(bzbarsky)
(Assignee)

Comment 29

8 years ago
Attachment #468220 - Attachment is obsolete: true
Attachment #518272 - Flags: review?(bzbarsky)
Attachment #468220 - Flags: review?(bzbarsky)
(Assignee)

Comment 30

8 years ago
Attachment #468221 - Attachment is obsolete: true
Attachment #518273 - Flags: review?(bzbarsky)
Attachment #468221 - Flags: review?(bzbarsky)
(Assignee)

Comment 31

8 years ago
Attachment #468222 - Attachment is obsolete: true
Attachment #518274 - Flags: review?(bzbarsky)
Attachment #468222 - Flags: review?(bzbarsky)
(Assignee)

Comment 32

8 years ago
Attachment #468223 - Attachment is obsolete: true
Attachment #518275 - Flags: review?(bzbarsky)
Attachment #468223 - Flags: review?(bzbarsky)
(Assignee)

Comment 33

8 years ago
Attachment #468224 - Attachment is obsolete: true
Attachment #518276 - Flags: review?(bzbarsky)
Attachment #468224 - Flags: review?(bzbarsky)
(Assignee)

Comment 34

8 years ago
Attachment #468225 - Attachment is obsolete: true
Attachment #518277 - Flags: review?(bzbarsky)
Attachment #468225 - Flags: review?(bzbarsky)
Comment on attachment 518272 [details] [diff] [review]
Part 1: Rename CSSStyleRuleImpl updated to use namespace

r=me
Attachment #518272 - Flags: review?(bzbarsky) → review+
Comment on attachment 518273 [details] [diff] [review]
Part 2: rename DOMCSSStyleRuleImpl updated to use namespace.

r=me
Attachment #518273 - Flags: review?(bzbarsky) → review+
Comment on attachment 518274 [details] [diff] [review]
Part 3: Rename CSSImportantRule updated to use namespace

r=me
Attachment #518274 - Flags: review?(bzbarsky) → review+
Comment on attachment 518276 [details] [diff] [review]
Part 5: Remove virtual from methods that came from nsICSSStyleRule

r=me
Attachment #518276 - Flags: review?(bzbarsky) → review+
Comment on attachment 518277 [details] [diff] [review]
Part 6: DeCOMtaminate StyleRule method signatures

r=me
Attachment #518277 - Flags: review?(bzbarsky) → review+
Comment on attachment 518275 [details] [diff] [review]
Part 4: Remove nsICSSStyleRule and move StyleRule to header file.

r=me
Attachment #518275 - Flags: review?(bzbarsky) → review+
Duplicate of this bug: 604596
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.