292958 - Blink timers not disabled in cached presentations

Status: RESOLVED INVALID

(Core :: DOM: Navigation, defect)

Description

[Boris Zbarsky [:bzbarsky]]

bfcache doesn't disable blink timers.  I'm not sure how much of a problem this
is, but could we have issues with the cached viewmanager trying to paint to the
same device context as the currently-showing viewmanager?  I'm not really too
clear on the device context thing and how it works...

So if this is not an issue, feel free to mark invalid or something.

Comment 1

[Brian Ryner (not reading)]

I'm not clear on what you mean by this -- PresShell::Freeze() calls
nsCaret::SetCaretVisible(PR_FALSE), which calls StopBlinking(), which calls
KillTimer().

Comment 2

[Boris Zbarsky [:bzbarsky]]

I mean the text-decoration:blink timer.  See
http://lxr.mozilla.org/seamonkey/source/layout/generic/nsTextFrame.cpp#261 etc.

Comment 3

[Brian Ryner (not reading)]

Painting is totally suppressed on cached pages, so I don't think this can cause
a problem.  Marking INVALID as you suggested.