[Webkit-unassigned] [Bug 38900] New: Refactoring HTMLLinkElement::tokenizeRelAttribute so its signature doesn't change so much

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue May 11 05:51:07 PDT 2010


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

           Summary: Refactoring HTMLLinkElement::tokenizeRelAttribute so
                    its signature doesn't change so much
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Mac OS X 10.5
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: HTML DOM
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: leonclarke at google.com
                CC: android-webkit-unforking at google.com


This is a bit of refactoring as part of providing support for link prefetching
The prefetching bug is here https://bugs.webkit.org/show_bug.cgi?id=3652

tokienizeRelAttribute takes a long list of bools indicating if each of the possible values of rel are present. This means that if support for some attributes is implemented as optional features, the function will have a different signature for every combination of features. In addition, the call to it in the preloader needs to change whenever an extra rel type is supported. In android, we have 1 un-upstreamed rel type, and I was adding prefetch as another.

A more elegant approach is to have a simple structure containing all the possible things the rel attribute could be, and pass this in. The corresponding set of variables in the object also become an instance of the new struct.

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