Bug 77816 - WKPreferences instances cannot be copied
Summary: WKPreferences instances cannot be copied
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: mitz
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2012-02-04 15:53 PST by mitz
Modified: 2012-02-06 20:48 PST (History)
4 users (show)

See Also:


Attachments
Add WKPreferencesCreateCopy() (5.52 KB, patch)
2012-02-04 16:11 PST, mitz
sam: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2012-02-04 15:53:21 PST
WKPreferences instances cannot be copied
Comment 1 mitz 2012-02-04 15:59:40 PST
<rdar://problem/10772406>
Comment 2 mitz 2012-02-04 16:11:58 PST
Created attachment 125504 [details]
Add WKPreferencesCreateCopy()
Comment 3 mitz 2012-02-04 16:18:46 PST
Fixed in <http://trac.webkit.org/r106749>.
Comment 4 Ryosuke Niwa 2012-02-06 11:31:55 PST
In GCC, I'm getting the following compilation error:

cc1plus: warnings being treated as errors
/Volumes/Data/webkit4/Source/WebKit2/UIProcess/WebPreferences.cpp: In copy constructor 'WebKit::WebPreferences::WebPreferences(const WebKit::WebPreferences&)':
/Volumes/Data/webkit4/Source/WebKit2/UIProcess/WebPreferences.cpp:44: warning: base class 'class WebKit::APIObject' should be explicitly initialized in the copy constructor
PhaseScriptExecution "Check For Framework Include Consistency" /Volumes/Data/webkit4/WebKitBuild/WebKit2.build/Debug/WebKit2.build/Script-5DF408C6131DD49700130071.sh
    cd /Volumes/Data/webkit4/Source/WebKit2
Comment 5 Ryosuke Niwa 2012-02-06 11:37:35 PST
"Fixed" in http://trac.webkit.org/changeset/106824.
Comment 6 Ryosuke Niwa 2012-02-06 11:40:28 PST
I'm not happy about using the default constructor of APIObject but I couldn't figure out a good way to fix it properly without being invasive here.

Maybe andersca, bdash, or darin would know?
Comment 7 Darin Adler 2012-02-06 15:57:03 PST
I think your fix was correct.
Comment 8 Ryosuke Niwa 2012-02-06 20:48:30 PST
(In reply to comment #7)
> I think your fix was correct.

That is good to hear.