Open Bug 1290158 Opened 3 years ago Updated Last year

Don't generate synthetic paint events for invisible documents

Categories

(Core :: Layout, defect)

Unspecified
Windows
defect
Not set

Tracking

()

REOPENED

People

(Reporter: neil, Assigned: neil)

References

Details

Attachments

(1 file)

There's some code in the pres shell to generate paint events for invisible documents. In particular, I accidentally triggered it by putting a browser element inside an arrow panel. It then becomes possible for the paint event to be generated before the window has been shown. This confuses any code that assumes that only visible elements can be painted.
Well, why is that code there?  Why is it safe to remove?
Flags: needinfo?(neil)
Comment on attachment 8775625 [details] [diff] [review]
Proposed patch

Unsetting review request pending an answer to my question, ideally in the form of a commit message.
Attachment #8775625 - Flags: review?(bzbarsky)
(In reply to Boris Zbarsky from comment #2)
> Well, why is that code there?
It's there because Matt added it. He told me that he can't remember why but it was probably due to a test failure.

> Why is it safe to remove?
Because the tests don't seem to need it.
Flags: needinfo?(neil)
OK, then presumably Matt or whoever reviewed the addition should review this change, not me...
Attachment #8775625 - Flags: review?(matt.woodrow)
Attachment #8775625 - Flags: review?(matt.woodrow) → review+
Assigning to neil. I suppose the patch should be checked in?
Assignee: nobody → neil
Pushed by neil@parkwaycc.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/472858f465b0
Don't generate synthetic paint events for invisible documents r=mattwoodrow
https://hg.mozilla.org/mozilla-central/rev/472858f465b0
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
backed out for perf regression:
https://hg.mozilla.org/integration/mozilla-inbound/rev/589d63647e0b
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: mozilla51 → ---
(In reply to Joel Maher from comment #9)
> backed out for perf regression:
> https://hg.mozilla.org/integration/mozilla-inbound/rev/589d63647e0b

How can a patch consisting of 100% code removal cause a perf regression?
Flags: needinfo?(jmaher)
Depends on: 1321422
Product: Core → Core Graveyard
Component: Layout: Misc Code → Layout
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.