[Webkit-unassigned] [Bug 83085] New: [BlackBerry] Make the switch statement in WebPage::notifyAppActivationStateChange() stronger

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 3 15:57:10 PDT 2012


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

           Summary: [BlackBerry] Make the switch statement in
                    WebPage::notifyAppActivationStateChange() stronger
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit BlackBerry
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rwlbuis at gmail.com
                CC: tonikitoo at webkit.org


In WebPage::notifyAppActivationStateChange(), suggested by Dan Bates, we should
get rid of the default case to force a compiler error instead of using a
runtime error:

    for (; it != last; ++it) {
        switch (activationState) {
        case ActivationActive:
            (*it)->handleAppActivatedEvent();
            break;
        case ActivationInactive:
            (*it)->handleAppDeactivatedEvent();
            break;
        case ActivationStandby:
            (*it)->handleAppStandbyEvent();
            break;
        default:
            ASSERT(0);
            break;
        }
    }

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



More information about the webkit-unassigned mailing list