Reviewed by Ken
authoradele <adele@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Oct 2004 21:35:16 +0000 (21:35 +0000)
committeradele <adele@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Oct 2004 21:35:16 +0000 (21:35 +0000)
This change makes these three functions virtual so that the work is being done in KWQHTMLPart
        instead of khtml_part, eliminating the need for the "#if APPLE_CHANGES" statements in the khtml code.

        * khtml/khtml_part.cpp:
        (KHTMLPart::shouldBeginEditing):
        (KHTMLPart::shouldEndEditing):
        (KHTMLPart::isContentEditable):
        * khtml/khtml_part.h:
        * kwq/KWQKHTMLPart.h:

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/khtml_part.cpp
WebCore/khtml/khtml_part.h
WebCore/kwq/KWQKHTMLPart.h

index 8e1a19b111b2f8c4080afa6700d21a43a9b1cd5d..f9a57adf70dc2274503dfbd437331ab7d2fb75b1 100644 (file)
@@ -1,3 +1,17 @@
+2004-10-14  Adele Amchan  <adele@apple.com>
+
+        Reviewed by Ken
+       
+       This change makes these three functions virtual so that the work is being done in KWQHTMLPart 
+        instead of khtml_part, eliminating the need for the "#if APPLE_CHANGES" statements in the khtml code.
+
+        * khtml/khtml_part.cpp:
+        (KHTMLPart::shouldBeginEditing):
+        (KHTMLPart::shouldEndEditing):
+        (KHTMLPart::isContentEditable):
+        * khtml/khtml_part.h:
+        * kwq/KWQKHTMLPart.h:
+
 2004-10-14  Ken Kocienda  <kocienda@apple.com>
 
         Reviewed by John
index 1fc9956568f271fa0430c28d6c66441eb84dda73..439e626aa51aba0566c4b9229d8e20399f3725be 100644 (file)
@@ -4954,29 +4954,17 @@ void KHTMLPart::selectAll()
 
 bool KHTMLPart::shouldBeginEditing(const Range &range) const
 {
-#if APPLE_CHANGES
-    return KWQ(this)->shouldBeginEditing(range);
-#else
     return true;
-#endif
 }
 
 bool KHTMLPart::shouldEndEditing(const Range &range) const
 {
-#if APPLE_CHANGES
-    return KWQ(this)->shouldEndEditing(range);
-#else
     return true;
-#endif
 }
 
 bool KHTMLPart::isContentEditable() const 
 {
-#if APPLE_CHANGES
-    return KWQ(this)->isContentEditable();
-#else
     return false;
-#endif
 }
 
 EditCommandPtr KHTMLPart::lastEditCommand()
index e2fb75ea341bf11b525967774a214ccaa609c54b..a0f2363529b833bc8cdabcc1ebd4e6ff2e7b8a5e 100644 (file)
@@ -669,17 +669,17 @@ public:
   /**
    * Returns whether editing should end in the given range
    */
-  bool shouldBeginEditing(const DOM::Range &) const;
+  virtual bool shouldBeginEditing(const DOM::Range &) const;
 
   /**
    * Returns whether editing should end in the given range
    */
-  bool shouldEndEditing(const DOM::Range &) const;
+  virtual bool shouldEndEditing(const DOM::Range &) const;
 
   /**
    * Returns the contentEditable "override" value for the part
    */
-  bool isContentEditable() const;
+  virtual bool isContentEditable() const;
 
   /**
    * Returns the most recent edit command applied.
index 9cb934ce99facba1cc1a39fc623576083ddbd689..57da465194e10d8ab2805feacefeda9ddbc20d15 100644 (file)
@@ -334,9 +334,9 @@ public:
     void issuePasteCommand();
     void respondToChangedSelection(const khtml::Selection &oldSelection, bool closeTyping);
     void respondToChangedContents();
-    bool isContentEditable() const;
-    bool shouldBeginEditing(const DOM::Range &) const;
-    bool shouldEndEditing(const DOM::Range &) const;
+    virtual bool isContentEditable() const;
+    virtual bool shouldBeginEditing(const DOM::Range &) const;
+    virtual bool shouldEndEditing(const DOM::Range &) const;
 
     KJS::Bindings::RootObject *executionContextForDOM();
     KJS::Bindings::RootObject *bindingRootObject();