New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 806176 link

Starred by 1 user
Status: Fixed
Owner:
█ youcheng@chromium.org
Last visit > 30 days ago
Closed: Jan 2018
Cc:
█ petershih@chromium.org
chromeos-factory-eng@google.com
█ pihsun@chromium.org
hungte@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Feature



Sign in to add a comment

Factory: Speed up stylus test.

Project Member Reported by youcheng@chromium.org, Jan 26 2018 
Currently, the Python back-end of our stylus pytest forwards all stylus events to Goofy front-end, one by one.
This might be too slow on some platforms and operators won't be able to see the feedback (what they draw) in realtime.

What we could do:
1. Reduce the latency of forwarding events to front-end.
2. Set an upper bound of event report rate. (For those styluses with high event report rate)
3. Buffer the events and send multiple events to front-end together.
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 27 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/factory/+/b583158a2fa277e939e200b81ccd78f2cb6ce01c

commit b583158a2fa277e939e200b81ccd78f2cb6ce01c
Author: You-Cheng Syu <youcheng@chromium.org>
Date: Sat Jan 27 07:10:17 2018

stylus: Buffer events and flush buffer periodically.

In order to speed up stylus and make operators able to see the feedback
in realtime, we buffer the stylus events and flush events periodically
to reduce the latency from CallJSFunction.

BUG= chromium:806176 
TEST=manually

Change-Id: I400d06698fba007f43b0059770b25f0ef04c1ed5
Reviewed-on: https://chromium-review.googlesource.com/888232
Commit-Ready: Youcheng Syu <youcheng@chromium.org>
Tested-by: Youcheng Syu <youcheng@chromium.org>
Reviewed-by: Shen-En Shih <petershih@chromium.org>

[modify] https://crrev.com/b583158a2fa277e939e200b81ccd78f2cb6ce01c/py/test/pytests/stylus.py
[modify] https://crrev.com/b583158a2fa277e939e200b81ccd78f2cb6ce01c/py/test/pytests/stylus_static/stylus.js

Comment 2 by youcheng@chromium.org, Jan 27 2018

Status: Fixed (was: Started)
►

Sign in to add a comment