Closed Bug 786978 Opened 12 years ago Closed 12 years ago

Disable refresh driver painting for firefox 17

Categories

(Core :: Web Painting, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17
Tracking Status
firefox17 + fixed

People

(Reporter: mattwoodrow, Assigned: mattwoodrow)

References

Details

Attachments

(1 file)

Attached patch DisableSplinter Review
Try push: https://tbpl.mozilla.org/?tree=Try&rev=ace3aa44a736

Looks like android isn't overly happy, will look into it.
Depends on: 787300
Comment on attachment 656795 [details] [diff] [review]
Disable

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Bug 539359
User impact if declined: Tp5 regressions.
Testing completed (on m-c, etc.): Only tryserver, which doesn't work for android and windows.
Risk to taking this patch (and alternatives if risky): Should be low risk, just reverting to existing behaviour. Alternatives are to not land this and take the perf regression instead.
String or UUID changes made by this patch: None.
Attachment #656795 - Flags: approval-mozilla-aurora?
(In reply to Matt Woodrow (:mattwoodrow) from comment #1)
> Comment on attachment 656795 [details] [diff] [review]
> Disable
> 
> [Approval Request Comment]
> Bug caused by (feature/regressing bug #): Bug 539359
> User impact if declined: Tp5 regressions.
> Testing completed (on m-c, etc.): Only tryserver, which doesn't work for
> android and windows.
> Risk to taking this patch (and alternatives if risky): Should be low risk,
> just reverting to existing behaviour. Alternatives are to not land this and
> take the perf regression instead.
> String or UUID changes made by this patch: None.

This needs an r+, especially if it's only landing on m-a.
Attachment #656795 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Blocks: 797167
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee: nobody → matt.woodrow
Target Milestone: --- → mozilla17
Component: Layout: View Rendering → Layout: Web Painting
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: