Bug 68331 – [chromium] frameContentAsPlainText shouldn't be using TextIterator

• Status: RESOLVED
• Resolution: WONTFIX
• Component: WebKit Misc.

[Ryosuke Niwa]

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

[Stephen Chenney]

https://code.google.com/p/chromium/issues/detail?id=230629