Fix Mac window frame coordinate conversion on multiple monitor setups

Fix Mac window frame coordinate conversion on multiple monitor setups

The same bug exists in our event coordinate conversion code in WebInputEventFactory, so this is half of a two-sided patch; WebKit side is:
https://bugs.webkit.org/show_bug.cgi?id=35950

BUG=36621
TEST=In conjuction with the WebKit patch, mouseovers in Carbon plugins will work when the plugin is on a non-primary monitor with a different height than the primary monitor.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=41516

[Avi (use Gerrit), 2010-03-09 23:13:22]

LG

Since you comment that we need it in another place, I take it you did the search
to see where else I screwed up :)

And hey, I did say it was gonna be wrong for a multi-monitor setup.

http://codereview.chromium.org/751002/diff/1/2
File chrome/browser/renderer_host/render_widget_host_view_mac.mm (right):

http://codereview.chromium.org/751002/diff/1/2#newcode602
chrome/browser/renderer_host/render_widget_host_view_mac.mm:602: if ([[NSScreen
screens] count] > 0) {
Now that's paranoia.

[stuartmorgan, 2010-03-09 23:23:35]

http://codereview.chromium.org/751002/diff/1/2
File chrome/browser/renderer_host/render_widget_host_view_mac.mm (right):

http://codereview.chromium.org/751002/diff/1/2#newcode602
chrome/browser/renderer_host/render_widget_host_view_mac.mm:602: if ([[NSScreen
screens] count] > 0) {
On 2010/03/09 23:13:22, Avi wrote:
> Now that's paranoia.

Not really; the docs specifically call out the need to check for this case. I
imagine it's for headless configurations.

[stuartmorgan, 2010-03-10 15:36:40]

Updated to fix the instance in the automation provider as well.

[Avi (use Gerrit), 2010-03-10 15:50:54]

SLGTM