1132854 - Make nsIntSize a typedef of gfx::IntSize

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Nicolas Silva [:nical]]

(Filed under gfx by default but it touches a lot of places)

There are still a lot of places where we use the old thebes size/point/rect/etc classes and it's a pain to have to have to convert between them and the cool new ones in Moz2D.
So far we have taken the approach of "moz2Difying" the different places where we use these classes, but it's a tedious process.

I propose that we just completely remove nsIntSize's code and make it a typedef of gfx::IntSize. Starting with these two because they're probably the most commonly used, but I'd like to do this with all of the Int classes since they are completely equivalent (the floating point classes are trickier because we go from double to float so we'll have to take the time to do it more incrementally).

Comment 1

[Nicolas Silva [:nical]]

Attachment: Make nsIntSize a typedef of gfx::IntSize

Makes for a big ugly patch, but it was actually very mechanical and easy to write.

Most of the modifications come from that we can't forward declare nsIntSize the way we used to now that it's a typedef.

Comment 2

[Nicolas Silva [:nical]]

Attachment: remove the ThebesIntSize conversion helper since it's now a no-op.

Comment 3

[Bas Schouten (:bas.schouten)]

Comment on attachment 8564035 [details] [diff] [review]
Make nsIntSize a typedef of gfx::IntSize

Review of attachment 8564035 [details] [diff] [review]:
-----------------------------------------------------------------

::: xpcom/glue/CondVar.h
@@ +68,5 @@
>    /**
>     * Wait
>     * @see prcvar.h
>     **/
> +  nsresult Wait(PRIntervalTime aInterval = PR_INTERVAL_NO_TfIMEOUT)

I bet this won't compile :P

Comment 4

[Nicolas Silva [:nical]]

(In reply to Bas Schouten (:bas.schouten) from comment #3)
> Comment on attachment 8564035 [details] [diff] [review]
> Make nsIntSize a typedef of gfx::IntSize
> 
> Review of attachment 8564035 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: xpcom/glue/CondVar.h
> @@ +68,5 @@
> >    /**
> >     * Wait
> >     * @see prcvar.h
> >     **/
> > +  nsresult Wait(PRIntervalTime aInterval = PR_INTERVAL_NO_TfIMEOUT)
> 
> I bet this won't compile :P

Yeah, I apparently made slipped on the keyboard just before uploading the patch, but I had it compile on linux. The patch is probably bitrotted by now so I'll have to redo it, push to try and land quickly before it bitrots again if you agree with the general idea.

Comment 5

[Nicolas Silva [:nical]]

Attachment: Make nsIntSize a typedef of gfx::IntSize (rebased, carrying r=roc)

Comment 6

[Nicolas Silva [:nical]]

Attachment: Remove the ToIntSize conversion helper since it's now a no-op

Comment 7

[Nicolas Silva [:nical]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/bc9e3d4db725
https://hg.mozilla.org/integration/mozilla-inbound/rev/1d5bdeca48eb

Comment 8

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/bc9e3d4db725
https://hg.mozilla.org/mozilla-central/rev/1d5bdeca48eb

Comment 9

[Bas Schouten (:bas.schouten)]

Comment on attachment 8584651 [details] [diff] [review]
Remove the ToIntSize conversion helper since it's now a no-op

Review of attachment 8584651 [details] [diff] [review]:
-----------------------------------------------------------------

This is satisfying.

Comment 10

[Nicolas Silva [:nical]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1172f2f1daab

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/1172f2f1daab