Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(609)

Issue 1910593002: Fix InlineTextBox::characterWidths() not to measure every substring (Closed)

Created:
4 years, 8 months ago by kojii
Modified:
4 years, 8 months ago
Reviewers:
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@2704
Target Ref:
refs/pending/branch-heads/2704
Project:
chromium
Visibility:
Public.

Description

Fix InlineTextBox::characterWidths() not to measure every substring This patch changes InlineTextBox::characterWidths() to use Font::individualCharacterRanges() rather than to measure every substring of InlineTextBox. Measuring every substring without context is very expensive and also is not correct. When dev tools have long strings without spaces such as minified JS and accessbility is turned on, the measuring of substrings can almost hang it up. Currently, WebAXObject::characterOffsets() is the only user of this function. BUG=593679, 568032 TEST=Existing tests cover, such as AutomationApiTest.BoundsForRange Review URL: https://codereview.chromium.org/1897853004 Cr-Commit-Position: refs/heads/master@{#388162} (cherry picked from commit 11f1c680cca2250e01121b9bddd7bc6f8fb28a96) Committed: https://chromium.googlesource.com/chromium/src/+/5b23d300baadb87a84e02b9e22546893858be9c3

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -9 lines) Patch
M third_party/WebKit/Source/core/layout/line/InlineTextBox.cpp View 2 chunks +8 lines, -9 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
kojii
4 years, 8 months ago (2016-04-20 19:11:46 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
5b23d300baadb87a84e02b9e22546893858be9c3.

Powered by Google App Engine
This is Rietveld 408576698