[Qt][WK2] Move code common to both ProxyAuthentication and Authentication context...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 14 Mar 2012 23:29:08 +0000 (23:29 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 14 Mar 2012 23:29:08 +0000 (23:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=80627

Patch by Dinu Jacob <dinu.jacob@nokia.com> on 2012-03-14
Reviewed by Kenneth Rohde Christiansen.

No impact to QML API.

* UIProcess/qt/QtDialogRunner.cpp:
(BaseAuthenticationContextObject):
(BaseAuthenticationContextObject::BaseAuthenticationContextObject):
(HttpAuthenticationDialogContextObject):
(HttpAuthenticationDialogContextObject::HttpAuthenticationDialogContextObject):
(HttpAuthenticationDialogContextObject::realm):
(ProxyAuthenticationDialogContextObject):
(ProxyAuthenticationDialogContextObject::ProxyAuthenticationDialogContextObject):
(QtDialogRunner::initForAuthentication):

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

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/qt/QtDialogRunner.cpp

index 8858a6c..eeb31ed 100644 (file)
@@ -1,3 +1,22 @@
+2012-03-14  Dinu Jacob  <dinu.jacob@nokia.com>
+
+        [Qt][WK2] Move code common to both ProxyAuthentication and Authentication context objects into a base class
+        https://bugs.webkit.org/show_bug.cgi?id=80627
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        No impact to QML API.
+
+        * UIProcess/qt/QtDialogRunner.cpp:
+        (BaseAuthenticationContextObject):
+        (BaseAuthenticationContextObject::BaseAuthenticationContextObject):
+        (HttpAuthenticationDialogContextObject):
+        (HttpAuthenticationDialogContextObject::HttpAuthenticationDialogContextObject):
+        (HttpAuthenticationDialogContextObject::realm):
+        (ProxyAuthenticationDialogContextObject):
+        (ProxyAuthenticationDialogContextObject::ProxyAuthenticationDialogContextObject):
+        (QtDialogRunner::initForAuthentication):
+
 2012-03-14  Anders Carlsson  <andersca@apple.com>
 
         With tiled drawing enabled, clicking a link to a PDF causes a cross-fade
index 16c6429..95bdc21 100644 (file)
@@ -69,23 +69,20 @@ private:
     QString m_defaultValue;
 };
 
-class AuthenticationDialogContextObject : public QObject {
+class BaseAuthenticationContextObject : public QObject {
     Q_OBJECT
     Q_PROPERTY(QString hostname READ hostname CONSTANT)
-    Q_PROPERTY(QString realm READ realm CONSTANT)
     Q_PROPERTY(QString prefilledUsername READ prefilledUsername CONSTANT)
 
 public:
-    AuthenticationDialogContextObject(const QString& hostname, const QString& realm, const QString& prefilledUsername)
+    BaseAuthenticationContextObject(const QString& hostname, const QString& prefilledUsername)
         : QObject()
         , m_hostname(hostname)
-        , m_realm(realm)
         , m_prefilledUsername(prefilledUsername)
     {
     }
 
     QString hostname() const { return m_hostname; }
-    QString realm() const { return m_realm; }
     QString prefilledUsername() const { return m_prefilledUsername; }
 
 public slots:
@@ -98,41 +95,41 @@ signals:
 
 private:
     QString m_hostname;
-    QString m_realm;
     QString m_prefilledUsername;
 };
 
-class ProxyAuthenticationDialogContextObject : public QObject {
+class HttpAuthenticationDialogContextObject : public BaseAuthenticationContextObject {
+    Q_OBJECT
+    Q_PROPERTY(QString realm READ realm CONSTANT)
+
+public:
+    HttpAuthenticationDialogContextObject(const QString& hostname, const QString& realm, const QString& prefilledUsername)
+        : BaseAuthenticationContextObject(hostname, prefilledUsername)
+        , m_realm(realm)
+    {
+    }
+
+    QString realm() const { return m_realm; }
+
+private:
+    QString m_realm;
+};
+
+class ProxyAuthenticationDialogContextObject : public BaseAuthenticationContextObject {
     Q_OBJECT
-    Q_PROPERTY(QString hostname READ hostname CONSTANT)
     Q_PROPERTY(quint16 port READ port CONSTANT)
-    Q_PROPERTY(QString prefilledUsername READ prefilledUsername CONSTANT)
 
 public:
     ProxyAuthenticationDialogContextObject(const QString& hostname, quint16 port, const QString& prefilledUsername)
-        : QObject()
-        , m_hostname(hostname)
+        : BaseAuthenticationContextObject(hostname, prefilledUsername)
         , m_port(port)
-        , m_prefilledUsername(prefilledUsername)
     {
     }
 
-    QString hostname() const { return m_hostname; }
     quint16 port() const { return m_port; }
-    QString prefilledUsername() const { return m_prefilledUsername; }
-
-public slots:
-    void accept(const QString& username, const QString& password) { emit accepted(username, password); }
-    void reject() { emit rejected(); }
-
-signals:
-    void accepted(const QString& username, const QString& password);
-    void rejected();
 
 private:
-    QString m_hostname;
     quint16 m_port;
-    QString m_prefilledUsername;
 };
 
 class CertificateVerificationDialogContextObject : public QObject {
@@ -196,7 +193,7 @@ bool QtDialogRunner::initForPrompt(QDeclarativeComponent* component, QQuickItem*
 
 bool QtDialogRunner::initForAuthentication(QDeclarativeComponent* component, QQuickItem* dialogParent, const QString& hostname, const QString& realm, const QString& prefilledUsername)
 {
-    AuthenticationDialogContextObject* contextObject = new AuthenticationDialogContextObject(hostname, realm, prefilledUsername);
+    HttpAuthenticationDialogContextObject* contextObject = new HttpAuthenticationDialogContextObject(hostname, realm, prefilledUsername);
     if (!createDialog(component, dialogParent, contextObject))
         return false;