Bug 50915

Summary: [GTK] Do not use GET_PRIVATE macros
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, commit-queue, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch to remove GET_PRIVATE macros none

Description Carlos Garcia Campos 2010-12-13 04:39:55 PST
GET_PRIVATE macros are slow and should only be used in the object init() method. We are currently using a pointer to the private structure in the instance structure so we don't need to call GET_PRIVATE. There are only a few places where GET_PRIVATE macros are used, but since they are confusing and useless (it's macro for something that it's going to be used only once), I think we can just remove them all to make sure they are not used anymore.
Comment 1 Carlos Garcia Campos 2010-12-13 04:42:39 PST
Created attachment 76370 [details]
Patch to remove GET_PRIVATE macros
Comment 2 WebKit Commit Bot 2010-12-13 05:27:37 PST
Comment on attachment 76370 [details]
Patch to remove GET_PRIVATE macros

Clearing flags on attachment: 76370

Committed r73906: <http://trac.webkit.org/changeset/73906>
Comment 3 WebKit Commit Bot 2010-12-13 05:27:41 PST
All reviewed patches have been landed.  Closing bug.
Comment 4 WebKit Review Bot 2010-12-13 06:17:40 PST
http://trac.webkit.org/changeset/73906 might have broken GTK Linux 64-bit Debug
The following tests are not passing:
editing/selection/extend-by-character-002.html