2009-01-31 Darin Adler <darin@apple.com>
authordarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 1 Feb 2009 00:55:03 +0000 (00:55 +0000)
committerdarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 1 Feb 2009 00:55:03 +0000 (00:55 +0000)
        Fix Mac build.

        * loader/FrameLoader.cpp: Moved FrameLoaderClient::hasHTMLView in here.
        We need it to not be an inline. Sadly, FrameLoaderClient.cpp was prematurely
        deleted and I don't want to bring it back just to fix the build.

        * loader/FrameLoaderClient.h: Rearrange virtual functions and corrected
        a possibly-confusing comment.

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

WebCore/ChangeLog
WebCore/loader/FrameLoader.cpp
WebCore/loader/FrameLoaderClient.h

index 2c01f8e..45fd183 100644 (file)
@@ -1,3 +1,14 @@
+2009-01-31  Darin Adler  <darin@apple.com>
+
+        Fix Mac build.
+
+        * loader/FrameLoader.cpp: Moved FrameLoaderClient::hasHTMLView in here.
+        We need it to not be an inline. Sadly, FrameLoaderClient.cpp was prematurely
+        deleted and I don't want to bring it back just to fix the build.
+
+        * loader/FrameLoaderClient.h: Rearrange virtual functions and corrected
+        a possibly-confusing comment.
+
 2009-01-31  Adam Treat  <adam.treat@torchmobile.com>
 
         Reviewed by David Hyatt.
index f99f565..1edd368 100644 (file)
@@ -5428,4 +5428,9 @@ void FrameLoader::switchOutLowBandwidthDisplayIfReady()
 
 #endif
 
+bool FrameLoaderClient::hasHTMLView() const
+{
+    return true;
+}
+
 } // namespace WebCore
index fc0e7ca..ec5ca5e 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.
+ * Copyright (C) 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -60,27 +60,28 @@ namespace WebCore {
     class ResourceError;
     class ResourceHandle;
     class ResourceLoader;
+    class ResourceRequest;
     class ResourceResponse;
     class SharedBuffer;
     class SubstituteData;
     class String;
     class Widget;
 
-    class ResourceRequest;
-
     typedef void (FrameLoader::*FramePolicyFunction)(PolicyAction);
 
     class FrameLoaderClient {
     public:
-        virtual void frameLoaderDestroyed() = 0;
+        // An inline function cannot be the first non-abstract virtual function declared
+        // in the class as it results in the vtable being generated as a weak symbol.
+        // This hurts performance (in Mac OS X at least, when loadig frameworks), so we
+        // don't want to do it in WebKit.
+        virtual bool hasHTMLView() const;
 
-        // The inline virtual destructor cannot be the first virtual function declared
-        // in the class as it results in the vtable being generated as a weak symbol
-        virtual ~FrameLoaderClient() {}
+        virtual ~FrameLoaderClient() { }
 
-        virtual bool hasWebView() const = 0; // mainly for assertions
+        virtual void frameLoaderDestroyed() = 0;
 
-        virtual bool hasHTMLView() const { return true; }
+        virtual bool hasWebView() const = 0; // mainly for assertions
 
         virtual void makeRepresentation(DocumentLoader*) = 0;
         virtual void forceLayout() = 0;