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

Issue 751814 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Task

Blocking:
issue 722935



Sign in to add a comment

Rename all tracing categories from cc to viz

Project Member Reported by weiliangc@chromium.org, Aug 2 2017

Issue description

As part of moving code to cc the name of the tracing categories should be renamed to viz.

This should be done at the end of move.
 

Comment 1 by danakj@chromium.org, May 25 2018

Owner: danakj@chromium.org
Status: Started (was: Available)
Project Member

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

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

commit 0cc06c998193b91dac6c89675202a95ee01da936
Author: danakj <danakj@chromium.org>
Date: Fri May 25 22:14:49 2018

Rename tracing categories in viz from "cc" to "viz".

The "cc" traces in the component become "viz".
The "cc.debug.overdraw" becomes "viz.overdraw".
The "cc.debug.triangles" becomes "viz.triangles".
The "cc.debug.quads" becomes "viz.quads". There is one instance of this
in cc/ as well, but renamed it to be consistent. The idea is to trace
quads that arrive in viz and include the quads from the cc origin, so
I *think* this makes sense.

R=weiliangc@chromium.org

Bug:  751814 
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I6b25ac8e86ffe8a2bbefc0824929e96eb12be104
Reviewed-on: https://chromium-review.googlesource.com/1073889
Reviewed-by: weiliangc <weiliangc@chromium.org>
Commit-Queue: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#562037}
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/cc/tiles/frame_viewer_instrumentation.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/cc/trees/layer_tree_host_impl.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/common/quads/render_pass.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/common/quads/shared_quad_state.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/common/skia_helper.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/service/display/direct_renderer.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/service/display/display_resource_provider.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/service/display/display_scheduler.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/service/display/gl_renderer.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/service/display/skia_renderer.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/service/surfaces/surface.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/service/surfaces/surface_manager.cc
[modify] https://crrev.com/0cc06c998193b91dac6c89675202a95ee01da936/components/viz/test/ordered_simple_task_runner.cc

Comment 3 by danakj@chromium.org, May 25 2018

Status: Fixed (was: Started)

Sign in to add a comment