JavaScriptCore:
authorbfulgham@webkit.org <bfulgham@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 15 Jul 2009 05:13:00 +0000 (05:13 +0000)
committerbfulgham@webkit.org <bfulgham@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 15 Jul 2009 05:13:00 +0000 (05:13 +0000)
2009-07-14  Zoltan Horvath  <hzoltan@inf.u-szeged.hu>

        Reviewed by Darin Adler.

        Change all Noncopyable inheriting visibility to public.
        https://bugs.webkit.org/show_bug.cgi?id=27225

        Change all Noncopyable inheriting visibility to public because
        it is needed to the custom allocation framework (bug #20422).

        * bytecode/SamplingTool.h:
        * bytecompiler/RegisterID.h:
        * interpreter/CachedCall.h:
        * interpreter/RegisterFile.h:
        * parser/Lexer.h:
        * parser/Parser.h:
        * runtime/ArgList.h:
        * runtime/BatchedTransitionOptimizer.h:
        * runtime/Collector.h:
        * runtime/CommonIdentifiers.h:
        * runtime/JSCell.h:
        * runtime/JSGlobalObject.h:
        * runtime/JSLock.h:
        * runtime/JSONObject.cpp:
        * runtime/SmallStrings.cpp:
        * runtime/SmallStrings.h:
        * wtf/CrossThreadRefCounted.h:
        * wtf/GOwnPtr.h:
        * wtf/Locker.h:
        * wtf/MessageQueue.h:
        * wtf/OwnArrayPtr.h:
        * wtf/OwnFastMallocPtr.h:
        * wtf/OwnPtr.h:
        * wtf/RefCounted.h:
        * wtf/ThreadSpecific.h:
        * wtf/Threading.h:
        * wtf/Vector.h:
        * wtf/unicode/Collator.h:

WebCore:

2009-07-14  Zoltan Horvath  <hzoltan@inf.u-szeged.hu>

        Reviewed by Darin Adler.

        Change all Noncopyable inheriting visibility to public.
        https://bugs.webkit.org/show_bug.cgi?id=27225

        Change all Noncopyable inheriting visibility to public because
        it is needed to the custom allocation framework (bug #20422).

        * bindings/js/GCController.h:
        * bindings/js/WorkerScriptController.h:
        * bindings/v8/V8DOMMap.cpp:
        (WebCore::):
        * bridge/runtime.h:
        * css/CSSSelector.h:
        * css/CSSSelectorList.h:
        * css/CSSStyleSelector.h:
        * dom/ClassNames.h:
        * dom/MessagePortChannel.h:
        * dom/XMLTokenizerLibxml2.cpp:
        * dom/XMLTokenizerScope.h:
        * editing/ReplaceSelectionCommand.cpp:
        * editing/SelectionController.h:
        * editing/TextIterator.cpp:
        * history/PageCache.h:
        * html/CanvasRenderingContext2D.h:
        * html/HTMLParser.h:
        * html/HTMLParserQuirks.h:
        * html/PreloadScanner.h:
        * loader/Cache.h:
        * loader/CrossOriginPreflightResultCache.h:
        * loader/FrameLoader.h:
        * loader/ProgressTracker.h:
        * loader/ThreadableLoader.h:
        * loader/appcache/ApplicationCacheGroup.h:
        * loader/archive/ArchiveResourceCollection.h:
        * loader/icon/IconDatabase.h:
        * loader/icon/IconLoader.h:
        * loader/icon/PageURLRecord.h:
        * loader/loader.h:
        * page/ContextMenuController.h:
        * page/EventHandler.h:
        * page/FrameTree.h:
        * page/Page.h:
        * page/PageGroup.h:
        * page/PageGroupLoadDeferrer.h:
        * page/mac/EventHandlerMac.mm:
        * platform/AutodrainedPool.h:
        * platform/ContextMenu.h:
        * platform/EventLoop.h:
        * platform/HostWindow.h:
        * platform/Pasteboard.h:
        * platform/PurgeableBuffer.h:
        * platform/RunLoopTimer.h:
        * platform/ThreadGlobalData.h:
        * platform/ThreadTimers.h:
        * platform/Timer.h:
        * platform/TreeShared.h:
        * platform/graphics/FontData.h:
        * platform/graphics/GlyphWidthMap.h:
        * platform/graphics/GraphicsContext.h:
        * platform/graphics/ImageBuffer.h:
        * platform/graphics/ImageSource.h:
        * platform/graphics/MediaPlayer.h:
        * platform/graphics/skia/GraphicsContextPlatformPrivate.h:
        * platform/graphics/skia/PlatformContextSkia.h:
        * platform/graphics/win/QTMovieWin.cpp:
        * platform/mac/LocalCurrentGraphicsContext.h:
        * platform/network/FormDataBuilder.h:
        * platform/network/ResourceHandleInternal.h:
        * platform/network/soup/ResourceHandleSoup.cpp:
        * platform/text/StringBuffer.h:
        * platform/text/TextCodec.h:
        * platform/win/WindowMessageBroadcaster.h:
        * rendering/CounterNode.h:
        * rendering/LayoutState.h:
        * rendering/RenderFrameSet.h:
        * rendering/RenderView.h:
        * rendering/TransformState.h:
        * svg/SVGAnimatedProperty.h:
        * svg/SynchronizableTypeWrapper.h:
        * workers/WorkerMessagingProxy.h:
        * workers/WorkerRunLoop.cpp:
        * xml/XPathExpressionNode.h:
        * xml/XPathParser.h:
        * xml/XPathPredicate.h:
        * xml/XPathStep.h:

WebKit/gtk:

2009-07-14  Zoltan Horvath  <hzoltan@inf.u-szeged.hu>

        Reviewed by Darin Adler.

        Change all Noncopyable inheriting visibility to public.
        https://bugs.webkit.org/show_bug.cgi?id=27225

        Change all Noncopyable inheriting visibility to public because
        it is needed to the custom allocation framework (bug #20422).

        * webkit/webkitdownload.cpp:

WebKit/win:

2009-07-14  Zoltan Horvath  <hzoltan@inf.u-szeged.hu>

        Reviewed by Darin Adler.

        Change all Noncopyable inheriting visibility to public.
        https://bugs.webkit.org/show_bug.cgi?id=27225

        Change all Noncopyable inheriting visibility to public because
        it is needed to the custom allocation framework (bug #20422).

        * COMEnumVariant.h:
        * WebDatabaseManager.cpp:
        * WebInspector.h:
        * WebLocalizableStrings.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@45891 268f45cc-cd09-0410-ab3c-d52691b4dbfc

113 files changed:
JavaScriptCore/ChangeLog
JavaScriptCore/bytecode/SamplingTool.h
JavaScriptCore/bytecompiler/RegisterID.h
JavaScriptCore/interpreter/CachedCall.h
JavaScriptCore/interpreter/RegisterFile.h
JavaScriptCore/parser/Lexer.h
JavaScriptCore/parser/Parser.h
JavaScriptCore/runtime/ArgList.h
JavaScriptCore/runtime/BatchedTransitionOptimizer.h
JavaScriptCore/runtime/Collector.h
JavaScriptCore/runtime/CommonIdentifiers.h
JavaScriptCore/runtime/JSCell.h
JavaScriptCore/runtime/JSGlobalObject.h
JavaScriptCore/runtime/JSLock.h
JavaScriptCore/runtime/JSONObject.cpp
JavaScriptCore/runtime/SmallStrings.cpp
JavaScriptCore/runtime/SmallStrings.h
JavaScriptCore/wtf/CrossThreadRefCounted.h
JavaScriptCore/wtf/GOwnPtr.h
JavaScriptCore/wtf/Locker.h
JavaScriptCore/wtf/MessageQueue.h
JavaScriptCore/wtf/OwnArrayPtr.h
JavaScriptCore/wtf/OwnFastMallocPtr.h
JavaScriptCore/wtf/OwnPtr.h
JavaScriptCore/wtf/ThreadSpecific.h
JavaScriptCore/wtf/Threading.h
JavaScriptCore/wtf/Vector.h
JavaScriptCore/wtf/unicode/Collator.h
WebCore/ChangeLog
WebCore/bindings/js/GCController.h
WebCore/bindings/js/WorkerScriptController.h
WebCore/bindings/v8/V8DOMMap.cpp
WebCore/bridge/runtime.h
WebCore/css/CSSSelector.h
WebCore/css/CSSSelectorList.h
WebCore/css/CSSStyleSelector.h
WebCore/dom/ClassNames.h
WebCore/dom/MessagePortChannel.h
WebCore/dom/XMLTokenizerLibxml2.cpp
WebCore/dom/XMLTokenizerScope.h
WebCore/editing/ReplaceSelectionCommand.cpp
WebCore/editing/SelectionController.h
WebCore/editing/TextIterator.cpp
WebCore/history/PageCache.h
WebCore/html/CanvasRenderingContext2D.h
WebCore/html/HTMLParser.h
WebCore/html/HTMLParserQuirks.h
WebCore/html/PreloadScanner.h
WebCore/loader/Cache.h
WebCore/loader/CrossOriginPreflightResultCache.h
WebCore/loader/FrameLoader.h
WebCore/loader/ProgressTracker.h
WebCore/loader/ThreadableLoader.h
WebCore/loader/appcache/ApplicationCacheGroup.h
WebCore/loader/archive/ArchiveResourceCollection.h
WebCore/loader/icon/IconDatabase.h
WebCore/loader/icon/IconLoader.h
WebCore/loader/icon/PageURLRecord.h
WebCore/loader/loader.h
WebCore/page/ContextMenuController.h
WebCore/page/EventHandler.h
WebCore/page/FrameTree.h
WebCore/page/Page.h
WebCore/page/PageGroup.h
WebCore/page/PageGroupLoadDeferrer.h
WebCore/page/mac/EventHandlerMac.mm
WebCore/platform/AutodrainedPool.h
WebCore/platform/ContextMenu.h
WebCore/platform/EventLoop.h
WebCore/platform/HostWindow.h
WebCore/platform/Pasteboard.h
WebCore/platform/PurgeableBuffer.h
WebCore/platform/RunLoopTimer.h
WebCore/platform/ThreadGlobalData.h
WebCore/platform/ThreadTimers.h
WebCore/platform/Timer.h
WebCore/platform/TreeShared.h
WebCore/platform/graphics/FontData.h
WebCore/platform/graphics/GlyphWidthMap.h
WebCore/platform/graphics/GraphicsContext.h
WebCore/platform/graphics/ImageBuffer.h
WebCore/platform/graphics/ImageSource.h
WebCore/platform/graphics/MediaPlayer.h
WebCore/platform/graphics/skia/GraphicsContextPlatformPrivate.h
WebCore/platform/graphics/skia/PlatformContextSkia.h
WebCore/platform/graphics/win/QTMovieWin.cpp
WebCore/platform/mac/LocalCurrentGraphicsContext.h
WebCore/platform/network/FormDataBuilder.h
WebCore/platform/network/ResourceHandleInternal.h
WebCore/platform/network/soup/ResourceHandleSoup.cpp
WebCore/platform/text/StringBuffer.h
WebCore/platform/text/TextCodec.h
WebCore/platform/win/WindowMessageBroadcaster.h
WebCore/rendering/CounterNode.h
WebCore/rendering/LayoutState.h
WebCore/rendering/RenderFrameSet.h
WebCore/rendering/RenderView.h
WebCore/rendering/TransformState.h
WebCore/svg/SVGAnimatedProperty.h
WebCore/svg/SynchronizableTypeWrapper.h
WebCore/workers/WorkerMessagingProxy.h
WebCore/workers/WorkerRunLoop.cpp
WebCore/xml/XPathExpressionNode.h
WebCore/xml/XPathParser.h
WebCore/xml/XPathPredicate.h
WebCore/xml/XPathStep.h
WebKit/gtk/ChangeLog
WebKit/gtk/webkit/webkitdownload.cpp
WebKit/win/COMEnumVariant.h
WebKit/win/ChangeLog
WebKit/win/WebDatabaseManager.cpp
WebKit/win/WebInspector.h
WebKit/win/WebLocalizableStrings.cpp

index ab36c2e..41bd3ce 100644 (file)
@@ -2,6 +2,45 @@
 
         Reviewed by Darin Adler.
 
+        Change all Noncopyable inheriting visibility to public.
+        https://bugs.webkit.org/show_bug.cgi?id=27225
+
+        Change all Noncopyable inheriting visibility to public because
+        it is needed to the custom allocation framework (bug #20422).
+
+        * bytecode/SamplingTool.h:
+        * bytecompiler/RegisterID.h:
+        * interpreter/CachedCall.h:
+        * interpreter/RegisterFile.h:
+        * parser/Lexer.h:
+        * parser/Parser.h:
+        * runtime/ArgList.h:
+        * runtime/BatchedTransitionOptimizer.h:
+        * runtime/Collector.h:
+        * runtime/CommonIdentifiers.h:
+        * runtime/JSCell.h:
+        * runtime/JSGlobalObject.h:
+        * runtime/JSLock.h:
+        * runtime/JSONObject.cpp:
+        * runtime/SmallStrings.cpp:
+        * runtime/SmallStrings.h:
+        * wtf/CrossThreadRefCounted.h:
+        * wtf/GOwnPtr.h:
+        * wtf/Locker.h:
+        * wtf/MessageQueue.h:
+        * wtf/OwnArrayPtr.h:
+        * wtf/OwnFastMallocPtr.h:
+        * wtf/OwnPtr.h:
+        * wtf/RefCounted.h:
+        * wtf/ThreadSpecific.h:
+        * wtf/Threading.h:
+        * wtf/Vector.h:
+        * wtf/unicode/Collator.h:
+
+2009-07-14  Zoltan Horvath  <hzoltan@inf.u-szeged.hu>
+
+        Reviewed by Darin Adler.
+
         Change ParserArenaRefCounted's superclass to RefCountedCustomAllocated
         https://bugs.webkit.org/show_bug.cgi?id=27249
 
index 7d7dc9c..fa95603 100644 (file)
@@ -140,7 +140,7 @@ namespace JSC {
         friend class HostCallRecord;
         
 #if ENABLE(OPCODE_SAMPLING)
-        class CallRecord : Noncopyable {
+        class CallRecord : public Noncopyable {
         public:
             CallRecord(SamplingTool* samplingTool)
                 : m_samplingTool(samplingTool)
@@ -170,7 +170,7 @@ namespace JSC {
             }
         };
 #else
-        class CallRecord : Noncopyable {
+        class CallRecord : public Noncopyable {
         public:
             CallRecord(SamplingTool*)
             {
index 0223c2a..3532ad8 100644 (file)
@@ -35,7 +35,7 @@
 
 namespace JSC {
 
-    class RegisterID : Noncopyable {
+    class RegisterID : public Noncopyable {
     public:
         RegisterID()
             : m_refCount(0)
index f48f4f4..767c262 100644 (file)
@@ -32,7 +32,7 @@
 #include "Interpreter.h"
 
 namespace JSC {
-    class CachedCall : Noncopyable {
+    class CachedCall : public Noncopyable {
     public:
         CachedCall(CallFrame* callFrame, JSFunction* function, int argCount, JSValue* exception)
             : m_valid(false)
index 3a6e63b..e391c24 100644 (file)
@@ -92,7 +92,7 @@ namespace JSC {
 
     class JSGlobalObject;
 
-    class RegisterFile : Noncopyable {
+    class RegisterFile : public Noncopyable {
         friend class JIT;
     public:
         enum CallFrameHeaderEntry {
index 0e1b618..2583162 100644 (file)
@@ -33,7 +33,7 @@ namespace JSC {
 
     class RegExp;
 
-    class Lexer : Noncopyable {
+    class Lexer : public Noncopyable {
     public:
         // Character manipulation functions.
         static bool isWhiteSpace(int character);
index 6f4c2b7..373dc00 100644 (file)
@@ -39,7 +39,7 @@ namespace JSC {
 
     template <typename T> struct ParserArenaData : ParserArenaDeletable { T data; };
 
-    class Parser : Noncopyable {
+    class Parser : public Noncopyable {
     public:
         template <class ParsedNode> PassRefPtr<ParsedNode> parse(ExecState*, Debugger*, const SourceCode&, int* errLine = 0, UString* errMsg = 0);
         template <class ParsedNode> PassRefPtr<ParsedNode> reparse(JSGlobalData*, ParsedNode*);
index 4d571fc..0899e85 100644 (file)
@@ -31,7 +31,7 @@
 
 namespace JSC {
     
-    class MarkedArgumentBuffer : Noncopyable {
+    class MarkedArgumentBuffer : public Noncopyable {
     private:
         static const unsigned inlineCapacity = 8;
         typedef Vector<Register, inlineCapacity> VectorType;
index 13dd95c..b9f738f 100644 (file)
@@ -32,7 +32,7 @@
 
 namespace JSC {
 
-    class BatchedTransitionOptimizer : Noncopyable {
+    class BatchedTransitionOptimizer : public Noncopyable {
     public:
         BatchedTransitionOptimizer(JSObject* object)
             : m_object(object)
index 23f9f15..852ac59 100644 (file)
@@ -63,7 +63,7 @@ namespace JSC {
         OperationInProgress operationInProgress;
     };
 
-    class Heap : Noncopyable {
+    class Heap : public Noncopyable {
     public:
         class Thread;
         typedef CollectorHeapIterator<PrimaryHeap> iterator;
index 408d819..7b275bd 100644 (file)
@@ -70,7 +70,7 @@
 
 namespace JSC {
 
-    class CommonIdentifiers : Noncopyable {
+    class CommonIdentifiers : public Noncopyable {
     private:
         CommonIdentifiers(JSGlobalData*);
         friend class JSGlobalData;
index e0a9b4d..a418b7f 100644 (file)
@@ -31,7 +31,7 @@
 
 namespace JSC {
 
-    class JSCell : Noncopyable {
+    class JSCell : public Noncopyable {
         friend class GetterSetter;
         friend class Heap;
         friend class JIT;
index 983274b..7ab759d 100644 (file)
@@ -391,7 +391,7 @@ namespace JSC {
         return globalData().dynamicGlobalObject;
     }
 
-    class DynamicGlobalObjectScope : Noncopyable {
+    class DynamicGlobalObjectScope : public Noncopyable {
     public:
         DynamicGlobalObjectScope(CallFrame* callFrame, JSGlobalObject* dynamicGlobalObject) 
             : m_dynamicGlobalObjectSlot(callFrame->globalData().dynamicGlobalObject)
index 3dde358..70f6a62 100644 (file)
@@ -50,7 +50,7 @@ namespace JSC {
 
     class ExecState;
 
-    class JSLock : Noncopyable {
+    class JSLock : public Noncopyable {
     public:
         JSLock(ExecState*);
 
@@ -85,7 +85,7 @@ namespace JSC {
 
         bool m_lockingForReal;
 
-        class DropAllLocks : Noncopyable {
+        class DropAllLocks : public Noncopyable {
         public:
             DropAllLocks(ExecState* exec);
             DropAllLocks(bool);
index 4a89c55..2f02b1d 100644 (file)
@@ -61,7 +61,7 @@ private:
     mutable JSValue m_value;
 };
 
-class Stringifier : Noncopyable {
+class Stringifier : public Noncopyable {
 public:
     Stringifier(ExecState*, JSValue replacer, JSValue space);
     ~Stringifier();
index 87b49f0..9d1f01a 100644 (file)
@@ -34,7 +34,7 @@
 namespace JSC {
 static const unsigned numCharactersToStore = 0x100;
 
-class SmallStringsStorage : Noncopyable {
+class SmallStringsStorage : public Noncopyable {
 public:
     SmallStringsStorage();
 
index e7f1170..f0dd8df 100644 (file)
@@ -36,7 +36,7 @@ namespace JSC {
 
     class SmallStringsStorage;
 
-    class SmallStrings : Noncopyable {
+    class SmallStrings : public Noncopyable {
     public:
         SmallStrings();
         ~SmallStrings();
index 281dfa6..6a05211 100644 (file)
@@ -51,7 +51,7 @@ namespace WTF {
     // with respect to the original and any other copies.  The underlying m_data is jointly
     // owned by the original instance and all copies.
     template<class T>
-    class CrossThreadRefCounted : Noncopyable {
+    class CrossThreadRefCounted : public Noncopyable {
     public:
         static PassRefPtr<CrossThreadRefCounted<T> > create(T* data)
         {
index 8d03ff2..4993348 100644 (file)
@@ -37,7 +37,7 @@ namespace WTF {
     template<> void freeOwnedGPtr<GDir>(GDir*);
     template<> void freeOwnedGPtr<GHashTable>(GHashTable*);
 
-    template <typename T> class GOwnPtr : Noncopyable {
+    template <typename T> class GOwnPtr : public Noncopyable {
     public:
         explicit GOwnPtr(T* ptr = 0) : m_ptr(ptr) { }
         ~GOwnPtr() { freeOwnedGPtr(m_ptr); }
index 9feec1f..41813d3 100644 (file)
@@ -32,7 +32,7 @@
 
 namespace WTF {
 
-template <typename T> class Locker : Noncopyable {
+template <typename T> class Locker : public Noncopyable {
 public:
     Locker(T& lockable) : m_lockable(lockable) { m_lockable.lock(); }
     ~Locker() { m_lockable.unlock(); }
index 7721dba..12291cc 100644 (file)
@@ -45,7 +45,7 @@ namespace WTF {
     };
 
     template<typename DataType>
-    class MessageQueue : Noncopyable {
+    class MessageQueue : public Noncopyable {
     public:
         MessageQueue() : m_killed(false) { }
         
index 344f813..f6b34cf 100644 (file)
@@ -27,7 +27,7 @@
 
 namespace WTF {
 
-    template <typename T> class OwnArrayPtr : Noncopyable {
+    template <typename T> class OwnArrayPtr : public Noncopyable {
     public:
         explicit OwnArrayPtr(T* ptr = 0) : m_ptr(ptr) { }
         ~OwnArrayPtr() { safeDelete(); }
index 5c0d064..c88235a 100644 (file)
@@ -27,7 +27,7 @@
 
 namespace WTF {
 
-    template<class T> class OwnFastMallocPtr : Noncopyable {
+    template<class T> class OwnFastMallocPtr : public Noncopyable {
     public:
         explicit OwnFastMallocPtr(T* ptr) : m_ptr(ptr)
         {
index 9e4bd32..b7e62b1 100644 (file)
@@ -34,7 +34,7 @@ namespace WTF {
 
     template <typename T> class PassOwnPtr;
 
-    template <typename T> class OwnPtr : Noncopyable {
+    template <typename T> class OwnPtr : public Noncopyable {
     public:
         typedef typename RemovePointer<T>::Type ValueType;
         typedef ValueType* PtrType;
index b07a9a2..4d5d2f7 100644 (file)
@@ -59,7 +59,7 @@ namespace WTF {
 void ThreadSpecificThreadExit();
 #endif
 
-template<typename T> class ThreadSpecific : Noncopyable {
+template<typename T> class ThreadSpecific : public Noncopyable {
 public:
     ThreadSpecific();
     T* operator->();
index b12f41f..6578151 100644 (file)
@@ -159,7 +159,7 @@ typedef void* PlatformReadWriteLock;
 typedef void* PlatformCondition;
 #endif
     
-class Mutex : Noncopyable {
+class Mutex : public Noncopyable {
 public:
     Mutex();
     ~Mutex();
@@ -176,7 +176,7 @@ private:
 
 typedef Locker<Mutex> MutexLocker;
 
-class ReadWriteLock : Noncopyable {
+class ReadWriteLock : public Noncopyable {
 public:
     ReadWriteLock();
     ~ReadWriteLock();
@@ -193,7 +193,7 @@ private:
     PlatformReadWriteLock m_readWriteLock;
 };
 
-class ThreadCondition : Noncopyable {
+class ThreadCondition : public Noncopyable {
 public:
     ThreadCondition();
     ~ThreadCondition();
@@ -234,7 +234,7 @@ inline int atomicDecrement(int volatile* addend) { return __gnu_cxx::__exchange_
 
 #endif
 
-class ThreadSafeSharedBase : Noncopyable {
+class ThreadSafeSharedBase : public Noncopyable {
 public:
     ThreadSafeSharedBase(int initialRefCount = 1)
         : m_refCount(initialRefCount)
index c378fd0..f03becd 100644 (file)
@@ -268,7 +268,7 @@ namespace WTF {
     };
 
     template<typename T>
-    class VectorBufferBase : Noncopyable {
+    class VectorBufferBase : public Noncopyable {
     public:
         void allocateBuffer(size_t newCapacity)
         {
index f04779d..51e8a06 100644 (file)
@@ -39,7 +39,7 @@ struct UCollator;
 
 namespace WTF {
 
-    class Collator : Noncopyable {
+    class Collator : public Noncopyable {
     public:
         enum Result { Equal = 0, Greater = 1, Less = -1 };
 
index 95afd7a..eb1c3dc 100644 (file)
@@ -1,3 +1,92 @@
+2009-07-14  Zoltan Horvath  <hzoltan@inf.u-szeged.hu>
+
+        Reviewed by Darin Adler.
+
+        Change all Noncopyable inheriting visibility to public.
+        https://bugs.webkit.org/show_bug.cgi?id=27225
+
+        Change all Noncopyable inheriting visibility to public because
+        it is needed to the custom allocation framework (bug #20422).
+
+        * bindings/js/GCController.h:
+        * bindings/js/WorkerScriptController.h:
+        * bindings/v8/V8DOMMap.cpp:
+        (WebCore::):
+        * bridge/runtime.h:
+        * css/CSSSelector.h:
+        * css/CSSSelectorList.h:
+        * css/CSSStyleSelector.h:
+        * dom/ClassNames.h:
+        * dom/MessagePortChannel.h:
+        * dom/XMLTokenizerLibxml2.cpp:
+        * dom/XMLTokenizerScope.h:
+        * editing/ReplaceSelectionCommand.cpp:
+        * editing/SelectionController.h:
+        * editing/TextIterator.cpp:
+        * history/PageCache.h:
+        * html/CanvasRenderingContext2D.h:
+        * html/HTMLParser.h:
+        * html/HTMLParserQuirks.h:
+        * html/PreloadScanner.h:
+        * loader/Cache.h:
+        * loader/CrossOriginPreflightResultCache.h:
+        * loader/FrameLoader.h:
+        * loader/ProgressTracker.h:
+        * loader/ThreadableLoader.h:
+        * loader/appcache/ApplicationCacheGroup.h:
+        * loader/archive/ArchiveResourceCollection.h:
+        * loader/icon/IconDatabase.h:
+        * loader/icon/IconLoader.h:
+        * loader/icon/PageURLRecord.h:
+        * loader/loader.h:
+        * page/ContextMenuController.h:
+        * page/EventHandler.h:
+        * page/FrameTree.h:
+        * page/Page.h:
+        * page/PageGroup.h:
+        * page/PageGroupLoadDeferrer.h:
+        * page/mac/EventHandlerMac.mm:
+        * platform/AutodrainedPool.h:
+        * platform/ContextMenu.h:
+        * platform/EventLoop.h:
+        * platform/HostWindow.h:
+        * platform/Pasteboard.h:
+        * platform/PurgeableBuffer.h:
+        * platform/RunLoopTimer.h:
+        * platform/ThreadGlobalData.h:
+        * platform/ThreadTimers.h:
+        * platform/Timer.h:
+        * platform/TreeShared.h:
+        * platform/graphics/FontData.h:
+        * platform/graphics/GlyphWidthMap.h:
+        * platform/graphics/GraphicsContext.h:
+        * platform/graphics/ImageBuffer.h:
+        * platform/graphics/ImageSource.h:
+        * platform/graphics/MediaPlayer.h:
+        * platform/graphics/skia/GraphicsContextPlatformPrivate.h:
+        * platform/graphics/skia/PlatformContextSkia.h:
+        * platform/graphics/win/QTMovieWin.cpp:
+        * platform/mac/LocalCurrentGraphicsContext.h:
+        * platform/network/FormDataBuilder.h:
+        * platform/network/ResourceHandleInternal.h:
+        * platform/network/soup/ResourceHandleSoup.cpp:
+        * platform/text/StringBuffer.h:
+        * platform/text/TextCodec.h:
+        * platform/win/WindowMessageBroadcaster.h:
+        * rendering/CounterNode.h:
+        * rendering/LayoutState.h:
+        * rendering/RenderFrameSet.h:
+        * rendering/RenderView.h:
+        * rendering/TransformState.h:
+        * svg/SVGAnimatedProperty.h:
+        * svg/SynchronizableTypeWrapper.h:
+        * workers/WorkerMessagingProxy.h:
+        * workers/WorkerRunLoop.cpp:
+        * xml/XPathExpressionNode.h:
+        * xml/XPathParser.h:
+        * xml/XPathPredicate.h:
+        * xml/XPathStep.h:
+
 2009-07-14  Darin Fisher  <darin@chromium.org>
 
         Reviewed by Darin Adler.
index 452019a..4c25407 100644 (file)
@@ -31,7 +31,7 @@
 
 namespace WebCore {
 
-    class GCController : Noncopyable {
+    class GCController : public Noncopyable {
         friend GCController& gcController();
 
     public:
index 0454721..bb33f60 100644 (file)
@@ -45,7 +45,7 @@ namespace WebCore {
     class String;
     class WorkerContext;
 
-    class WorkerScriptController : Noncopyable {
+    class WorkerScriptController : public Noncopyable {
     public:
         WorkerScriptController(WorkerContext*);
         ~WorkerScriptController();
index 5044936..8bcf87d 100644 (file)
@@ -115,7 +115,7 @@ typedef WTF::Vector<DOMDataStore*> DOMDataList;
 // This class doesn't manage the lifetime of the store.  The data store
 // lifetime is managed by subclasses.
 //
-class DOMDataStore : Noncopyable {
+class DOMDataStore : public Noncopyable {
 public:
     enum DOMWrapperMapType {
         DOMNodeMap,
@@ -287,7 +287,7 @@ typedef WTF::Vector<DOMDataStore*> DOMDataStoreList;
 // thread.  The DOMData for the main thread and the DOMData for child threads
 // use different subclasses.
 //
-class DOMData: Noncopyable {
+class DOMData: public Noncopyable {
 public:
     DOMData()
         : m_delayedProcessingScheduled(false)
index 72736d4..2f74a4e 100644 (file)
@@ -55,14 +55,14 @@ public:
     virtual ~Field() { }
 };
 
-class Method : Noncopyable {
+class Method : public Noncopyable {
 public:
     virtual int numParameters() const = 0;
         
     virtual ~Method() { }
 };
 
-class Class : Noncopyable {
+class Class : public Noncopyable {
 public:
     virtual MethodList methodsNamed(const Identifier&, Instance*) const = 0;
     virtual Field* fieldNamed(const Identifier&, Instance*) const = 0;
@@ -120,7 +120,7 @@ protected:
     RefPtr<RootObject> _rootObject;
 };
 
-class Array : Noncopyable {
+class Array : public Noncopyable {
 public:
     Array(PassRefPtr<RootObject>);
     virtual ~Array();
index b24f057..5b7f9fa 100644 (file)
@@ -31,7 +31,7 @@
 namespace WebCore {
 
     // this class represents a selector for a StyleRule
-    class CSSSelector : Noncopyable {
+    class CSSSelector : public Noncopyable {
     public:
         CSSSelector()
             : m_tag(anyQName())
index 7a41fcf..3518139 100644 (file)
@@ -31,7 +31,7 @@
 
 namespace WebCore {
     
-    class CSSSelectorList : Noncopyable {
+    class CSSSelectorList : public Noncopyable {
     public:
         CSSSelectorList() : m_selectorArray(0) { }
         ~CSSSelectorList();
index c9df876..de8e915 100644 (file)
@@ -78,7 +78,7 @@ public:
 };
 
     // This class selects a RenderStyle for a given element based on a collection of stylesheets.
-    class CSSStyleSelector : Noncopyable {
+    class CSSStyleSelector : public Noncopyable {
     public:
         CSSStyleSelector(Document*, const String& userStyleSheet, StyleSheetList*, CSSStyleSheet*, bool strictParsing, bool matchAuthorAndUserStyles);
         ~CSSStyleSelector();
index 8f4852f..a836606 100644 (file)
@@ -27,7 +27,7 @@
 
 namespace WebCore {
 
-    class ClassNamesData : Noncopyable {
+    class ClassNamesData : public Noncopyable {
     public:
         ClassNamesData(const String& string, bool shouldFoldCase)
             : m_string(string), m_shouldFoldCase(shouldFoldCase), m_createdVector(false)
index 3841020..6d32bd3 100644 (file)
@@ -48,7 +48,7 @@ namespace WebCore {
 
     // MessagePortChannel is a platform-independent interface to the remote side of a message channel.
     // It acts as a wrapper around the platform-dependent PlatformMessagePortChannel implementation which ensures that the platform-dependent close() method is invoked before destruction.
-    class MessagePortChannel : Noncopyable {
+    class MessagePortChannel : public Noncopyable {
     public:
         static void createChannel(PassRefPtr<MessagePort>, PassRefPtr<MessagePort>);
 
index 95f63e9..4387a66 100644 (file)
@@ -72,7 +72,7 @@ using namespace std;
 
 namespace WebCore {
 
-class PendingCallbacks : Noncopyable {
+class PendingCallbacks : public Noncopyable {
 public:
     ~PendingCallbacks()
     {
index a3c1188..c29b796 100644 (file)
@@ -36,7 +36,7 @@ namespace WebCore {
 
     class DocLoader;
 
-    class XMLTokenizerScope : Noncopyable {
+    class XMLTokenizerScope : public Noncopyable {
     public:
         XMLTokenizerScope(DocLoader* docLoader);
         ~XMLTokenizerScope();
index c6da864..589e360 100644 (file)
@@ -60,7 +60,7 @@ enum EFragmentType { EmptyFragment, SingleTextNodeFragment, TreeFragment };
 
 // --- ReplacementFragment helper class
 
-class ReplacementFragment : Noncopyable {
+class ReplacementFragment : public Noncopyable {
 public:
     ReplacementFragment(Document*, DocumentFragment*, bool matchStyle, const VisibleSelection&);
 
index bbd343c..4a13a30 100644 (file)
@@ -38,7 +38,7 @@ class GraphicsContext;
 class RenderObject;
 class VisiblePosition;
 
-class SelectionController : Noncopyable {
+class SelectionController : public Noncopyable {
 public:
     enum EAlteration { MOVE, EXTEND };
     enum EDirection { FORWARD, BACKWARD, RIGHT, LEFT };
index 3d8296b..f82a972 100644 (file)
@@ -57,7 +57,7 @@ using namespace HTMLNames;
 // Keeps enough of the previous text to be able to search in the future, but no more.
 // Non-breaking spaces are always equal to normal spaces.
 // Case folding is also done if <isCaseSensitive> is false.
-class SearchBuffer : Noncopyable {
+class SearchBuffer : public Noncopyable {
 public:
     SearchBuffer(const String& target, bool isCaseSensitive);
     ~SearchBuffer();
index ad15ab6..3a817b5 100644 (file)
@@ -37,7 +37,7 @@ namespace WebCore {
     class CachedPage;
     class HistoryItem;
     
-    class PageCache : Noncopyable {
+    class PageCache : public Noncopyable {
     public:
         friend PageCache* pageCache();
 
index f6baa70..9648ffc 100644 (file)
@@ -54,7 +54,7 @@ namespace WebCore {
 
     typedef int ExceptionCode;
 
-    class CanvasRenderingContext2D : Noncopyable {
+    class CanvasRenderingContext2D : public Noncopyable {
     public:
         CanvasRenderingContext2D(HTMLCanvasElement*);
         
index bd12049..0945826 100644 (file)
@@ -49,7 +49,7 @@ struct Token;
  * The parser for HTML. It receives a stream of tokens from the HTMLTokenizer, and
  * builds up the Document structure from it.
  */
-class HTMLParser : Noncopyable {
+class HTMLParser : public Noncopyable {
 public:
     HTMLParser(HTMLDocument*, bool reportErrors);
     HTMLParser(DocumentFragment*);
index b5972a6..176bbfb 100644 (file)
@@ -33,7 +33,7 @@ namespace WebCore {
 class AtomicString;
 class Node;
 
-class HTMLParserQuirks : Noncopyable {
+class HTMLParserQuirks : public Noncopyable {
 public:
     HTMLParserQuirks() { }
     virtual ~HTMLParserQuirks() { }
index f1d2cf8..6022ee7 100644 (file)
@@ -37,7 +37,7 @@ namespace WebCore {
     class CachedResourceClient;
     class Document;
     
-    class PreloadScanner : Noncopyable {
+    class PreloadScanner : public Noncopyable {
     public:
         PreloadScanner(Document*);
         ~PreloadScanner();
index 86a6ceb..a0023da 100644 (file)
@@ -55,7 +55,7 @@ class KURL;
 // -------|-----+++++++++++++++|
 // -------|-----+++++++++++++++|+++++
 
-class Cache : Noncopyable {
+class Cache : public Noncopyable {
 public:
     friend Cache* cache();
 
index 39c3cd1..f71d1c8 100644 (file)
@@ -32,7 +32,7 @@ namespace WebCore {
     class HTTPHeaderMap;
     class ResourceResponse;
 
-    class CrossOriginPreflightResultCacheItem : Noncopyable {
+    class CrossOriginPreflightResultCacheItem : public Noncopyable {
     public:
         CrossOriginPreflightResultCacheItem(bool credentials)
             : m_absoluteExpiryTime(0)
@@ -57,7 +57,7 @@ namespace WebCore {
         HeadersSet m_headers;
     };
 
-    class CrossOriginPreflightResultCache : Noncopyable {
+    class CrossOriginPreflightResultCache : public Noncopyable {
     public:
         static CrossOriginPreflightResultCache& shared();
 
index b80a87c..b95a24d 100644 (file)
@@ -111,7 +111,7 @@ namespace WebCore {
         void* m_argument;
     };
 
-    class FrameLoader : Noncopyable {
+    class FrameLoader : public Noncopyable {
     public:
         FrameLoader(Frame*, FrameLoaderClient*);
         ~FrameLoader();
index b8d4532..744e101 100644 (file)
@@ -36,7 +36,7 @@ class Frame;
 class ResourceResponse;
 struct ProgressItem;
 
-class ProgressTracker : Noncopyable {
+class ProgressTracker : public Noncopyable {
 public:
     ProgressTracker();
     ~ProgressTracker();
index 87ae229..1ac12cb 100644 (file)
@@ -65,7 +65,7 @@ namespace WebCore {
 
     // Useful for doing loader operations from any thread (not threadsafe, 
     // just able to run on threads other than the main thread).
-    class ThreadableLoader : Noncopyable {
+    class ThreadableLoader : public Noncopyable {
     public:
         static void loadResourceSynchronously(ScriptExecutionContext*, const ResourceRequest&, ThreadableLoaderClient&, StoredCredentials);
         static PassRefPtr<ThreadableLoader> create(ScriptExecutionContext*, ThreadableLoaderClient*, const ResourceRequest&, LoadCallbacks, ContentSniff, StoredCredentials, CrossOriginRedirectPolicy);
index 063fb3b..ed68bf6 100644 (file)
@@ -52,7 +52,7 @@ enum ApplicationCacheUpdateOption {
     ApplicationCacheUpdateWithoutBrowsingContext
 };
 
-class ApplicationCacheGroup : Noncopyable, ResourceHandleClient {
+class ApplicationCacheGroup : public Noncopyable, ResourceHandleClient {
 public:
     ApplicationCacheGroup(const KURL& manifestURL, bool isCopy = false);    
     ~ApplicationCacheGroup();
index f898a8d..9d630d1 100644 (file)
@@ -39,7 +39,7 @@
 
 namespace WebCore {
 
-class ArchiveResourceCollection : Noncopyable {
+class ArchiveResourceCollection : public Noncopyable {
 public:
     ArchiveResourceCollection();
 
index 675e6c8..44ef22a 100644 (file)
@@ -62,7 +62,7 @@ enum IconLoadDecision {
     IconLoadUnknown
 };
 
-class IconDatabase : Noncopyable {
+class IconDatabase : public Noncopyable {
 
 // *** Main Thread Only ***
 public:
index a7194d8..7b96ed8 100644 (file)
@@ -38,7 +38,7 @@ class Frame;
 class KURL;
 class SharedBuffer;
 
-class IconLoader : private SubresourceLoaderClient, Noncopyable {
+class IconLoader : private SubresourceLoaderClient, public Noncopyable {
 public:
     static std::auto_ptr<IconLoader> create(Frame*);
     ~IconLoader();
index bc52f5b..f7ccb8f 100644 (file)
@@ -51,7 +51,7 @@ public:
     String iconURL;
 };
 
-class PageURLRecord : Noncopyable {
+class PageURLRecord : public Noncopyable {
 public:
     PageURLRecord(const String& pageURL);
     ~PageURLRecord();
index 3cced3d..d0a526f 100644 (file)
@@ -38,7 +38,7 @@ namespace WebCore {
     class KURL;
     class Request;
 
-    class Loader : Noncopyable {
+    class Loader : public Noncopyable {
     public:
         Loader();
         ~Loader();
index cb7e6ee..38095f6 100644 (file)
@@ -37,7 +37,7 @@ namespace WebCore {
     class Event;
     class Page;
 
-    class ContextMenuController : Noncopyable {
+    class ContextMenuController : public Noncopyable {
     public:
         ContextMenuController(Page*, ContextMenuClient*);
         ~ContextMenuController();
index d5c0b97..42f24fe 100644 (file)
@@ -69,7 +69,7 @@ extern const int GeneralDragHysteresis;
 
 enum HitTestScrollbars { ShouldHitTestScrollbars, DontHitTestScrollbars };
 
-class EventHandler : Noncopyable {
+class EventHandler : public Noncopyable {
 public:
     EventHandler(Frame*);
     ~EventHandler();
index d4c8c43..9ab999f 100644 (file)
@@ -26,7 +26,7 @@ namespace WebCore {
 
     class Frame;
 
-    class FrameTree : Noncopyable {
+    class FrameTree : public Noncopyable {
     public:
         FrameTree(Frame* thisFrame, Frame* parentFrame) 
             : m_thisFrame(thisFrame)
index b5d6f4b..ccc1d4c 100644 (file)
@@ -73,7 +73,7 @@ namespace WebCore {
 
     enum FindDirection { FindDirectionForward, FindDirectionBackward };
 
-    class Page : Noncopyable {
+    class Page : public Noncopyable {
     public:
         static void setNeedsReapplyStyles();
 
index cbde1c3..f92c2e6 100644 (file)
@@ -37,7 +37,7 @@ namespace WebCore {
     class Page;
     class StorageNamespace;
 
-    class PageGroup : Noncopyable {
+    class PageGroup : public Noncopyable {
     public:
         PageGroup(const String& name);
         PageGroup(Page*);
index 1bdb45c..d443ebd 100644 (file)
@@ -28,7 +28,7 @@ namespace WebCore {
     class Frame;
     class Page;
 
-    class PageGroupLoadDeferrer : Noncopyable {
+    class PageGroupLoadDeferrer : public Noncopyable {
     public:
         PageGroupLoadDeferrer(Page*, bool deferSelf);
         ~PageGroupLoadDeferrer();
index 9cc5f5b..927c80d 100644 (file)
@@ -73,7 +73,7 @@ NSEvent *EventHandler::currentNSEvent()
     return currentNSEventSlot().get();
 }
 
-class CurrentEventScope : Noncopyable {
+class CurrentEventScope : public Noncopyable {
 public:
     CurrentEventScope(NSEvent *);
     ~CurrentEventScope();
index 1cbbec6..d44ee1e 100644 (file)
@@ -39,7 +39,7 @@ class NSAutoreleasePool;
 
 namespace WebCore {
 
-class AutodrainedPool : Noncopyable {
+class AutodrainedPool : public Noncopyable {
 public:
     AutodrainedPool(int iterationLimit = 1);
     ~AutodrainedPool();
index 75899ba..dc484b2 100644 (file)
@@ -42,7 +42,7 @@ namespace WebCore {
 
     class ContextMenuController;
 
-    class ContextMenu : Noncopyable
+    class ContextMenu : public Noncopyable
     {
     public:
         ContextMenu(const HitTestResult&);
index 6687c23..b0507f7 100644 (file)
@@ -30,7 +30,7 @@
 
 namespace WebCore {
 
-    class EventLoop : Noncopyable {
+    class EventLoop : public Noncopyable {
     public:
         EventLoop()
             : m_ended(false)
index 3e982e1..3a024de 100644 (file)
@@ -31,7 +31,7 @@
 
 namespace WebCore {
 
-class HostWindow : Noncopyable {
+class HostWindow : public Noncopyable {
 public:
     virtual ~HostWindow() { }
 
index f98683f..883364a 100644 (file)
@@ -76,7 +76,7 @@ class Node;
 class Range;
 class String;
     
-class Pasteboard : Noncopyable {
+class Pasteboard : public Noncopyable {
 public:
 #if PLATFORM(MAC)
     //Helper functions to allow Clipboard to share code
index 9c8e3cb..c487eb9 100644 (file)
@@ -31,7 +31,7 @@
 
 namespace WebCore {
     
-    class PurgeableBuffer : Noncopyable {
+    class PurgeableBuffer : public Noncopyable {
     public:
         static PurgeableBuffer* create(const char* data, size_t);
         static PurgeableBuffer* create(const Vector<char>& v) { return create(v.data(), v.size()); }
index 96eb8d8..65f253e 100644 (file)
@@ -37,7 +37,7 @@ namespace WebCore {
 
 // Time intervals are all in seconds.
 
-class RunLoopTimerBase : Noncopyable {
+class RunLoopTimerBase : public Noncopyable {
 public:
     virtual ~RunLoopTimerBase();
 
index e0aa092..68f5ec0 100644 (file)
@@ -38,7 +38,7 @@ namespace WebCore {
     struct TECConverterWrapper;
     class ThreadTimers;
 
-    class ThreadGlobalData : Noncopyable {
+    class ThreadGlobalData : public Noncopyable {
     public:
         ThreadGlobalData();
         ~ThreadGlobalData();
index 366c320..ea0a366 100644 (file)
@@ -37,7 +37,7 @@ namespace WebCore {
     class TimerBase;
 
     // A collection of timers per thread. Kept in ThreadGlobalData.
-    class ThreadTimers : Noncopyable {
+    class ThreadTimers : public Noncopyable {
     public:
         ThreadTimers();
 
index 8723515..9221df0 100644 (file)
@@ -35,7 +35,7 @@ namespace WebCore {
 
 class TimerHeapElement;
 
-class TimerBase : Noncopyable {
+class TimerBase : public Noncopyable {
 public:
     TimerBase();
     virtual ~TimerBase();
index aaa26aa..1ac1b33 100644 (file)
@@ -26,7 +26,7 @@
 
 namespace WebCore {
 
-template<class T> class TreeShared : Noncopyable {
+template<class T> class TreeShared : public Noncopyable {
 public:
     TreeShared()
         : m_refCount(0)
index cb79919..42f06b9 100644 (file)
@@ -33,7 +33,7 @@ namespace WebCore {
 
 class SimpleFontData;
 
-class FontData : Noncopyable {
+class FontData : public Noncopyable {
 public:
     FontData()
         : m_maxGlyphPageTreeLevel(0)
index e194ecf..66dea1f 100644 (file)
@@ -39,7 +39,7 @@ typedef unsigned short Glyph;
 
 const float cGlyphWidthUnknown = -1;
 
-class GlyphWidthMap : Noncopyable {
+class GlyphWidthMap : public Noncopyable {
 public:
     GlyphWidthMap() : m_filledPrimaryPage(false) { }
     ~GlyphWidthMap() { if (m_pages) { deleteAllValues(*m_pages); } }
index 01e2fe6..5f2f7a3 100644 (file)
@@ -137,7 +137,7 @@ namespace WebCore {
         InterpolationHigh
     };
 
-    class GraphicsContext : Noncopyable {
+    class GraphicsContext : public Noncopyable {
     public:
         GraphicsContext(PlatformGraphicsContext*);
         ~GraphicsContext();
index 573e274..841a891 100644 (file)
@@ -43,7 +43,7 @@ namespace WebCore {
     class IntRect;
     class String;
 
-    class ImageBuffer : Noncopyable {
+    class ImageBuffer : public Noncopyable {
     public:
         // Will return a null pointer on allocation failure.
         static PassOwnPtr<ImageBuffer> create(const IntSize& size, bool grayScale)
index e9f066e..acb64de 100644 (file)
@@ -83,7 +83,7 @@ typedef NativeImageSkia* NativeImagePtr;
 const int cAnimationLoopOnce = -1;
 const int cAnimationNone = -2;
 
-class ImageSource : Noncopyable {
+class ImageSource : public Noncopyable {
 public:
     ImageSource();
     ~ImageSource();
index dae1a92..7f5f2a0 100644 (file)
@@ -96,7 +96,7 @@ public:
 #endif
 };
 
-class MediaPlayer : Noncopyable {
+class MediaPlayer : public Noncopyable {
 public:
     MediaPlayer(MediaPlayerClient*);
     virtual ~MediaPlayer();
index 29738f4..5e12ad6 100644 (file)
@@ -38,7 +38,7 @@ class PlatformContextSkia;
 namespace WebCore {
 
 // This class just holds onto a PlatformContextSkia for GraphicsContext.
-class GraphicsContextPlatformPrivate : Noncopyable {
+class GraphicsContextPlatformPrivate : public Noncopyable {
 public:
     GraphicsContextPlatformPrivate(PlatformContextSkia* platformContext)
         : m_context(platformContext) { }
index 30f8666..5a8c715 100644 (file)
@@ -60,7 +60,7 @@
 // responsible for managing the painting state which is store in separate
 // SkPaint objects. This class provides the adaptor that allows the painting
 // state to be pushed and popped along with the bitmap.
-class PlatformContextSkia : Noncopyable {
+class PlatformContextSkia : public Noncopyable {
 public:
     // For printing, there shouldn't be any canvas. canvas can be NULL. If you
     // supply a NULL canvas, you can also call setCanvas later.
index f84826a..aaa61f1 100644 (file)
@@ -76,7 +76,7 @@ static void updateTaskTimer(int maxInterval = 1000)
     setSharedTimerFireDelay(static_cast<float>(intervalInMS) / 1000);
 }
 
-class QTMovieWinPrivate : Noncopyable {
+class QTMovieWinPrivate : public Noncopyable {
 public:
     QTMovieWinPrivate();
     ~QTMovieWinPrivate();
index 1c5cae7..90beb40 100644 (file)
@@ -31,7 +31,7 @@ class GraphicsContext;
     
 // This class automatically saves and restores the current NSGraphicsContext for
 // functions which call out into AppKit and rely on the currentContext being set
-class LocalCurrentGraphicsContext : Noncopyable {
+class LocalCurrentGraphicsContext : public Noncopyable {
 public:
     LocalCurrentGraphicsContext(GraphicsContext* graphicsContext);
     ~LocalCurrentGraphicsContext();
index 666f0c1..286f59f 100644 (file)
@@ -30,7 +30,7 @@ class CString;
 class Document;
 class TextEncoding;
 
-class FormDataBuilder : Noncopyable {
+class FormDataBuilder : public Noncopyable {
 public:
     FormDataBuilder();
     ~FormDataBuilder();
index 676129a..091a26b 100644 (file)
@@ -75,7 +75,7 @@ class NSURLConnection;
 namespace WebCore {
     class ResourceHandleClient;
 
-    class ResourceHandleInternal : Noncopyable {
+    class ResourceHandleInternal : public Noncopyable {
     public:
         ResourceHandleInternal(ResourceHandle* loader, const ResourceRequest& request, ResourceHandleClient* c, bool defersLoading, bool shouldContentSniff, bool mightDownloadFromHandle)
             : m_client(c)
index f58bfae..613f693 100644 (file)
@@ -55,7 +55,7 @@
 
 namespace WebCore {
 
-class WebCoreSynchronousLoader : public ResourceHandleClient, Noncopyable {
+class WebCoreSynchronousLoader : public ResourceHandleClient, public Noncopyable {
 public:
     WebCoreSynchronousLoader(ResourceError&, ResourceResponse &, Vector<char>&);
     ~WebCoreSynchronousLoader();
index 28d4e89..353a44a 100644 (file)
@@ -35,7 +35,7 @@
 
 namespace WebCore {
 
-class StringBuffer : Noncopyable {
+class StringBuffer : public Noncopyable {
 public:
     explicit StringBuffer(unsigned length)
         : m_length(length)
index df42582..3c74165 100644 (file)
@@ -56,7 +56,7 @@ namespace WebCore {
 
     typedef char UnencodableReplacementArray[32];
 
-    class TextCodec : Noncopyable {
+    class TextCodec : public Noncopyable {
     public:
         virtual ~TextCodec();
 
index 734f4b1..e7856e7 100644 (file)
@@ -37,7 +37,7 @@ namespace WebCore {
 
     class WindowMessageListener;
 
-    class WindowMessageBroadcaster : Noncopyable {
+    class WindowMessageBroadcaster : public Noncopyable {
     public:
         static void addListener(HWND, WindowMessageListener*);
         static void removeListener(HWND, WindowMessageListener*);
index 57f9563..b432e1d 100644 (file)
@@ -37,7 +37,7 @@ namespace WebCore {
 
 class RenderObject;
 
-class CounterNode : Noncopyable {
+class CounterNode : public Noncopyable {
 public:
     CounterNode(RenderObject*, bool isReset, int value);
 
index afa2952..2f040c8 100644 (file)
@@ -36,7 +36,7 @@ class RenderArena;
 class RenderBox;
 class RenderObject;
 
-class LayoutState : Noncopyable {
+class LayoutState : public Noncopyable {
 public:
     LayoutState()
         : m_clipped(false)
index 713a248..9f90502 100644 (file)
@@ -83,7 +83,7 @@ public:
 private:
     static const int noSplit = -1;
 
-    class GridAxis : Noncopyable {
+    class GridAxis : public Noncopyable {
     public:
         GridAxis();
         void resize(int);
index b0de7dd..854a421 100644 (file)
@@ -222,7 +222,7 @@ void toRenderView(const RenderView*);
 
 
 // Stack-based class to assist with LayoutState push/pop
-class LayoutStateMaintainer : Noncopyable {
+class LayoutStateMaintainer : public Noncopyable {
 public:
     // ctor to push now
     LayoutStateMaintainer(RenderView* view, RenderBox* root, IntSize offset, bool disableState = false)
index 92275f9..d2c962a 100644 (file)
@@ -37,7 +37,7 @@
 
 namespace WebCore {
 
-class TransformState : Noncopyable {
+class TransformState : public Noncopyable {
 public:
     enum TransformDirection { ApplyTransformDirection, UnapplyInverseTransformDirection };
     enum TransformAccumulation { FlattenTransform, AccumulateTransform };
index 334a6eb..6803055 100644 (file)
@@ -66,7 +66,7 @@ namespace WebCore {
     void synchronizeProperty(const OwnerElement* ownerElement, const QualifiedName& attributeName, DecoratedType baseValue);
 
     // Abstract base class
-    class SVGAnimatedPropertyBase : Noncopyable {
+    class SVGAnimatedPropertyBase : public Noncopyable {
     public:
         virtual ~SVGAnimatedPropertyBase() { }
         virtual void synchronize() const = 0;
index 418c76d..12f8426 100644 (file)
@@ -26,7 +26,7 @@
 namespace WebCore {
 
     template<typename StoredType>
-    class SynchronizableTypeWrapperBase : Noncopyable {
+    class SynchronizableTypeWrapperBase : public Noncopyable {
     protected:
         SynchronizableTypeWrapperBase();
 
index f9e1cd4..e13eb11 100644 (file)
@@ -47,7 +47,7 @@ namespace WebCore {
     class Worker;
     class WorkerThread;
 
-    class WorkerMessagingProxy : public WorkerContextProxy, public WorkerObjectProxy, public WorkerLoaderProxy, Noncopyable {
+    class WorkerMessagingProxy : public WorkerContextProxy, public WorkerObjectProxy, public WorkerLoaderProxy, public Noncopyable {
     public:
         WorkerMessagingProxy(Worker*);
 
index b6f6487..cb31fe7 100644 (file)
@@ -125,7 +125,7 @@ String WorkerRunLoop::defaultMode()
     return String();
 }
 
-class RunLoopSetup : Noncopyable {
+class RunLoopSetup : public Noncopyable {
 public:
     RunLoopSetup(WorkerRunLoop& runLoop)
         : m_runLoop(runLoop)
index d12b451..74b134e 100644 (file)
@@ -53,7 +53,7 @@ namespace WebCore {
             virtual ~ParseNode() { }
         };
 
-        class Expression : public ParseNode, Noncopyable {
+        class Expression : public ParseNode, public Noncopyable {
         public:
             static EvaluationContext& evaluationContext();
 
index 8d6da3f..e779603 100644 (file)
@@ -58,7 +58,7 @@ namespace WebCore {
             Token(int t, EqTestOp::Opcode v): type(t), eqop(v) {}
         };
 
-        class Parser : Noncopyable {
+        class Parser : public Noncopyable {
         public:
             Parser();
 
index 6c9d413..5f2482a 100644 (file)
@@ -105,7 +105,7 @@ namespace WebCore {
             virtual Value::Type resultType() const { return Value::NodeSetValue; }
         };
 
-        class Predicate : Noncopyable {
+        class Predicate : public Noncopyable {
         public:
             Predicate(Expression*);
             ~Predicate();
index 1c26327..11612e9 100644 (file)
@@ -39,7 +39,7 @@ namespace WebCore {
 
         class Predicate;
         
-        class Step : public ParseNode, Noncopyable {
+        class Step : public ParseNode, public Noncopyable {
         public:
             enum Axis {
                 AncestorAxis, AncestorOrSelfAxis, AttributeAxis,
index 7b99908..1ecfa4a 100644 (file)
@@ -1,3 +1,15 @@
+2009-07-14  Zoltan Horvath  <hzoltan@inf.u-szeged.hu>
+
+        Reviewed by Darin Adler.
+
+        Change all Noncopyable inheriting visibility to public.
+        https://bugs.webkit.org/show_bug.cgi?id=27225
+
+        Change all Noncopyable inheriting visibility to public because
+        it is needed to the custom allocation framework (bug #20422).
+
+        * webkit/webkitdownload.cpp:
+
 2009-07-13  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
 
         Unreviewed. Late NEWS file for 1.1.11.
index 3b4ba8c..ccd21bd 100644 (file)
@@ -47,7 +47,7 @@ using namespace WebCore;
  * out what is to be downloaded, and do it itself.
  */
 
-class DownloadClient : Noncopyable, public ResourceHandleClient {
+class DownloadClient : public Noncopyable, public ResourceHandleClient {
     public:
         DownloadClient(WebKitDownload*);
 
index 98b770d..a93a3c2 100644 (file)
@@ -34,7 +34,7 @@
 #include "COMVariantSetter.h"
 
 template<typename ContainerType>
-class COMEnumVariant : public IEnumVARIANT, Noncopyable {
+class COMEnumVariant : public IEnumVARIANT, public Noncopyable {
 public:
     static COMEnumVariant* adopt(ContainerType&);
     static COMEnumVariant* createInstance(const ContainerType&);
index 2695986..6612837 100644 (file)
@@ -1,3 +1,18 @@
+2009-07-14  Zoltan Horvath  <hzoltan@inf.u-szeged.hu>
+
+        Reviewed by Darin Adler.
+
+        Change all Noncopyable inheriting visibility to public.
+        https://bugs.webkit.org/show_bug.cgi?id=27225
+
+        Change all Noncopyable inheriting visibility to public because
+        it is needed to the custom allocation framework (bug #20422).
+
+        * COMEnumVariant.h:
+        * WebDatabaseManager.cpp:
+        * WebInspector.h:
+        * WebLocalizableStrings.cpp:
+
 2009-07-13  Brent Fulgham  <bfulgham@webkit.org>
 
         Reviewed by Adam Roben.
index c16a600..969bde7 100644 (file)
@@ -51,7 +51,7 @@ static inline bool isEqual(LPCWSTR s1, LPCWSTR s2)
     return !wcscmp(s1, s2);
 }
 
-class DatabaseDetailsPropertyBag : public IPropertyBag, Noncopyable {
+class DatabaseDetailsPropertyBag : public IPropertyBag, public Noncopyable {
 public:
     static DatabaseDetailsPropertyBag* createInstance(const DatabaseDetails&);
 
index b300098..5303a61 100644 (file)
@@ -34,7 +34,7 @@
 
 class WebView;
 
-class WebInspector : public IWebInspector, Noncopyable {
+class WebInspector : public IWebInspector, public Noncopyable {
 public:
     static WebInspector* createInstance(WebView*);
 
index df70bd4..e2da4cb 100644 (file)
@@ -48,7 +48,7 @@ WebLocalizableStringsBundle WebKitLocalizableStringsBundle = { "com.apple.WebKit
 static HashMap<String, LocalizedString*> mainBundleLocStrings;
 static HashMap<String, LocalizedString*> frameworkLocStrings;
 
-class LocalizedString : Noncopyable {
+class LocalizedString : public Noncopyable {
 public:
     LocalizedString(CFStringRef string)
         : m_cfString(string)