[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