New issue
Advanced search Search tips

Issue 786605 link

Starred by 1 user
Status: Fixed
Owner:
kainino@chromium.org
Closed: Jan 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression

Blocked on:
issue 772177


Sign in to add a comment

GPU list urls broken on ChromeOS

Project Member Reported by kainino@chromium.org, Nov 18 2017 
In about:gpu on my Chromebook Pixel 2013:

Software rendering list URL	https://chromium.googlesource.com/chromium/src/+/0/gpu/config/software_rendering_list.json
Driver bug list URL	https://chromium.googlesource.com/chromium/src/+/0/gpu/config/gpu_driver_bug_list.json

Similarly in about:version:

Google Chrome	64.0.3270.0 (Official Build) canary (64-bit)
Revision	0
Platform	10137.0.0 (Official Build) canary-channel link

Not sure if revision being 0 is a bug with the build, or just how it works in ChromeOS. I don't have a non-canary device to check against.

Comment 1 by kainino@chromium.org, Dec 18 2017

Summary: GPU list urls broken on ChromeOS (was: GPU list urls broken on ChromeOS (Canary at least))
This also manifests on Beta, so I think it's just how it works in ChromeOS.

I think we'll have to solve this by using the x.x.x.x version instead of the Revision in the URL generation. It works just as well on all of the official releases, just not on continuous builds or developer builds. If we do that only on ChromeOS, it should be no problem at all.

Comment 2 by kainino@chromium.org, Dec 29 2017

Status: Started (was: Assigned)
Project Member

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

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

commit 0867089c08e4fa4920829d5796b1a01115182bea
Author: Kai Ninomiya <kainino@chromium.org>
Date: Tue Jan 02 18:17:24 2018

about:gpu: fix blacklist urls on CrOS, and add "copy to clipboard" button

Makes a few updates to about:gpu.

* Add a "Copy to Clipboard" button, and removes the "To properly save
  this page" instructions. (797440)
* Fixes blacklist URLs on CrOS: fall back to the 0.0.0.0 version
  identifier whenever the 40-char hash is not available. (786605)
* Print the date in ISO format, rather than localized format, so that
  Chrome developers will be able to read it consistently. (no bug)

Bug:  786605 
Bug:  797440 
Change-Id: I6ab25b19248c49e8ba16bedad5060a48dc16b8b8
Reviewed-on: https://chromium-review.googlesource.com/846283
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Kai Ninomiya <kainino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526505}
[modify] https://crrev.com/0867089c08e4fa4920829d5796b1a01115182bea/content/browser/resources/gpu/info_view.css
[modify] https://crrev.com/0867089c08e4fa4920829d5796b1a01115182bea/content/browser/resources/gpu/info_view.html
[modify] https://crrev.com/0867089c08e4fa4920829d5796b1a01115182bea/content/browser/resources/gpu/info_view.js

Comment 4 by kainino@google.com, Jan 2 2018

Status: Fixed (was: Started)
►

Sign in to add a comment