Closed
Bug 467295
Opened 14 years ago
Closed 14 years ago
Plugin timer is re-created everytime the page is scrolled
Categories
(Core Graveyard :: Plug-ins, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: smaug, Assigned: smaug)
References
Details
Attachments
(1 file, 1 obsolete file)
3.18 KB,
patch
|
Details | Diff | Splinter Review |
Something like this could be done to keep the current behavior. I have also a WIP patch to make all the plugins to use only 2 timers - one for normal and one for hidden plugins. But I guess with Compositor we could get rid of all the plugin timers.
Comment 1•14 years ago
|
||
Comment on attachment 350696 [details] [diff] [review] possible patch >diff --git a/layout/generic/nsObjectFrame.cpp b/layout/generic/nsObjectFrame.cpp >--- a/layout/generic/nsObjectFrame.cpp >+++ b/layout/generic/nsObjectFrame.cpp >@@ -482,16 +482,17 @@ private: > PRUint32 mLastEventloopNestingLevel; > PRPackedBool mContentFocused; > PRPackedBool mWidgetVisible; // used on Mac to store our widget's visible state > PRPackedBool mPluginPortChanged; > > // If true, destroy the widget on destruction. Used when plugin stop > // is being delayed to a safer point in time. > PRPackedBool mDestroyWidget; >+ PRPackedBool mTimerCancelled; Nit: mTimerCanceled. US spelling is shorter and (as pronunciation guide: stress moves to vowel before doubled consonant, but everyone pronounces "CANceled"). Who should review this? /be
Comment 2•14 years ago
|
||
I meant to write "US spelling is shorter and (...) in this case superior". Or "superiour", I won't argue that one ;-). /be
Attachment #350696 -
Flags: review?(joshmoz)
Comment on attachment 350696 [details] [diff] [review] possible patch r+ with brendan's nit addressed
Attachment #350696 -
Flags: review?(joshmoz) → review+
Assignee | ||
Updated•14 years ago
|
Attachment #350696 -
Flags: superreview?(roc)
(In reply to comment #2) > I meant to write "US spelling is shorter and (...) in this case superior". Or > "superiour", I won't argue that one ;-). I'm not sure why, but it's "superior" for us too.
Attachment #350696 -
Flags: superreview?(roc) → superreview+
Assignee | ||
Comment 5•14 years ago
|
||
Attachment #350696 -
Attachment is obsolete: true
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → Olli.Pettay
Assignee | ||
Updated•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•2 months ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•