catapult-project / catapult

Deprecated Catapult GitHub. Please instead use http://crbug.com "Speed>Benchmarks" component for bugs and https://chromium.googlesource.com/catapult for downloading and editing source code..
https://chromium.googlesource.com/catapult
BSD 3-Clause "New" or "Revised" License
1.93k stars 563 forks source link

exynos parser falsely interprets the exynos_flip event #386

Open catapult-bot opened 9 years ago

catapult-bot commented 9 years ago

Issue by natduca Monday Sep 22, 2014 at 20:16 GMT Originally opened as https://github.com/google/trace-viewer/issues/386


From k...@google.com on August 08, 2013 03:05:06

What steps will reproduce the problem? 1. start chrome://tracing with system events

  1. run webgl "many planets deep" test
  2. capture a trace, see in the exynos_flip row

What is the expected output? Every swap should result an exynos_flip

What do you see instead? Some flips are missing. What version of the product are you using? On what operating system? version: 30.0.1585.0 OS: chrome OS Please provide any additional information below. I am working to add another parser in trace-viewer and corresponding events in the kernel. For the original bug, see http://crbug.com/264087

Original issue: http://code.google.com/p/trace-viewer/issues/detail?id=381

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:16 GMT


From dsinclair@chromium.org on August 08, 2013 20:07:20

From reading the other bug, am I correct in that this is actively being worked on? If so, I'll see about getting this bug correctly assigned to you.

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:16 GMT


From k...@google.com on August 08, 2013 20:36:28

Yes. It is me that is working on this bug. But I encounter a problem when trying to submit my patch. I use "git cl upload", but it keeps saying "Invalid username or password." Both my google.com and chromium.org account doesn't seem to work (with password generated by the settings page). Could you help me? Thanks.

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:16 GMT


From dsinclair@chromium.org on August 08, 2013 20:43:41

If you followed the directions on: https://code.google.com/p/trace-viewer/wiki/Contributing then you should be able to upload using your @chromium account. The password is the password you use to sign into code.google.com, not what you'd use for the chromium repo.

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:16 GMT


From dsinclair@chromium.org on August 08, 2013 20:46:35

Status: Assigned
Owner: k...@chromium.org

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:16 GMT


From k...@chromium.org on August 08, 2013 21:15:23

Thank you! I have uploaded the code. It's at https://codereview.appspot.com/12692044

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:16 GMT


From nd...@chromium.org on August 29, 2013 12:57:23

Labels: -Tool-All