New issue
Advanced search Search tips

Issue 845378 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Add test coverage on touch scrolling cases

Project Member Reported by yiyix@chromium.org, May 22 2018

Issue description

Add test coverage on touch scrolling cases: add some tests to measure the fps during scrolling for tough compositing web sites.
 

Comment 1 by yiyix@chromium.org, May 22 2018

Cc: vovoy@chromium.org cywang@chromium.org
Project Member

Comment 2 by bugdroid1@chromium.org, May 30 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/30a3ed662ad005f9c1deb627abfbfe321437cb29

commit 30a3ed662ad005f9c1deb627abfbfe321437cb29
Author: Yi Xu <yiyix@chromium.org>
Date: Wed May 30 19:50:52 2018

Remove fps_meter ebuild files

This patch is a part of a series of patches. The goal is to move
the fps_meter to autotest library and update the ebuild file so
that it is added to the /usr/local/bin and it can be used as a
tool accordingly. In this particular patch, fps_meter ebuild
files are removed.

BUG=chromium:845378
TEST=Manual tested

Change-Id: Ia0900ae6313875e527772a86d240065fea579148
Reviewed-on: https://chromium-review.googlesource.com/1074781
Commit-Ready: Yi Xu <yiyix@chromium.org>
Tested-by: Chung-yih Wang <cywang@chromium.org>
Reviewed-by: Hsu-Cheng Tsai <hctsai@chromium.org>

[delete] https://crrev.com/2f154f0425e13f64c241dc1dd0e4a14df09300c7/dev-util/fps_meter/fps_meter-0.0.1-r4.ebuild
[delete] https://crrev.com/2f154f0425e13f64c241dc1dd0e4a14df09300c7/dev-util/fps_meter/fps_meter-9999.ebuild
[modify] https://crrev.com/30a3ed662ad005f9c1deb627abfbfe321437cb29/virtual/target-chromium-os-test/target-chromium-os-test-1.ebuild
[rename] https://crrev.com/30a3ed662ad005f9c1deb627abfbfe321437cb29/virtual/target-chromium-os-test/target-chromium-os-test-1-r68.ebuild

Project Member

Comment 3 by bugdroid1@chromium.org, Jun 2 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/c91893309444eadaac499ed4fa61d646567d0b49

commit c91893309444eadaac499ed4fa61d646567d0b49
Author: Yi Xu <yiyix@chromium.org>
Date: Sat Jun 02 00:44:42 2018

Add fps_meter to autotest library

This patch is a part of a series of patches. The goal is to move
the fps_meter to autotest library and update the ebuild file so
that it is added to the /usr/local/bin and it can be used as a
tool accordingly. In this particular patch, fps_meter is added to
the autotest library.

BUG=chromium:845378
TEST=Manual tested

Change-Id: Id2d96e6997193a2db035407744aa6a91608f77c8
Reviewed-on: https://chromium-review.googlesource.com/1072971
Commit-Ready: Yi Xu <yiyix@chromium.org>
Tested-by: Yi Xu <yiyix@chromium.org>
Reviewed-by: Chung-yih Wang <cywang@chromium.org>
Reviewed-by: Yi Xu <yiyix@chromium.org>

[add] https://crrev.com/c91893309444eadaac499ed4fa61d646567d0b49/client/bin/fps_meter.py

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 2 2018

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

commit bcc00ca58726d9cf774c56005715a5ee330da045
Author: Yi Xu <yiyix@chromium.org>
Date: Sat Jun 02 21:43:56 2018

Remove fps_meter from tool library

This patch is a part of a series of patches. The goal is to move
the fps_meter to autotest library and update the ebuild file so
that it is added to the /usr/local/bin and it can be used as a
tool accordingly. In this particular patch, the fps_meter is
removed from the tool library.

BUG=chromium:845378
TEST=Manual tested
CQ-DEPEND=CL:1074781

Change-Id: Ib7ef74fc088cba5ce65dc5b8f1e361a9b63fdb39
Reviewed-on: https://chromium-review.googlesource.com/1073350
Commit-Ready: Yi Xu <yiyix@chromium.org>
Tested-by: Chung-yih Wang <cywang@chromium.org>
Reviewed-by: Chung-yih Wang <cywang@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[delete] https://crrev.com/71e236cd2eb8aa7aef05aa0d63c42c21f490c50d/fps_meter/fps_meter.py

Project Member

Comment 5 by bugdroid1@chromium.org, Jul 10

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/b5d9a840d950bdbc0658ef093fc836d05a48bad4

commit b5d9a840d950bdbc0658ef093fc836d05a48bad4
Author: Yi Xu <yiyix@chromium.org>
Date: Tue Jul 10 00:59:27 2018

Add autotest coverage on scrolling cases

2 test cases are added that simulate user scrolling on tough compositing websites
to measure fps changes.

BUG=chromium:845378

TEST=platform_ScrollTest, platform_MouseScrollTest

Change-Id: I5f562f8b483ea21791cdad84e874c3bee932cec1
Reviewed-on: https://chromium-review.googlesource.com/1068637
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Yi Xu <yiyix@chromium.org>
Reviewed-by: Chung-yih Wang <cywang@chromium.org>

[add] https://crrev.com/b5d9a840d950bdbc0658ef093fc836d05a48bad4/client/site_tests/platform_MouseScrollTest/mouse.prop
[add] https://crrev.com/b5d9a840d950bdbc0658ef093fc836d05a48bad4/client/site_tests/platform_MouseScrollTest/mouse_event
[add] https://crrev.com/b5d9a840d950bdbc0658ef093fc836d05a48bad4/client/site_tests/platform_ScrollTest/control
[add] https://crrev.com/b5d9a840d950bdbc0658ef093fc836d05a48bad4/client/site_tests/platform_ScrollTest/platform_ScrollTest.py
[add] https://crrev.com/b5d9a840d950bdbc0658ef093fc836d05a48bad4/client/site_tests/platform_MouseScrollTest/control
[add] https://crrev.com/b5d9a840d950bdbc0658ef093fc836d05a48bad4/client/site_tests/platform_MouseScrollTest/platform_MouseScrollTest.py
[modify] https://crrev.com/b5d9a840d950bdbc0658ef093fc836d05a48bad4/client/cros/touch_playback_test_base.py

Project Member

Comment 6 by bugdroid1@chromium.org, Jul 10

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/a76ec2aab31f77831030fe88b75692e943f35957

commit a76ec2aab31f77831030fe88b75692e943f35957
Author: Yi Xu <yiyix@chromium.org>
Date: Tue Jul 10 05:35:44 2018

Add tests to autotest-chrome target

Add the newly added tests cases to autotest-chrome target.

BUG=chromium:845378
CQ-DEPEND=CL:1068637
TEST=platform_ScrollTest, platform_MouseScrollTest

Change-Id: Ife24863e23aa1193c18082d2f3ed92608d51b528
Reviewed-on: https://chromium-review.googlesource.com/1074575
Commit-Ready: Yi Xu <yiyix@chromium.org>
Tested-by: Yi Xu <yiyix@chromium.org>
Reviewed-by: Yi Xu <yiyix@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/a76ec2aab31f77831030fe88b75692e943f35957/chromeos-base/autotest-chrome/autotest-chrome-9999.ebuild

Sign in to add a comment