Bug 3825

Summary: GW: need a KWQTimer with QTimer(parent) support
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: New BugsAssignee: Eric Seidel (no email) <eric>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P4    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Bug Depends on:    
Bug Blocks: 3250    
Attachments:
Description Flags
adds QTimer(parent) support mjs: review+

Description Eric Seidel (no email) 2005-07-02 20:30:02 PDT
This is a temporary solution to allow compiling.  Any code which actually depends on this method will log 
to the console.  Before shipping code depending on QTimer(parent) we would need to actually make these 
timers destruct when their parent objects do.
Comment 1 Eric Seidel (no email) 2005-07-02 20:31:57 PDT
Created attachment 2764 [details]
adds QTimer(parent) support
Comment 2 Maciej Stachowiak 2005-07-03 03:16:37 PDT
Comment on attachment 2764 [details]
adds QTimer(parent) support

r=me

*but*, the side effect could be a lot worse than a leak if this is really used
- it could lead to incorrect behavior or a crash if the timer fires after it's
expected to be gone.
Comment 3 Eric Seidel (no email) 2005-07-03 16:30:49 PDT
landed with better log message:

if (parent) LOG(NotYetImplemented, "Parent pointer ignored.  QTimer will be 
leaked and may fire after parent dealloc causing crash.");