WebCore:
authorjchaffraix@webkit.org <jchaffraix@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 19 Jun 2008 21:28:35 +0000 (21:28 +0000)
committerjchaffraix@webkit.org <jchaffraix@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 19 Jun 2008 21:28:35 +0000 (21:28 +0000)
2008-06-19  Julien Chaffraix  <jchaffraix@webkit.org>

        Reviewed by Darin.

        Bug 19529 : Empty clients need to be refactored
        https://bugs.webkit.org/show_bug.cgi?id=19529

        - SVGImageEmptyClients' refactoring in order to be able to use them for other
        elements.

        - Trimmed empty spaces that were pointed out by git.

        * WebCore.xcodeproj/project.pbxproj:
        * loader/EmptyClients.h: Copied from WebCore/svg/graphics/SVGImageEmptyClients.h.

        (WebCore::EmptyChromeClient::~EmptyChromeClient):
        (WebCore::EmptyFrameLoaderClient::~EmptyFrameLoaderClient):
        (WebCore::EmptyEditorClient::~EmptyEditorClient):
        (WebCore::EmptyEditorClient::smartInsertDeleteEnabled):
        (WebCore::EmptyContextMenuClient::~EmptyContextMenuClient):
        (WebCore::EmptyDragClient::~EmptyDragClient):
        (WebCore::EmptyDragClient::createDragImageForLink):
        (WebCore::EmptyInspectorClient::~EmptyInspectorClient):
        Renamed SVGImageEmpty*Client classes to Empty*Client.

        * svg/graphics/SVGImage.cpp:
        (WebCore::SVGImage::dataChanged):
        * svg/graphics/SVGImageEmptyClients.h: Removed.

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

WebCore/ChangeLog
WebCore/WebCore.xcodeproj/project.pbxproj
WebCore/loader/EmptyClients.h [moved from WebCore/svg/graphics/SVGImageEmptyClients.h with 92% similarity]
WebCore/svg/graphics/SVGImage.cpp

index 4fdaef999e0eb1bf2a4fb72d1de42a14f7f385d0..4de2ecd5b3302665056f2fb86bac63c1e0cb1876 100644 (file)
@@ -1,3 +1,32 @@
+2008-06-19  Julien Chaffraix  <jchaffraix@webkit.org>
+
+        Reviewed by Darin.
+
+        Bug 19529 : Empty clients need to be refactored
+        https://bugs.webkit.org/show_bug.cgi?id=19529
+
+        - SVGImageEmptyClients' refactoring in order to be able to use them for other
+        elements.
+
+        - Trimmed empty spaces that were pointed out by git.
+
+        * WebCore.xcodeproj/project.pbxproj:
+        * loader/EmptyClients.h: Copied from WebCore/svg/graphics/SVGImageEmptyClients.h.
+
+        (WebCore::EmptyChromeClient::~EmptyChromeClient):
+        (WebCore::EmptyFrameLoaderClient::~EmptyFrameLoaderClient):
+        (WebCore::EmptyEditorClient::~EmptyEditorClient):
+        (WebCore::EmptyEditorClient::smartInsertDeleteEnabled):
+        (WebCore::EmptyContextMenuClient::~EmptyContextMenuClient):
+        (WebCore::EmptyDragClient::~EmptyDragClient):
+        (WebCore::EmptyDragClient::createDragImageForLink):
+        (WebCore::EmptyInspectorClient::~EmptyInspectorClient):
+        Renamed SVGImageEmpty*Client classes to Empty*Client.
+
+        * svg/graphics/SVGImage.cpp:
+        (WebCore::SVGImage::dataChanged):
+        * svg/graphics/SVGImageEmptyClients.h: Removed.
+
 2008-06-19  Dan Bernstein  <mitz@apple.com>
 
         Reviewed by Darin Adler.
index 8f682fd11da079f339b0ebe39a5420b255728441..e054d38efb5ae9c02552405ca403378b5189dfa5 100644 (file)
                B25599980D00D8BA00BB825C /* SVGResourceFilterPlatformDataMac.mm in Sources */ = {isa = PBXBuildFile; fileRef = B25598FF0D00D8B900BB825C /* SVGResourceFilterPlatformDataMac.mm */; };
                B25599A30D00D8BA00BB825C /* SVGImage.cpp in Sources */ = {isa = PBXBuildFile; fileRef = B255990B0D00D8B900BB825C /* SVGImage.cpp */; };
                B25599A40D00D8BA00BB825C /* SVGImage.h in Headers */ = {isa = PBXBuildFile; fileRef = B255990C0D00D8B900BB825C /* SVGImage.h */; };
-               B25599A50D00D8BA00BB825C /* SVGImageEmptyClients.h in Headers */ = {isa = PBXBuildFile; fileRef = B255990D0D00D8B900BB825C /* SVGImageEmptyClients.h */; };
+               B25599A50D00D8BA00BB825C /* EmptyClients.h in Headers */ = {isa = PBXBuildFile; fileRef = B255990D0D00D8B900BB825C /* EmptyClients.h */; };
                B25599A60D00D8BA00BB825C /* SVGPaintServer.cpp in Sources */ = {isa = PBXBuildFile; fileRef = B255990E0D00D8B900BB825C /* SVGPaintServer.cpp */; };
                B25599A70D00D8BA00BB825C /* SVGPaintServer.h in Headers */ = {isa = PBXBuildFile; fileRef = B255990F0D00D8B900BB825C /* SVGPaintServer.h */; };
                B25599A80D00D8BA00BB825C /* SVGPaintServerGradient.cpp in Sources */ = {isa = PBXBuildFile; fileRef = B25599100D00D8B900BB825C /* SVGPaintServerGradient.cpp */; };
                B25598FF0D00D8B900BB825C /* SVGResourceFilterPlatformDataMac.mm */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.objcpp; path = SVGResourceFilterPlatformDataMac.mm; sourceTree = "<group>"; };
                B255990B0D00D8B900BB825C /* SVGImage.cpp */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.cpp; path = SVGImage.cpp; sourceTree = "<group>"; };
                B255990C0D00D8B900BB825C /* SVGImage.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = SVGImage.h; sourceTree = "<group>"; };
-               B255990D0D00D8B900BB825C /* SVGImageEmptyClients.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = SVGImageEmptyClients.h; sourceTree = "<group>"; };
+               B255990D0D00D8B900BB825C /* EmptyClients.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = EmptyClients.h; sourceTree = "<group>"; };
                B255990E0D00D8B900BB825C /* SVGPaintServer.cpp */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.cpp; path = SVGPaintServer.cpp; sourceTree = "<group>"; };
                B255990F0D00D8B900BB825C /* SVGPaintServer.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = SVGPaintServer.h; sourceTree = "<group>"; };
                B25599100D00D8B900BB825C /* SVGPaintServerGradient.cpp */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.cpp; path = SVGPaintServerGradient.cpp; sourceTree = "<group>"; };
                                B25598FD0D00D8B900BB825C /* mac */,
                                B255990B0D00D8B900BB825C /* SVGImage.cpp */,
                                B255990C0D00D8B900BB825C /* SVGImage.h */,
-                               B255990D0D00D8B900BB825C /* SVGImageEmptyClients.h */,
                                B255990E0D00D8B900BB825C /* SVGPaintServer.cpp */,
                                B255990F0D00D8B900BB825C /* SVGPaintServer.h */,
                                B25599100D00D8B900BB825C /* SVGPaintServerGradient.cpp */,
                BCB16BFB0979C38700467741 /* loader */ = {
                        isa = PBXGroup;
                        children = (
+                               B255990D0D00D8B900BB825C /* EmptyClients.h */,
                                AB40484B0E083FA8007D6920 /* MediaDocument.cpp */,
                                AB40484C0E083FA8007D6920 /* MediaDocument.h */,
                                1A8F6BB00DB55CDC001DB794 /* appcache */,
                                B2227A2A0D00BF220071B782 /* SVGGradientElement.h in Headers */,
                                B25599A40D00D8BA00BB825C /* SVGImage.h in Headers */,
                                B2227A2D0D00BF220071B782 /* SVGImageElement.h in Headers */,
-                               B25599A50D00D8BA00BB825C /* SVGImageEmptyClients.h in Headers */,
+                               B25599A50D00D8BA00BB825C /* EmptyClients.h in Headers */,
                                B28C6A2A0D00C44800334AA4 /* SVGImageLoader.h in Headers */,
                                853CA9E70AEEC608002372DC /* SVGInlineFlowBox.h in Headers */,
                                AA0972CD0B6947A800A705E9 /* SVGInlineTextBox.h in Headers */,
similarity index 92%
rename from WebCore/svg/graphics/SVGImageEmptyClients.h
rename to WebCore/loader/EmptyClients.h
index 7b76465fb5a53e887d91a2558c8fb9d33a883b83..fcd49a4544ceef4f20aefe1bec94a26d825d3d1a 100644 (file)
  * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef SVGImageEmptyClients_h
-#define SVGImageEmptyClients_h
-
-#if ENABLE(SVG)
+#ifndef EmptyClients_h
+#define EmptyClients_h
 
 #include "ChromeClient.h"
 #include "ContextMenuClient.h"
 #include "DragClient.h"
+#include "DocumentLoader.h"
 #include "EditCommand.h"
 #include "EditorClient.h"
 #include "FocusDirection.h"
+#include "FloatRect.h"
 #include "FrameLoaderClient.h"
 #include "InspectorClient.h"
+#include "ResourceError.h"
 #include "SharedBuffer.h"
 
 /*
- This file holds empty Client stubs for use by SVGImage.
SVGImage needs to create a dummy Page->Frame->FrameView tree for use in parsing an SVGDocument.
+ This file holds empty Client stubs for use by WebCore.
Viewless element needs to create a dummy Page->Frame->FrameView tree for use in parsing or executing JavaScript.
  This tree depends heavily on Clients (usually provided by WebKit classes).
- SVGImage has no way to access the current Page (nor should it, since Images are not tied to a page).
- See http://bugs.webkit.org/show_bug.cgi?id=5971 for more discussion on this issue.
+
+ This file was first created for SVGImage as it had no way to access the current Page (nor should it,
+ since Images are not tied to a page).
+ See http://bugs.webkit.org/show_bug.cgi?id=5971 for the original discussion about this file.
+
  Ideally, whenever you change a Client class, you should add a stub here.
  Brittle, yes.  Unfortunate, yes.  Hopefully temporary.
 */
 
 namespace WebCore {
 
-class SVGEmptyChromeClient : public ChromeClient {
+class EmptyChromeClient : public ChromeClient {
 public:
-    virtual ~SVGEmptyChromeClient() { }
+    virtual ~EmptyChromeClient() { }
     virtual void chromeDestroyed() { }
-    
+
     virtual void setWindowRect(const FloatRect&) { }
     virtual FloatRect windowRect() { return FloatRect(); }
-    
+
     virtual FloatRect pageRect() { return FloatRect(); }
-    
+
     virtual float scaleFactor() { return 1.f; }
-    
+
     virtual void focus() { }
     virtual void unfocus() { }
-    
+
     virtual bool canTakeFocus(FocusDirection) { return false; }
     virtual void takeFocus(FocusDirection) { }
-    
+
     virtual Page* createWindow(Frame*, const FrameLoadRequest&, const WindowFeatures&) { return 0; }
     virtual void show() { }
-    
+
     virtual bool canRunModal() { return false; }
     virtual void runModal() { }
-    
+
     virtual void setToolbarsVisible(bool) { }
     virtual bool toolbarsVisible() { return false; }
-    
+
     virtual void setStatusbarVisible(bool) { }
     virtual bool statusbarVisible() { return false; }
-    
+
     virtual void setScrollbarsVisible(bool) { }
     virtual bool scrollbarsVisible() { return false; }
-    
+
     virtual void setMenubarVisible(bool) { }
     virtual bool menubarVisible() { return false; }
-    
+
     virtual void setResizable(bool) { }
-    
+
     virtual void addMessageToConsole(const String& message, unsigned int lineNumber, const String& sourceID) { }
-    
+
     virtual bool canRunBeforeUnloadConfirmPanel() { return false; }
     virtual bool runBeforeUnloadConfirmPanel(const String& message, Frame* frame) { return true; }
-    
+
     virtual void closeWindowSoon() { }
-    
+
     virtual void runJavaScriptAlert(Frame*, const String&) { }
     virtual bool runJavaScriptConfirm(Frame*, const String&) { return false; }
     virtual bool runJavaScriptPrompt(Frame*, const String& message, const String& defaultValue, String& result) { return false; }
     virtual bool shouldInterruptJavaScript() { return false; }
-    
+
     virtual void setStatusbarText(const String&) { }
-    
+
     virtual bool tabsToLinks() const { return false; }
-    
+
     virtual IntRect windowResizerRect() const { return IntRect(); }
     virtual void addToDirtyRegion(const IntRect&) { }
     virtual void scrollBackingStore(int dx, int dy, const IntRect& scrollViewRect, const IntRect& clipRect) { }
     virtual void updateBackingStore() { }
 
     virtual void mouseDidMoveOverElement(const HitTestResult&, unsigned modifierFlags) { }
-    
+
     virtual void setToolTip(const String&) { }
 
     virtual void print(Frame*) { }
@@ -120,35 +122,35 @@ public:
     virtual void exceededDatabaseQuota(Frame*, const String&) { }
 };
 
-class SVGEmptyFrameLoaderClient : public FrameLoaderClient {
+class EmptyFrameLoaderClient : public FrameLoaderClient {
 public:
-    virtual ~SVGEmptyFrameLoaderClient() {  }
+    virtual ~EmptyFrameLoaderClient() {  }
     virtual void frameLoaderDestroyed() { }
-    
+
     virtual bool hasWebView() const { return true; } // mainly for assertions
     virtual bool hasFrameView() const { return true; } // ditto
-    
+
     virtual void makeRepresentation(DocumentLoader*) { }
     virtual void forceLayout() { }
     virtual void forceLayoutForNonHTML() { }
-    
+
     virtual void updateHistoryForCommit() { }
-    
+
     virtual void updateHistoryForBackForwardNavigation() { }
     virtual void updateHistoryForReload() { }
     virtual void updateHistoryForStandardLoad() { }
     virtual void updateHistoryForInternalLoad() { }
-    
+
     virtual void updateHistoryAfterClientRedirect() { }
-    
+
     virtual void setCopiesOnScroll() { }
-        
+
     virtual void detachedFromParent2() { }
     virtual void detachedFromParent3() { }
     virtual void detachedFromParent4() { }
-    
+
     virtual void download(ResourceHandle*, const ResourceRequest&, const ResourceRequest&, const ResourceResponse&) { }
-    
+
     virtual void assignIdentifierToInitialRequest(unsigned long identifier, DocumentLoader*, const ResourceRequest&) { }
     virtual void dispatchWillSendRequest(DocumentLoader*, unsigned long identifier, ResourceRequest&, const ResourceResponse& redirectResponse) { }
     virtual void dispatchDidReceiveAuthenticationChallenge(DocumentLoader*, unsigned long identifier, const AuthenticationChallenge&) { }
@@ -158,7 +160,7 @@ public:
     virtual void dispatchDidFinishLoading(DocumentLoader*, unsigned long identifier) { }
     virtual void dispatchDidFailLoading(DocumentLoader*, unsigned long identifier, const ResourceError&) { }
     virtual bool dispatchDidLoadResourceFromMemoryCache(DocumentLoader*, const ResourceRequest&, const ResourceResponse&, int length) { return false; }
-    
+
     virtual void dispatchDidHandleOnloadEvents() { }
     virtual void dispatchDidReceiveServerRedirectForProvisionalLoad() { }
     virtual void dispatchDidCancelClientRedirect() { }
@@ -174,54 +176,54 @@ public:
     virtual void dispatchDidFinishDocumentLoad() { }
     virtual void dispatchDidFinishLoad() { }
     virtual void dispatchDidFirstLayout() { }
-    
+
     virtual Frame* dispatchCreatePage() { return 0; }
     virtual void dispatchShow() { }
-    
+
     virtual void dispatchDecidePolicyForMIMEType(FramePolicyFunction, const String& MIMEType, const ResourceRequest&) { }
     virtual void dispatchDecidePolicyForNewWindowAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>, const String& frameName) { }
     virtual void dispatchDecidePolicyForNavigationAction(FramePolicyFunction, const NavigationAction&, const ResourceRequest&, PassRefPtr<FormState>) { }
     virtual void cancelPolicyCheck() { }
-    
+
     virtual void dispatchUnableToImplementPolicy(const ResourceError&) { }
 
     virtual void dispatchWillSubmitForm(FramePolicyFunction, PassRefPtr<FormState>) { }
-    
+
     virtual void dispatchDidLoadMainResource(DocumentLoader*) { }
     virtual void revertToProvisionalState(DocumentLoader*) { }
     virtual void setMainDocumentError(DocumentLoader*, const ResourceError&) { }
-    
+
     virtual void willChangeEstimatedProgress() { }
     virtual void didChangeEstimatedProgress() { }
     virtual void postProgressStartedNotification() { }
     virtual void postProgressEstimateChangedNotification() { }
     virtual void postProgressFinishedNotification() { }
-    
+
     virtual void setMainFrameDocumentReady(bool) { }
-    
+
     virtual void startDownload(const ResourceRequest&) { }
-    
+
     virtual void willChangeTitle(DocumentLoader*) { }
     virtual void didChangeTitle(DocumentLoader*) { }
-    
+
     virtual void committedLoad(DocumentLoader*, const char*, int) { }
     virtual void finishedLoading(DocumentLoader*) { }
-    
+
     virtual ResourceError cancelledError(const ResourceRequest&) { return ResourceError(); }
     virtual ResourceError blockedError(const ResourceRequest&) { return ResourceError(); }
     virtual ResourceError cannotShowURLError(const ResourceRequest&) { return ResourceError(); }
     virtual ResourceError interruptForPolicyChangeError(const ResourceRequest&) { return ResourceError(); }
-    
+
     virtual ResourceError cannotShowMIMETypeError(const ResourceResponse&) { return ResourceError(); }
     virtual ResourceError fileDoesNotExistError(const ResourceResponse&) { return ResourceError(); }
-    
+
     virtual bool shouldFallBack(const ResourceError&) { return false; }
-        
+
     virtual bool canHandleRequest(const ResourceRequest&) const { return false; }
     virtual bool canShowMIMEType(const String& MIMEType) const { return false; }
     virtual bool representationExistsForURLScheme(const String& URLScheme) const { return false; }
     virtual String generatedMIMETypeForURLScheme(const String& URLScheme) const { return ""; }
-    
+
     virtual void frameLoadCompleted() { }
     virtual void restoreViewState() { }
     virtual void provisionalLoadStarted() { }
@@ -229,10 +231,10 @@ public:
     virtual void addHistoryItemForFragmentScroll() { }
     virtual void didFinishLoad() { }
     virtual void prepareForDataSourceReplacement() { }
-    
+
     virtual PassRefPtr<DocumentLoader> createDocumentLoader(const ResourceRequest& request, const SubstituteData& substituteData) { return DocumentLoader::create(request, substituteData); }
     virtual void setTitle(const String& title, const KURL&) { }
-    
+
     virtual String userAgent(const KURL&) { return ""; }
 
     virtual void savePlatformDataToCachedPage(CachedPage*) { }
@@ -248,7 +250,7 @@ public:
                                const String& referrer, bool allowsScrolling, int marginWidth, int marginHeight) { return 0; }
     virtual Widget* createPlugin(const IntSize&,Element*, const KURL&, const Vector<String>&, const Vector<String>&, const String&, bool) { return 0; }
     virtual Widget* createJavaAppletWidget(const IntSize&, Element*, const KURL&, const Vector<String>&, const Vector<String>&) { return 0; }
-    
+
     virtual ObjectContentType objectContentType(const KURL& url, const String& mimeType) { return ObjectContentType(); }
     virtual String overrideMediaType() const { return String(); }
 
@@ -264,23 +266,23 @@ public:
 
 };
 
-class SVGEmptyEditorClient : public EditorClient {
+class EmptyEditorClient : public EditorClient {
 public:
-    virtual ~SVGEmptyEditorClient() { }
+    virtual ~EmptyEditorClient() { }
     virtual void pageDestroyed() { }
-    
+
     virtual bool shouldDeleteRange(Range*) { return false; }
     virtual bool shouldShowDeleteInterface(HTMLElement*) { return false; }
-    virtual bool smartInsertDeleteEnabled() { return false; } 
+    virtual bool smartInsertDeleteEnabled() { return false; }
     virtual bool isContinuousSpellCheckingEnabled() { return false; }
     virtual void toggleContinuousSpellChecking() { }
     virtual bool isGrammarCheckingEnabled() { return false; }
     virtual void toggleGrammarChecking() { }
     virtual int spellCheckerDocumentTag() { return -1; }
-    
+
     virtual bool selectWordBeforeMenuEvent() { return false; }
     virtual bool isEditable() { return false; }
-    
+
     virtual bool shouldBeginEditing(Range*) { return false; }
     virtual bool shouldEndEditing(Range*) { return false; }
     virtual bool shouldInsertNode(Node*, Range*, EditorInsertAction) { return false; }
@@ -302,14 +304,14 @@ public:
     //  virtual void webViewDidChangeTypingStyle:(NSNotification *)notification { }
     //  virtual void webViewDidChangeSelection:(NSNotification *)notification { }
     //  virtual NSUndoManager* undoManagerForWebView:(WebView *)webView { return 0; }
-    
+
     virtual void registerCommandForUndo(PassRefPtr<EditCommand>) { }
     virtual void registerCommandForRedo(PassRefPtr<EditCommand>) { }
     virtual void clearUndoRedoOperations() { }
-    
+
     virtual bool canUndo() const { return false; }
     virtual bool canRedo() const { return false; }
-    
+
     virtual void undo() { }
     virtual void redo() { }
 
@@ -322,10 +324,10 @@ public:
     virtual bool doTextFieldCommandFromEvent(Element*, KeyboardEvent*) { return false; }
     virtual void textWillBeDeletedInTextField(Element*) { }
     virtual void textDidChangeInTextArea(Element*) { }
-    
+
 #if PLATFORM(MAC)
     virtual void markedTextAbandoned(Frame*) { }
-    
+
     virtual NSString* userVisibleString(NSURL*) { return 0; }
 #ifdef BUILDING_ON_TIGER
     virtual NSArray* pasteboardTypesForSelection(Frame*) { return 0; }
@@ -341,18 +343,18 @@ public:
     virtual bool spellingUIIsShowing() { return false; }
     virtual void getGuessesForWord(const String&, Vector<String>& guesses) { }
     virtual void setInputMethodState(bool enabled) { }
-  
-    
+
+
 };
 
-class SVGEmptyContextMenuClient : public ContextMenuClient {
+class EmptyContextMenuClient : public ContextMenuClient {
 public:
-    virtual ~SVGEmptyContextMenuClient() {  }
+    virtual ~EmptyContextMenuClient() {  }
     virtual void contextMenuDestroyed() { }
-    
+
     virtual PlatformMenuDescription getCustomMenuFromDefaultItems(ContextMenu*) { return 0; }
     virtual void contextMenuItemSelected(ContextMenuItem*, const ContextMenu*) { }
-    
+
     virtual void downloadURL(const KURL& url) { }
     virtual void copyImageToClipboard(const HitTestResult&) { }
     virtual void searchWithGoogle(const Frame*) { }
@@ -365,21 +367,21 @@ public:
 #endif
 };
 
-class SVGEmptyDragClient : public DragClient {
+class EmptyDragClient : public DragClient {
 public:
-    virtual ~SVGEmptyDragClient() {}
+    virtual ~EmptyDragClient() {}
     virtual void willPerformDragDestinationAction(DragDestinationAction, DragData*) { }
     virtual void willPerformDragSourceAction(DragSourceAction, const IntPoint&, Clipboard*) { }
     virtual DragDestinationAction actionMaskForDrag(DragData*) { return DragDestinationActionNone; }
     virtual DragSourceAction dragSourceActionMaskForPoint(const IntPoint&) { return DragSourceActionNone; }
     virtual void startDrag(DragImageRef, const IntPoint&, const IntPoint&, Clipboard*, Frame*, bool) { }
-    virtual DragImageRef createDragImageForLink(KURL&, const String& label, Frame*) { return 0; } 
+    virtual DragImageRef createDragImageForLink(KURL&, const String& label, Frame*) { return 0; }
     virtual void dragControllerDestroyed() { }
 };
 
-class SVGEmptyInspectorClient : public InspectorClient {
+class EmptyInspectorClient : public InspectorClient {
 public:
-    virtual ~SVGEmptyInspectorClient() {}
+    virtual ~EmptyInspectorClient() {}
 
     virtual void inspectorDestroyed() {};
 
@@ -397,10 +399,7 @@ public:
     virtual void hideHighlight() {};
     virtual void inspectedURLChanged(const String& newURL) {};
 };
-    
-}
 
-#endif // ENABLE(SVG)
-
-#endif // SVGImageEmptyClients_h
+}
 
+#endif // EmptyClients_h
index c8a4e5f085a37603f6281252e51d33d600f20c1e..713fb240f9e4ee8993ff2823f923d14e862579c7 100644 (file)
@@ -46,7 +46,7 @@
 #include "SVGSVGElement.h"
 #include "Settings.h"
 
-#include "SVGImageEmptyClients.h"
+#include "EmptyClients.h"
 
 namespace WebCore {
 
@@ -194,12 +194,12 @@ bool SVGImage::dataChanged(bool allDataReceived)
         return true;
     
     if (allDataReceived) {
-        static ChromeClient* dummyChromeClient = new SVGEmptyChromeClient;
-        static FrameLoaderClient* dummyFrameLoaderClient =  new SVGEmptyFrameLoaderClient;
-        static EditorClient* dummyEditorClient = new SVGEmptyEditorClient;
-        static ContextMenuClient* dummyContextMenuClient = new SVGEmptyContextMenuClient;
-        static DragClient* dummyDragClient = new SVGEmptyDragClient;
-        static InspectorClient* dummyInspectorClient = new SVGEmptyInspectorClient;
+        static ChromeClient* dummyChromeClient = new EmptyChromeClient;
+        static FrameLoaderClient* dummyFrameLoaderClient =  new EmptyFrameLoaderClient;
+        static EditorClient* dummyEditorClient = new EmptyEditorClient;
+        static ContextMenuClient* dummyContextMenuClient = new EmptyContextMenuClient;
+        static DragClient* dummyDragClient = new EmptyDragClient;
+        static InspectorClient* dummyInspectorClient = new EmptyInspectorClient;
 
         // FIXME: If this SVG ends up loading itself, we'll leak this Frame (and associated DOM & render trees).
         // The Cache code does not know about CachedImages holding Frames and won't know to break the cycle.