RESOLVED FIXED139034
Avoid String concatenation with line break iterator 
https://bugs.webkit.org/show_bug.cgi?id=139034
Summary Avoid String concatenation with line break iterator
Antti Koivisto
Reported 2014-11-24 14:44:57 PST
Use LazyLineBreakIterator::setPriorContext to move between segments.
Attachments
patch (13.90 KB, patch)
2014-11-24 14:53 PST, Antti Koivisto 		zalan: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Antti Koivisto
Comment 1 2014-11-24 14:53:43 PST
Created attachment 242174 [details] patch
alan
Comment 2 2014-11-24 15:36:00 PST
Comment on attachment 242174 [details] patch Looks good. r=me(provided there's no significant performance hit with the iterator change)
Antti Koivisto
Comment 3 2014-11-24 16:22:58 PST
https://trac.webkit.org/r176528
Antti Koivisto
Comment 4 2014-11-24 16:23:51 PST
line-layout-long-long-text.html went from mean: 80.41964673536093 runs/s median: 80.5499174535072 runs/s stdev: 0.6590877774334845 runs/s min: 78.8112513308298 runs/s max: 81.26653108623233 runs/s to mean: 87.01240050592219 runs/s median: 87.00959276489445 runs/s stdev: 0.38577332225810396 runs/s min: 86.46174424670227 runs/s max: 87.5463699183578 runs/s
Note You need to log in before you can comment on or make changes to this bug.