Bug 118515

Summary: 8 bytes memory leaks in WebProcessMainQt.cpp when proxy is initialized
Product: WebKit Reporter: Minju Kim <pmuarteo>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abecsi, cdumez, cmarcelo, commit-queue, gyuyoung.kim, menard, pmuarteo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Description Minju Kim 2013-07-09 10:56:50 PDT 
8 bytes memory leaks in WebProcessMainQt.cpp when proxy is initialized
Comment 1 Minju Kim 2013-07-09 11:06:46 PDT 
Created attachment 206344 [details]
Patch
Comment 2 WebKit Commit Bot 2013-07-09 11:08:07 PDT 
Attachment 206344 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit2/WebProcess/qt/WebProcessMainQt.cpp']" exit_code: 1
Source/WebKit2/ChangeLog:10:  Line contains tab character.  [whitespace/tab] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Minju Kim 2013-07-09 11:13:02 PDT 
Created attachment 206345 [details]
Patch
Comment 4 Chris Dumez 2013-07-09 11:14:36 PDT 
Comment on attachment 206345 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=206345&action=review

> Source/WebKit2/WebProcess/qt/WebProcessMainQt.cpp:140
> +                delete proxyFactory;

You might want to use OwnPtr and QNetworkProxyFactory::setApplicationProxyFactory(proxyFactory.leakPtr()); to avoid manual memory handling.
Comment 5 Minju Kim 2013-07-09 12:07:59 PDT 
Created attachment 206346 [details]
Patch
Comment 6 Chris Dumez 2013-07-09 12:36:44 PDT 
Comment on attachment 206346 [details]
Patch

Looks good, r=me.
Comment 7 Minju Kim 2013-07-09 14:38:41 PDT 
Thanks (In reply to comment #6)
> (From update of attachment 206346 [details])
> Looks good, r=me.
Comment 8 Minju Kim 2013-09-25 22:56:33 PDT 
What else can I do to make it landing? Is there anyone to help me about it?
Comment 9 Gyuyoung Kim 2013-09-25 23:05:33 PDT 
Comment on attachment 206346 [details]
Patch

Though I'm not a QT port developer, this patch looks good.
Comment 10 Gyuyoung Kim 2013-09-25 23:08:18 PDT 
(In reply to comment #8)
> What else can I do to make it landing? Is there anyone to help me about it?

Do not change bug status with "Resolved Fixed" manually. commit bot will change the status after landing.

See also https://bugs.webkit.org/page.cgi?id=fields.html#status
Comment 11 WebKit Commit Bot 2013-09-25 23:32:04 PDT 
Comment on attachment 206346 [details]
Patch

Clearing flags on attachment: 206346

Committed r156449: <http://trac.webkit.org/changeset/156449>
Comment 12 WebKit Commit Bot 2013-09-25 23:32:07 PDT 
All reviewed patches have been landed.  Closing bug.