Comment #2 on issue 23285 by [email protected]: Implement  
PlainTextController for test shell
http://code.google.com/p/chromium/issues/detail?id=23285

I uploaded a first implementation of a PlainTextController class to

   http://codereview.chromium.org/243032

However, as described there, this is not complete, at least not for full  
compatibility:

.) It does not test whether the passed-in object is indeed a range
.) It uses the standard toString() method rather than the range's  
(non-standard) text() method,
which would use TextIterator.

I haven't yet found a good way to implement/circumvent these shortcomings.  
OTOH, if actually using
TextIterator is not a primary goal with these tests, then just  
re-baselining the layout test with
the returned text output may be an option.

Any further comments and insights on this are very much welcome!


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to