[Webkit-unassigned] [Bug 54976] New: Incorrect switch-case fall-through inside InspectorClientQt.cpp

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=54976

           Summary: Incorrect switch-case fall-through inside
                    InspectorClientQt.cpp
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Trivial
          Priority: P2
         Component: WebKit Qt
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: fabrizio.machado at nokia.com


In function variantToSetting(), code falls through to next case if qvariant.type() is QVariant::Bool.

This does not make sense since the QVariant::Bool case sets QString retval to "true" or "false".

Falling through and executing the block under QVariant::String case would overwrite this value.  The correct behavior is to break from the switch.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.