[webkit-reviews] review granted: [Bug 101689] [BlackBerry] remove obsolete tokenizer settings : [Attachment 173173] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Nov 9 07:26:18 PST 2012


Yong Li <yoli at rim.com> has granted George Staikos <staikos at kde.org>'s request
for review:
Bug 101689: [BlackBerry] remove obsolete tokenizer settings
https://bugs.webkit.org/show_bug.cgi?id=101689

Attachment 173173: Patch
https://bugs.webkit.org/attachment.cgi?id=173173&action=review

------- Additional Comments from Yong Li <yoli at rim.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=173173&action=review


>> Source/WebKit/blackberry/Api/WebPage.cpp:-575
>> -	m_page->setCustomHTMLTokenizerTimeDelay(0.3);
> 
> These are still in use :
> 
>
http://trac.webkit.org/browser/trunk/Source/WebCore/html/parser/HTMLParserSched
uler.cpp
> 
> Does this mean the default behavior is fine for us?

I think so. I am surprised by the number "256". I believe it hurts...


More information about the webkit-reviews mailing list