Changeset 79412 in webkit

Timestamp:

Feb 22, 2011 11:13:55 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2011-02-22 Fabrizio Machado <​fabrizio.machado@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Don't fall through case in variantToSetting() if qvariant.type() is Bool
​https://bugs.webkit.org/show_bug.cgi?id=54976

Test not needed.

• WebCoreSupport/InspectorClientQt.cpp:

Location:

trunk/Source/WebKit/qt

Files:

• ChangeLog (modified) (1 diff)
• WebCoreSupport/InspectorClientQt.cpp (modified) (1 diff)

trunk/Source/WebKit/qt/ChangeLog

@@ +1 @@
+2011-02-22  Fabrizio Machado  <fabrizio.machado@nokia.com>
+
+        Reviewed by Laszlo Gombos.
+
+        [Qt] Don't fall through case in variantToSetting() if qvariant.type() is Bool
+        https://bugs.webkit.org/show_bug.cgi?id=54976
+
+        Test not needed.
+        
+        * WebCoreSupport/InspectorClientQt.cpp:
+
 2011-02-22  Alexis Menard  <alexis.menard@openbossa.org>
 

trunk/Source/WebKit/qt/WebCoreSupport/InspectorClientQt.cpp

@@ -153 +153 @@
         case QVariant::Bool:
             retVal = qvariant.toBool() ? "true" : "false";
+            break;
         case QVariant::String:
             retVal = qvariant.toString();
+            break;
         default:
             break;