The default bug view has changed. See this FAQ.

Intergrated gpu no longer used on Nightly

RESOLVED FIXED in Firefox 11

Status

()

Core
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jrmuizel, Assigned: jrmuizel)

Tracking

({regression})

unspecified
mozilla12
x86
Mac OS X
regression
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox11+ fixed)

Details

(Whiteboard: [qa?])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Looks like it was caused by 696376
Jeff, this needs to sort of block Fx11, right?
Blocks: 696376
tracking-firefox11: --- → ?
Keywords: regression
Jeff, was this caused by
https://hg.mozilla.org/mozilla-central/rev/c9a74f4ee1f7

That is really strange, but most things about 696376 are. Can you confirm that that was the patch that broke it?
(Assignee)

Comment 3

5 years ago
Created attachment 582874 [details] [diff] [review]
Do TriggerQuirks earlier

This moves the copy of NS_CompareVersions into the header to make things work.
(Assignee)

Updated

5 years ago
Attachment #582874 - Flags: review?(bgirard)
Comment on attachment 582874 [details] [diff] [review]
Do TriggerQuirks earlier

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

Thanks!
Attachment #582874 - Flags: review?(bgirard) → review+

Comment 5

5 years ago
Please land on m-c and nominate for Aurora 11 approval. Thanks!
tracking-firefox11: ? → +
https://hg.mozilla.org/mozilla-central/rev/2325e88b6026
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
(Assignee)

Comment 7

5 years ago
Comment on attachment 582874 [details] [diff] [review]
Do TriggerQuirks earlier

[Approval Request Comment]
Regression caused by (bug #): 696376
User impact if declined: We end up always using the discrete GPU (i.e. worse battery life)
Testing completed (on m-c, etc.): none yet
Risk to taking this patch (and alternatives if risky): Low, this just calls an independent function earlier.
Attachment #582874 - Flags: approval-mozilla-aurora?

Updated

5 years ago
Blocks: 715802

Comment 8

5 years ago
Comment on attachment 582874 [details] [diff] [review]
Do TriggerQuirks earlier

[Triage Comment]
Approved for Aurora given where we are in the cycle, the risk assessment of this code change, and the impact to a user's battery life.
Attachment #582874 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 9

5 years ago
This actually made it into Aurora but I forgot to include a link:
https://hg.mozilla.org/mozilla-beta/rev/2325e88b6026

Updated

5 years ago
Assignee: nobody → jmuizelaar
status-firefox11: --- → fixed
Is there anything QA can do to verify this bug?
Whiteboard: [qa?]
You need to log in before you can comment on or make changes to this bug.