[Webkit-unassigned] [Bug 48842] New: Move options handling in QtTestBrowser to separate OptionsHandler class

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Nov 2 09:17:22 PDT 2010


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

           Summary: Move options handling in QtTestBrowser to separate
                    OptionsHandler class
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebKit Qt
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: keith.kyzivat at nokia.com


Move options handling in QtTestBrowser to separate OptionsHandler class, in preparation for adding more options and reducing complexity in main.cpp

The options handling in QtTestBrowser is currently done all within main.cpp.  This is fine when the number of options to handle is low, but I am working on adding more command line options, and this will increase the complexity and confusion of QtTestBrowser's main.cpp.  This option handling can be moved to a separate OptionsHandler class for handling, so that options handling is isolated, and app startup is easier to read and maintain.

This is being done in preparation for another patch that I will be providing that expands the command line options to provide runtime configurability of more QWebSettings settings.

-- 
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