New issue
Advanced search Search tips

Issue 721490 link

Starred by 1 user
Status: Archived
Owner: ----
Closed: Aug 20
Cc:
xiaoche...@chromium.org
yosin@chromium.org
editing-dev@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 3
Type: Task



Sign in to add a comment

Understand, and if possible, optimize the request frequency of idle time spell checker

Project Member Reported by xiaoche...@chromium.org, May 11 2017 
We need to understand how aggressive the current spell checker and idle time spell checker make spellcheck requests.

For practical use, requests don't need to be made so frequently. It should never go more frequent than one request per word, and should be even slower in fast typing.

Not marking this as a blocking issue of the launching of idle time spell checker because:
1. We don't know if this is a real issue
2. This one is orthogonal to the main purpose, to improve input latency and reduce pipeline jam, of idle time spell checker
Project Member

Comment 1 by bugdroid1@chromium.org, May 12 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6ddb331240ae137b7e74ec611fd7caa4114fdea4

commit 6ddb331240ae137b7e74ec611fd7caa4114fdea4
Author: xiaochengh <xiaochengh@chromium.org>
Date: Fri May 12 22:24:49 2017

Add histogram for Blink's spell check request interval

This patch adds a histogram for Blink's spell check request interval,
so that we can understand how aggressive Blink's spellchecker is, and
decide if any optimization is needed.

BUG= 721490 

Review-Url: https://codereview.chromium.org/2871313002
Cr-Commit-Position: refs/heads/master@{#471466}

[modify] https://crrev.com/6ddb331240ae137b7e74ec611fd7caa4114fdea4/third_party/WebKit/Source/core/editing/spellcheck/SpellCheckRequester.cpp
[modify] https://crrev.com/6ddb331240ae137b7e74ec611fd7caa4114fdea4/third_party/WebKit/Source/core/editing/spellcheck/SpellCheckRequester.h
[modify] https://crrev.com/6ddb331240ae137b7e74ec611fd7caa4114fdea4/tools/metrics/histograms/histograms.xml

Comment 2 by xiaoche...@chromium.org, Aug 19 2017

Cc: xiaoche...@chromium.org
Owner: ----
Status: Available (was: Assigned)
While the necessity of this task still remains open, the priority seems pretty low for now. Hence unassigning myself.
Project Member

Comment 3 by sheriffbot@chromium.org, Aug 20

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 4 by xiaoche...@chromium.org, Aug 20

Status: Archived (was: Untriaged)
►

Sign in to add a comment