Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(273)

Issue 468223002: Revert 289154 "Revert 289067 "Use RE string pattern matching for..." (Closed)

Created:
6 years, 4 months ago by oshima
Modified:
6 years, 4 months ago
Reviewers:
oshima
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 289154 "Revert 289067 "Use RE string pattern matching for..." > Revert 289067 "Use RE string pattern matching for blacklist stri..." > > Reason for revert: uninitialize memory access > > http://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20Mac%20%28valgrind%29%282%29/builds/29469/steps/memory%20test%3A%20content/logs/stdio > > > Use RE string pattern matching for blacklist strings. > > > > Including cpu_brand, gl_vendor, gl_renderer, gl_extension, driver_vendor, > > and machine_model_name. > > > > BUG=396578 > > TBR=piman@chromium.org > > TEST=gpu_unittests > > > > Review URL: https://codereview.chromium.org/452293002 > > TBR=zmo@chromium.org > > Review URL: https://codereview.chromium.org/465163002 TBR=oshima@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=289308

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+299 lines, -563 lines) Patch
M trunk/src/content/browser/gpu/gpu_data_manager_impl_private_unittest.cc View 2 chunks +2 lines, -8 lines 0 comments Download
M trunk/src/gpu/BUILD.gn View 1 chunk +0 lines, -1 line 0 comments Download
M trunk/src/gpu/config/gpu_control_list.h View 4 chunks +10 lines, -41 lines 0 comments Download
M trunk/src/gpu/config/gpu_control_list.cc View 14 chunks +44 lines, -103 lines 0 comments Download
M trunk/src/gpu/config/gpu_control_list_entry_unittest.cc View 9 chunks +166 lines, -34 lines 0 comments Download
M trunk/src/gpu/config/gpu_control_list_format.txt View 3 chunks +9 lines, -11 lines 0 comments Download
D trunk/src/gpu/config/gpu_control_list_string_info_unittest.cc View 1 chunk +0 lines, -98 lines 0 comments Download
M trunk/src/gpu/config/gpu_control_list_unittest.cc View 1 chunk +1 line, -4 lines 0 comments Download
M trunk/src/gpu/config/gpu_driver_bug_list_json.cc View 33 chunks +38 lines, -149 lines 0 comments Download
M trunk/src/gpu/config/software_rendering_list_json.cc View 20 chunks +29 lines, -113 lines 0 comments Download
M trunk/src/gpu/gpu.gyp View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
oshima
6 years, 4 months ago (2014-08-13 16:13:10 UTC) #1
oshima
6 years, 4 months ago (2014-08-13 16:13:36 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r289308.

Powered by Google App Engine
This is Rietveld 408576698