1239007 - Use strongly-typed rects for GeckoRectToCocoaRect and CocoaRectToGeckoRect in nsCocoaUtils

Status: RESOLVED FIXED

(Core :: Widget: Cocoa, defect)

Description

[Jonathan Kew [:jfkthame]]

As noted in bug 890156 comments 153 and 154, we can replace nsIntRect by correctly-typed rects here to make the code more self-documenting and less error-prone.

Comment 1

[Jonathan Kew [:jfkthame]]

Attachment: Replace nsIntRect by strongly-typed rects in a couple of nsCocoaUtils functions

This is pretty straightforward; should not change any behavior.

Comment 2

[Markus Stange [:mstange]]

Comment on attachment 8706983 [details] [diff] [review]
Replace nsIntRect by strongly-typed rects in a couple of nsCocoaUtils functions

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

Great cleanup!

::: widget/cocoa/nsCocoaUtils.h
@@ +220,5 @@
>    // (NSRect) contain an origin (x,y) in a coordinate system with (0,0)
>    // in the bottom-left of the primary screen. Both nsRect and NSRect
>    // contain width/height info, with no difference in their use.
>    // This function does no scaling, so the Gecko coordinates are
> +  // expected to be desktop pixels, which we treat as equal to Cocoa points.

Would it be correct to change this to
"expected to be desktop pixels, which are the same size as Cocoa points by definition." ?

Comment 3

[Jonathan Kew [:jfkthame]]

Indeed - I'll do that, thanks.

Comment 4

[Jonathan Kew [:jfkthame]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a9f9b36c1a2eec7626e6b749e46ab0a8bf3323e2
Bug 1239007 - Replace nsIntRect by strongly-typed rects in a couple of nsCocoaUtils functions. r=mstange

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/a9f9b36c1a2e

Comment 6

[Jonathan Kew [:jfkthame]]

Backed out of aurora-46, see bug 890156 comment 168. Bumping target milestone to 47.