Closed Bug 1223215 Opened 7 years ago Closed 7 years ago

Remove ToSize() and ThebesSize()

Categories

(Core :: Graphics, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: n.nethercote, Assigned: n.nethercote)

References

(Blocks 1 open bug)

Details

Attachments

(5 files)

More trimming of gfx2DGlue.h.
This lets us remove some Thebes{Point,Size}() and To{Point,Size}() calls.
Attachment #8685165 - Flags: review?(roc)
I'm not totally sure about this one, because there are still plenty of gfxSize
uses left in the codebase and it might be necessary to have these converters
for more Moz2Dification in the future...
Attachment #8685181 - Flags: review?(jmuizelaar)
Attachment #8685179 - Flags: review?(jmuizelaar) → review+
Attachment #8685181 - Flags: review?(jmuizelaar) → review+
Attachment #8685176 - Flags: review?(jwatt) → review+
Attachment #8685265 - Flags: review?(jmuizelaar)
Attachment #8685265 - Flags: review?(jmuizelaar) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/55cea35b3cc657886368c1b22f11cf70def6ba0c
Backout ea759b57d58a (bug 1223215 part 5) because the removed functions are still useful. r=me.
> Backout ea759b57d58a (bug 1223215 part 5) because the removed functions are
> still useful. r=me.

I reverted part 5 because I ended up needing ToSize() and ThebesSize() in some subsequent Moz2Dification work.
You need to log in before you can comment on or make changes to this bug.