2009-04-15 Antonio Gomes <antonio.gomes@openbossa.org>
authorariya@webkit.org <ariya@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 15 Apr 2009 15:09:55 +0000 (15:09 +0000)
committerariya@webkit.org <ariya@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 15 Apr 2009 15:09:55 +0000 (15:09 +0000)
        Reviewed by Ariya Hidayat.

        Rename extend() method in QWebElementSelection to append().

        * Api/qwebelement.cpp:
        (QWebElementSelection::append):
        * Api/qwebelement.h:
        * tests/qwebelement/tst_qwebelement.cpp:
        (tst_QWebElement::appendSelection):

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

WebKit/qt/Api/qwebelement.cpp
WebKit/qt/Api/qwebelement.h
WebKit/qt/ChangeLog
WebKit/qt/tests/qwebelement/tst_qwebelement.cpp

index 899ecc4..53005de 100644 (file)
@@ -1311,7 +1311,7 @@ QWebElementSelection::~QWebElementSelection()
     Appends the items of the \a other list to this list and returns a
     reference to this list.
 
-    \sa operator+(), extend()
+    \sa operator+(), append()
 */
 
 /*!
@@ -1332,7 +1332,7 @@ QWebElementSelection QWebElementSelection::operator+(const QWebElementSelection
 
     \sa operator+=()
 */
-void QWebElementSelection::extend(const QWebElementSelection &other)
+void QWebElementSelection::append(const QWebElementSelection &other)
 {
     if (!d) {
         *this = other;
index 62289c6..3fd20ac 100644 (file)
@@ -151,10 +151,10 @@ public:
     QWebElementSelection operator+(const QWebElementSelection &other) const;
     inline QWebElementSelection &operator+=(const QWebElementSelection &other)
     {
-        extend(other); return *this;
+        append(other); return *this;
     }
 
-    void extend(const QWebElementSelection &selection);
+    void append(const QWebElementSelection &selection);
 
     int count() const;
     QWebElement at(int i) const;
index 7632c05..9e3a7d8 100644 (file)
@@ -1,5 +1,17 @@
 2009-04-15  Antonio Gomes   <antonio.gomes@openbossa.org>
 
+        Reviewed by Ariya Hidayat.
+
+        Rename extend() method in QWebElementSelection to append().
+
+        * Api/qwebelement.cpp:
+        (QWebElementSelection::append):
+        * Api/qwebelement.h:
+        * tests/qwebelement/tst_qwebelement.cpp:
+        (tst_QWebElement::appendSelection):
+
+2009-04-15  Antonio Gomes   <antonio.gomes@openbossa.org>
+
         Reviewed by Simon Hausmann.
 
         Fixed nit/typo in QWebElement documentation.
index ec9e28d..e517765 100644 (file)
@@ -79,7 +79,7 @@ private slots:
     void frame();
     void emptySelection();
     void style();
-    void extendSelection();
+    void appendSelection();
     void properties();
     void appendAndPrepend();
     void insertBeforeAndAfter();
@@ -432,7 +432,7 @@ void tst_QWebElement::style()
     QCOMPARE(p.styleProperty("cursor"), QLatin1String("auto"));
 }
 
-void tst_QWebElement::extendSelection()
+void tst_QWebElement::appendSelection()
 {
     QString html = "<body><span class='a'>aaa</span><p>first para</p><div>foo</div>"
         "<span class='b'>bbb</span><p>second para</p><div>bar</div></body>";
@@ -442,7 +442,7 @@ void tst_QWebElement::extendSelection()
     QWebElementSelection selection = body.findAll("p");
     QCOMPARE(selection.count(), 2);
 
-    selection.extend(body.findAll("div"));
+    selection.append(body.findAll("div"));
     QCOMPARE(selection.count(), 4);
 
     selection += body.findAll("span.a");
@@ -457,9 +457,9 @@ void tst_QWebElement::extendSelection()
 
     QCOMPARE(selection.count(), 5);
     QWebElementSelection test;
-    test.extend(selection);
+    test.append(selection);
     QCOMPARE(test.count(), 5);
-    test.extend(QWebElementSelection());
+    test.append(QWebElementSelection());
     QCOMPARE(test.count(), 5);
 }