Bug 5686

Summary: make Shared::deref faster by eliminating multiple branches
Product: WebKit Reporter: Darin Adler <darin>
Component: WebKit Misc.Assignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Enhancement    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
See Also: https://bugs.webkit.org/show_bug.cgi?id=195661
Attachments:
Description Flags
get rid of the double if statement in deref without changing semantics eric: review+

Description Darin Adler 2005-11-09 08:42:45 PST
 
Comment 1 Darin Adler 2005-11-09 08:43:21 PST
Created attachment 4649 [details]
get rid of the double if statement in deref without changing semantics
Comment 2 Eric Seidel (no email) 2005-11-09 12:15:36 PST
Comment on attachment 4649 [details]
get rid of the double if statement in deref without changing semantics

The ref changes look fine.  Changing RenderWidget not to be a subclass of
Shared seems a bit unnecessary (all the Shared functions are inline to begin
with), but it's not a big deal either.