2009-06-10 Antonio Gomes <antonio.gomes@openbossa.org>
authorariya@webkit.org <ariya@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 Jun 2009 14:48:42 +0000 (14:48 +0000)
committerariya@webkit.org <ariya@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 Jun 2009 14:48:42 +0000 (14:48 +0000)
        Reviewed by Ariya Hidayat.

        Documented ResolveRule and StylePriority enum values as well as their use in
        QWebElement::styleProperty and QWebElement::setStyleProperty methods.

        Based on the work of Simon Hausmann.

        * Api/qwebelement.cpp:
        (QWebElement::styleProperty):
        (QWebElement::setStyleProperty):
        * Api/qwebelement.h:

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

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

index 3c80085..ff0463b 100644 (file)
@@ -878,6 +878,34 @@ QStringList QWebElement::scriptableProperties() const
 }
 
 /*!
+    \enum QWebElement::ResolveRule
+    \since 4.6
+
+    This enum describes how QWebElement's styleProperty resolves the given
+    property name.
+
+    \value IgnoreCascadingStyles Return the property value as it is defined
+    in the element, without respecting style inheritance and other CSS rules.
+    \value RespectCascadingStyles The property's value is determined using
+    the inheritance and importance rules defined in the document's stylesheet.
+*/
+
+/*!
+    \enum QWebElement::StylePriority
+    \since 4.6
+
+    This enum describes the priority newly set CSS properties should have when
+    set using QWebElement::setStyleProperty().
+
+    \value NormalStylePriority Define the property without important
+    priority even if "!important" is explicitly set in \a value.
+    \value DeclaredStylePriority Define the property respecting the
+    priority specified in \a value.
+    \value ImportantStylePriority Define the property to have
+    an important priority, this is equal to appending "!important" to the value.
+*/
+
+/*!
     Returns the value of the style named \a name or an empty string if such one
     does not exist.
 
@@ -896,7 +924,7 @@ QStringList QWebElement::scriptableProperties() const
     highest priority, unless other declarations also use the !important
     declaration, in which the last !important declaration takes predecence.
 */
-QString QWebElement::styleProperty(const QString &name, const ResolveRule rule) const
+QString QWebElement::styleProperty(const QString &name, ResolveRule rule) const
 {
     if (!m_element || !m_element->isStyledElement())
         return QString();
@@ -958,9 +986,13 @@ QString QWebElement::styleProperty(const QString &name, const ResolveRule rule)
     This syntax is supported when using DeclaredStylePriority as \a priority.
 
     Using NormalStylePriority as \a priority, the property will have normal
-    priority, and any "!important" declaration will be ignored.
+    priority, and any "!important" declaration will be ignored. On the other
+    hand, using ImportantStylePriority sets the important priority even when
+    not explicit passed in \a value.
+    By using DeclaredStylePriority as \a priority the property will respect the
+    priority specified in \a value.
 */
-void QWebElement::setStyleProperty(const QString &name, const QString &value, const StylePriority priority)
+void QWebElement::setStyleProperty(const QString &name, const QString &value, StylePriority priority)
 {
     if (!m_element || !m_element->isStyledElement())
         return;
index 644a293..7e56d0f 100644 (file)
@@ -131,10 +131,10 @@ public:
     QStringList scriptableProperties() const;
 
     enum ResolveRule { IgnoreCascadingStyles, RespectCascadingStyles };
-    QString styleProperty(const QString &name, const ResolveRule = IgnoreCascadingStyles) const;
+    QString styleProperty(const QString &name, ResolveRule = IgnoreCascadingStyles) const;
 
     enum StylePriority { NormalStylePriority, DeclaredStylePriority, ImportantStylePriority };
-    void setStyleProperty(const QString &name, const QString &value, const StylePriority = DeclaredStylePriority);
+    void setStyleProperty(const QString &name, const QString &value, StylePriority = DeclaredStylePriority);
 
     QString computedStyleProperty(const QString &name) const;
 
index e614f27..08dcefe 100644 (file)
@@ -1,3 +1,17 @@
+2009-06-10  Antonio Gomes   <antonio.gomes@openbossa.org>
+
+        Reviewed by Ariya Hidayat.
+
+        Documented ResolveRule and StylePriority enum values as well as their use in
+        QWebElement::styleProperty and QWebElement::setStyleProperty methods.
+
+        Based on the work of Simon Hausmann.
+
+        * Api/qwebelement.cpp:
+        (QWebElement::styleProperty):
+        (QWebElement::setStyleProperty):
+        * Api/qwebelement.h:
+
 2009-06-09  Ariya Hidayat  <ariya.hidayat@nokia.com>
 
         Rubber-stamped by Simon Hausmann.