RESOLVED FIXED Bug 45893
Add the namespace in a friend declaration to get MSVC recognize it properly.
https://bugs.webkit.org/show_bug.cgi?id=45893
Summary Add the namespace in a friend declaration to get MSVC recognize it properly.
Satish Sampath
Reported 2010-09-16 08:26:07 PDT
This is a fix for a build break caused by r67614 in the chromium windows builders (which use VS 2008). Looks like without the proper namespace, the friend declaration was treated as a forward declaration and the build failed with the following error message: 1>JavaScriptCore\wtf/RefCounted.h(139) : error C2248: 'WebCore::FileWriter::~FileWriter' : cannot access private member declared in class 'WebCore::FileWriter' 1> webcore\fileapi\FileWriter.h(124) : compiler has generated 'WebCore::FileWriter::~FileWriter' here 1> webcore\fileapi\FileWriter.h(52) : see declaration of 'WebCore::FileWriter' 1> JavaScriptCore\wtf/RefCounted.h(137) : while compiling class template member function 'void WTF::RefCounted<T>::deref(void)' 1> with 1> [ 1> T=WebCore::FileWriter 1> ] 1> webcore\fileapi\FileWriter.h(52) : see reference to class template instantiation 'WTF::RefCounted<T>' being compiled 1> with 1> [ 1> T=WebCore::FileWriter 1> ]
Attachments
Patch (1.02 KB, patch)
2010-09-16 08:28 PDT, Satish Sampath
no flags
Satish Sampath
Comment 1 2010-09-16 08:28:24 PDT
Jeremy Orlow
Comment 2 2010-09-16 08:33:01 PDT
Comment on attachment 67800 [details] Patch r=me
Satish Sampath
Comment 3 2010-09-16 08:40:27 PDT
Comment on attachment 67800 [details] Patch Clearing flags on attachment: 67800 Committed r67622: <http://trac.webkit.org/changeset/67622>
Note You need to log in before you can comment on or make changes to this bug.