Added some more comments to the API after another round of API review with Lars.
authorhausmann <hausmann@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Nov 2007 14:31:22 +0000 (14:31 +0000)
committerhausmann <hausmann@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Nov 2007 14:31:22 +0000 (14:31 +0000)
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@27554 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/qt/Api/qwebpage.h
WebKit/qt/ChangeLog

index dfa3cf6f98985626fa96afb4f7d99f433a6ac21d..c4f349d79886ec5eda7ed4cf81be478ffef692df 100644 (file)
@@ -140,18 +140,17 @@ public:
 
     QSize sizeHint() const;
 
-    // ### should be windowTitle
     QString title() const;
     QUrl url() const;
-    // ### should be QWidget::icon
     QPixmap icon() const;
 
     bool isModified() const;
     QUndoStack *undoStack() const;
-    
+
     void setNetworkInterface(QWebNetworkInterface *interface);
     QWebNetworkInterface *networkInterface() const;
 
+    // #### why is this in the page itself?
 #ifndef QT_NO_NETWORKPROXY
     void setNetworkProxy(const QNetworkProxy& proxy);
     QNetworkProxy networkProxy() const;
@@ -186,7 +185,6 @@ public slots:
     void cut();
     void copy();
     void paste();
-    // ### should we have execCommand() or something similar?
 
 signals:
     /**
@@ -234,6 +232,7 @@ signals:
      */
     void initialLayoutComplete();
 
+    // ### call addedToHistory instead, something more signal'ish
     void addToHistory(const QUrl&);
 
     void frameCreated(QWebFrame *frame);
@@ -246,9 +245,9 @@ protected:
     virtual void webActionTriggered(WebAction action, bool checked = false);
 
     virtual NavigationRequestResponse navigationRequested(QWebFrame *frame, const QWebNetworkRequest &request, NavigationType type);
-    // ### what does this do?
+    // ### do we need the frame? maybe include "file dialog" in the name
     virtual QString chooseFile(QWebFrame *frame, const QString& oldFile);
-    // ### can we unify the next 3 methods? 
+    // ### can we unify the next 3 methods? do we need the frame?
     virtual void javaScriptAlert(QWebFrame *frame, const QString& msg);
     virtual bool javaScriptConfirm(QWebFrame *frame, const QString& msg);
     virtual bool javaScriptPrompt(QWebFrame *frame, const QString& msg, const QString& defaultValue, QString* result);
index 03af3c50f092afb1ea3938b96cd9e692662678de..f5e096c66a1f4d381f7f57098e864f1d95a94c4c 100644 (file)
@@ -1,3 +1,11 @@
+2007-11-07  Simon Hausmann  <hausmann@kde.org>
+
+        Reviewed by Lars.
+
+        Added some more comments to the API after another round of API review with Lars.
+
+        * Api/qwebpage.h:
+
 2007-11-07  Simon Hausmann  <hausmann@kde.org>
 
         Reviewed by Lars.