The default bug view has changed. See this FAQ.

Port |Bug 555839 - Turn on OpenGL LayerManager for fullscreen video widget| to SeaMonkey

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
General
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Tracking

(Depends on: 1 bug)

Trunk
seamonkey2.1b2
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Created attachment 436498 [details] [diff] [review]
Turn on acceleration for fullscreen video [Checkin: comment 4]

We have our own copy of fullscreen-video.xhtml, so this bug is to not forget to sync it with the browser (FF) version.

I don't know if the back-end is already available for SM or if it would work on my WinXP but given bug 555839 comment 11 it's too early to try anyway. That's also why I'm not requesting review or assign it to me yet.

Updated

7 years ago
Depends on: 556540

Comment 1

7 years ago
Although bug 556540 is still open, there haven't been any comments there for 6 months, and many improvements have been made to hardware acceleration since then.  Could this be looked at for inclusion in 2.1 so that we are on par with Firefox in this area?
status-seamonkey2.1: --- → ?
(Assignee)

Comment 2

7 years ago
Comment on attachment 436498 [details] [diff] [review]
Turn on acceleration for fullscreen video [Checkin: comment 4]

Unfortunately I don't know how to test this. My CPU usage never goes above 4% even with the below test case full screen (= 1680x1050 for me), and that is without the patch applied! I have Win7 (Aero active) and a recent DX11 graphics card. Thus, unless one of the guys over at bug 555839 knows better, my only argument is consistency.

http://videos.mozilla.org/serv/marketing/sfx_edutoolkit/asa_historymozilla.ogv
Attachment #436498 - Attachment description: Turn on acceleration for fullscreen video → Turn on acceleration for fullscreen video
Attachment #436498 - Flags: review?(neil)

Comment 3

7 years ago
Comment on attachment 436498 [details] [diff] [review]
Turn on acceleration for fullscreen video [Checkin: comment 4]

I don't even have hardware on which I can conceive of testing this :-(
Attachment #436498 - Flags: review?(neil) → review+
(Assignee)

Comment 4

7 years ago
Comment on attachment 436498 [details] [diff] [review]
Turn on acceleration for fullscreen video [Checkin: comment 4]

http://hg.mozilla.org/comm-central/rev/5773513ab09b
Attachment #436498 - Attachment description: Turn on acceleration for fullscreen video → Turn on acceleration for fullscreen video [Checkin: comment 4]
(Assignee)

Updated

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
status-seamonkey2.1: ? → ---
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2
Assignee: nobody → jh
Depends on: 555839
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.