[Webkit-unassigned] [Bug 68331] New: [chromium] frameContentAsPlainText shouldn't be using TextIterator

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Sep 18 20:46:18 PDT 2011


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

           Summary: [chromium] frameContentAsPlainText shouldn't be using
                    TextIterator
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rniwa at webkit.org
                CC: tony at chromium.org, tkent at chromium.org,
                    morrita at google.com, sullivan at chromium.org


We should avoid using TextIterator directly in WebKit layer code. Justification for not using plainText is that:
"        // The text iterator will walk nodes giving us text. This is similar to
        // the plainText() function in TextIterator.h, but we implement the maximum
        // size and also copy the results directly into a wstring, avoiding the
        // string conversion."

However, this comment is not true at all as the function currently takes Vector<UChar>.

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