Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(526)

Issue 135510043: Added touch point input visualization

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 2 months ago by Vicențiu Ciorbaru
Modified:
10 years, 8 months ago
Reviewers:
nduca, Sami Kyöstilä
CC:
trace-viewer-review_googlegroups.com
Base URL:
http://trace-viewer.googlecode.com/svn/trunk
Visibility:
Public.

Description

Added touch point input visualization

Patch Set 1 #

Total comments: 5
Unified diffs Side-by-side diffs Delta from patch set Stats (+47 lines, -0 lines) Patch
M trace_viewer/cc/layer_tree_impl.html View 2 chunks +30 lines, -0 lines 5 comments Download
M trace_viewer/cc/layer_tree_quad_stack_view.html View 1 chunk +17 lines, -0 lines 0 comments Download

Messages

Total messages: 4
Vicențiu Ciorbaru
The first draft in getting touch input visualization.
11 years, 2 months ago (2014-09-05 17:08:37 UTC) #1
nduca
code structing comments but lg when sami is happy https://codereview.appspot.com/135510043/diff/1/trace_viewer/cc/layer_tree_impl.html File trace_viewer/cc/layer_tree_impl.html (right): https://codereview.appspot.com/135510043/diff/1/trace_viewer/cc/layer_tree_impl.html#newcode19 trace_viewer/cc/layer_tree_impl.html:19: ...
11 years, 2 months ago (2014-09-05 19:22:12 UTC) #2
dsinclair
https://codereview.appspot.com/135510043/diff/1/trace_viewer/cc/layer_tree_impl.html File trace_viewer/cc/layer_tree_impl.html (right): https://codereview.appspot.com/135510043/diff/1/trace_viewer/cc/layer_tree_impl.html#newcode57 trace_viewer/cc/layer_tree_impl.html:57: if (process.findAllThreadsNamed(BROWSER_PROCESS_NAME).length != 0) nit: !== https://codereview.appspot.com/135510043/diff/1/trace_viewer/cc/layer_tree_impl.html#newcode65 trace_viewer/cc/layer_tree_impl.html:65: if ...
11 years, 1 month ago (2014-09-08 21:17:35 UTC) #3
dsinclair
10 years, 8 months ago (2015-03-07 15:20:42 UTC) #4
 
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b