Summary: | [ATK] Leak: more leaks in AccessibilityUIElement | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brian Holt <brian.holt> | ||||||
Component: | Accessibility | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, eflews.bot, gtk-ews, gyuyoung.kim, xan.lopez | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 116317 | ||||||||
Attachments: |
|
Description
Brian Holt
2013-07-03 10:15:26 PDT
Created attachment 206012 [details]
Patch
Comment on attachment 206012 [details] Patch Attachment 206012 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/1015764 The title one is a bit trickier because fun:g_strdup fun:_ZL27webkitAccessibleTextGetTextP8_AtkTextii fun:_ZL23webkitAccessibleGetNameP10_AtkObject fun:_ZN22AccessibilityUIElement5titleEv webkitAccessibleGetName(AtkObject*) sometimes returns a char* that is created by g_strdup but it also sometimes returns a value stored in a cache. Freeing that would be a disaster. My feeling at this stage is that the effort required to refactor is not worth the few bytes leaking. Comment on attachment 206012 [details] Patch Attachment 206012 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/1012992 Comment on attachment 206012 [details] Patch Attachment 206012 [details] did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/1022262 Comment on attachment 206012 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=206012&action=review > Tools/DumpRenderTree/atk/AccessibilityUIElementAtk.cpp:439 > + GOwnPtr<char> axValue(g_strdup_printf("AXLanguage: %s", locale)) missing ; > Tools/WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:625 > + GOwnPtr<char> axValue(g_strdup_printf("AXLanguage: %s", locale)) ditto. Created attachment 206013 [details]
Patch
Comment on attachment 206013 [details]
Patch
Ok, r=me. Do you want me to commit? You did not set the cq flag.
(In reply to comment #8) > (From update of attachment 206013 [details]) > Ok, r=me. Do you want me to commit? You did not set the cq flag. Yes please, I forgot to do that with webkit-patch. Comment on attachment 206013 [details] Patch Clearing flags on attachment: 206013 Committed r152371: <http://trac.webkit.org/changeset/152371> All reviewed patches have been landed. Closing bug. |