[Webkit-unassigned] [Bug 25911] Apply href in base elements to anchors shown on the source viewer
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue May 26 23:39:14 PDT 2009
https://bugs.webkit.org/show_bug.cgi?id=25911
------- Comment #11 from tyoshino at google.com 2009-05-26 23:39 PDT -------
(In reply to comment #9)
> (From update of attachment 30644 [review])
> > + if (token->tagName == baseTag && equalIgnoringCase(attr->name().localName(), "href")) {
>
> I'm not sure why the code in this class checks attribute names in this
> inefficient way. In normal WebCore code we would just compare with hrefAttr
> using == instead.
>
> r=me
>
Thank you. After your comment, I investigated and understood the reason. In
order to show HTML sources as is on the view-source window, HTMLTokenizer
doesn't lower names when inViewSourceMode() is true. So, we have to lower names
before comparison in HTMLViewSourceDocument class. Patch revision 2 was
comparing tagName and baseTag case sensitively, so I updated to revision 3.
Please take a look again.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
More information about the webkit-unassigned
mailing list