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

Issue 643538 link

Starred by 1 user
Status: Fixed
Owner:
█ jbau...@chromium.org
not on Chrome anymore
Closed: Sep 2016
Cc:
█ skaslev@chromium.org
█ jbau...@chromium.org
skare@chromium.org
isherman@chromium.org
asvitk...@chromium.org
█ rkaplow@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug

Blocking:
issue 640921



Sign in to add a comment

Histogram GPU.GPU.GPUProcessSoftwareRendering is unmapped

Project Member Reported by ricea@chromium.org, Sep 2 2016 
The histogram 'GPU.GPU.GPUProcessSoftwareRendering' is present in Chromium
source code but does not appear in histograms.xml. This means that data is
collected for the histogram but nothing useful is done with it.

The histogram was defined in content/browser/gpu/gpu_process_host.cc at line
1035.

https://cs.chromium.org/chromium/src/content/browser/gpu/gpu_process_host.cc?l=1035

It may have moved by the time you read this.

Please remove the histogram 'GPU.GPU.GPUProcessSoftwareRendering' from the
source code. If it was very recently added, it may be worth adding it to
histograms.xml instead, but probably not.

This bug was automatically assigned based on git blame information. If you
are not the correct assignee for this bug, please delete the histogram
anyway.
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 2 2016 

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

commit b80245fdf918e044c00da35261257193739d681e
Author: jbauman <jbauman@chromium.org>
Date: Fri Sep 02 23:02:36 2016

Fix name of GPU.GPUProcessSoftwareRendering histogram.

The histograms.xml file has the correct name, but apparently the source
code has always been reporting under an incorrect name.

BUG= 643538 

Review-Url: https://codereview.chromium.org/2302193003
Cr-Commit-Position: refs/heads/master@{#416378}

[modify] https://crrev.com/b80245fdf918e044c00da35261257193739d681e/content/browser/gpu/gpu_process_host.cc

Comment 2 by jbau...@chromium.org, Sep 7 2016

Owner: jbau...@chromium.org
Status: Fixed (was: Available)
►

Sign in to add a comment