653453 - Full screen video plays back on wrong monitor (multiple monitor setup) [Port Bug 520805]

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Philip Chee]

Bug 520805 Comment 0:

> with a secondary monitor (desktop sharing mode), drag your browser to the 2nd
> screen, then go to FS.  you'll notice the FS occurs on the first monitor, while
> the browser video is paused.   In flash, FS applies to the screen that has
> focus.
> 
> Repro:
> 1) install Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US;
> rv:1.9.2b1pre) Gecko/20091006 Namoroka/3.6b1pre or trunk
> 2) Attach a secondary monitor, enabling desktop sharing mode
> 3) Drag the browser to the secondary monitor
> 4) Play an OGG video in Full screen  (use URL example)
> 5) Verify OGG video plays in FS in the primary monitor, while the secondary
> monitor pauses the video in browser

Comment 1

[Philip Chee]

Attachment: Patch v1.0 fix.

> Bug 520805 Full screen video plays back on wrong monitor (multiple monitor setup).
Port the front end bit.

> Bug 556482 Remove dead onDestroy method.
Remove useless method. I don't know why KaiRo ported this as he also copied the comment which said that this didn't do anything.

> Bug 425575 Huge context menu when holding right-click on a page, dragging mouse and letting go anywhere out of content area.
I see this with SeaMonkey 2.0 but I can't reproduce this on 2.1b3. I guess this was fixed in the backend.
Skipping.

Comment 2

[[e-mail]]

Comment on attachment 528875 [details] [diff] [review]
Patch v1.0 fix.

>     openDialog("chrome://communicator/content/fullscreen-video.xhtml",
>-               "", "chrome,dialog=no", this.target, isPaused);
>+               "", "chrome,centerscreen,dialog=no", this.target, isPaused);
Bah, what a lame hack...

Comment 3

[Philip Chee]

Pushed:
http://hg.mozilla.org/releases/comm-2.0/rev/0be2ed5d3575
http://hg.mozilla.org/comm-central/rev/e53b93cb1f83