comments on API changes that we'd like to do.
authorhausmann <hausmann@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Nov 2007 14:23:14 +0000 (14:23 +0000)
committerhausmann <hausmann@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Nov 2007 14:23:14 +0000 (14:23 +0000)
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@27528 268f45cc-cd09-0410-ab3c-d52691b4dbfc

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

index c0508b1f4550bee5756c7acb55e40d7c6acd77af..f1b731cb9cb3cf64caf13cdab9486c736f8f0be3 100644 (file)
@@ -75,31 +75,33 @@ public:
 
     QWebFrame *mainFrame() const;
 
+    // ### should return a pointer
     QWebPageHistory history() const;
 
+    // ### owned by webpage, returns a pointer, no setter!
     void setSettings(const QWebSettings &settings);
     QWebSettings settings() const;
 
     QSize sizeHint() const;
 
+    // ### should be windowTitle
     QString title() const;
-
     QUrl url() const;
+    // ### should be QWidget::icon
+    QPixmap icon() const;
 
     bool isModified() const;
-
     QUndoStack *undoStack();
     
     void setNetworkInterface(QWebNetworkInterface *interface);
     QWebNetworkInterface *networkInterface() const;
 
-    QPixmap icon() const;
-
 #ifndef QT_NO_NETWORKPROXY
     void setNetworkProxy(const QNetworkProxy& proxy);
     QNetworkProxy networkProxy() const;
 #endif
 
+    // #### remove after adding editor actions
     bool canCut() const;
     bool canCopy() const;
     bool canPaste() const;
@@ -113,15 +115,18 @@ public slots:
      */
     void stop();
 
+    // #### should these be actions?
     void goBack();
     void goForward();
     void goToHistoryItem(const QWebHistoryItem &item);
 
+    // ### should be a signal: void geometryChangeRequest(const QRect &geom);
     virtual void setWindowGeometry(const QRect& geom);
 
     void cut();
     void copy();
     void paste();
+    // ### should we have execCommand() or something similar?
 
 signals:
     /**
@@ -162,6 +167,7 @@ signals:
      */
     void iconLoaded();
 
+    // #### doesn't give you anything without a way to actually get the selection
     void selectionChanged();
 
     /**
@@ -174,6 +180,7 @@ signals:
     void frameCreated(QWebFrame *frame);
 
 private slots:
+    // ### should go!
     void onLoadProgressChanged(int);
 
 protected:
@@ -182,11 +189,15 @@ protected:
     virtual QObject *createPlugin(const QString &classid, const QUrl &url, const QStringList &paramNames, const QStringList &paramValues);
 
     virtual NavigationRequestResponse navigationRequested(QWebFrame *frame, const QWebNetworkRequest &request, NavigationType type);
+    // ### what does this do?
     virtual QString chooseFile(QWebFrame *frame, const QString& oldFile);
+    // ### can we unify the next 3 methods? 
     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);
     virtual void javaScriptConsoleMessage(const QString& message, unsigned int lineNumber, const QString& sourceID);
+
+    // ### should the be here?
     virtual QString userAgentStringForUrl(const QUrl& forUrl) const;
 
     virtual void resizeEvent(QResizeEvent*);
index d1d7ec830ed0fb7c9cecaf193e7a66cb68866fbd..8dddf5385c42ab0ef2f4f727abd16957d79608d5 100644 (file)
@@ -1,3 +1,11 @@
+2007-11-07  Lars Knoll  <lars@trolltech.com>
+
+        Reviewed by Simon.
+
+        comments on API changes that we'd like to do.
+
+        * Api/qwebpage.h:
+
 2007-11-07  Simon Hausmann  <hausmann@kde.org>
 
         Reviewed by Lars Knoll <lars@trolltech.com>.