Commit 63875916 authored by Antoine Labour's avatar Antoine Labour Committed by Commit Bot

Merge //gpu/config:crash_keys into //gpu/config

After crrev.com/c/812525, crash_keys doesn't need to be a separate
source_set duplicated in many components. Just make it part of the gpu
component.

Bug: 598854
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ibaa2e994087401f267a685e0a3c3c7a8f930656d
Reviewed-on: https://chromium-review.googlesource.com/849492Reviewed-by: default avatarKenneth Russell <kbr@chromium.org>
Commit-Queue: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526905}
parent fa5ed9f5
...@@ -4856,7 +4856,6 @@ if (is_android) { ...@@ -4856,7 +4856,6 @@ if (is_android) {
"//chrome/child", "//chrome/child",
"//components/crash/core/common", "//components/crash/core/common",
"//components/flags_ui:switches", "//components/flags_ui:switches",
"//gpu/config:crash_keys",
] ]
if (!is_fuchsia) { if (!is_fuchsia) {
# TODO(crbug.com/753619): Enable crash reporting on Fuchsia. # TODO(crbug.com/753619): Enable crash reporting on Fuchsia.
......
...@@ -18,17 +18,6 @@ group("config") { ...@@ -18,17 +18,6 @@ group("config") {
} }
} }
source_set("crash_keys") {
sources = [
"gpu_crash_keys.cc",
"gpu_crash_keys.h",
]
deps = [
"//components/crash/core/common:crash_key",
]
}
process_json_outputs = [ process_json_outputs = [
"$target_gen_dir/gpu_driver_bug_list_arrays_and_structs_autogen.h", "$target_gen_dir/gpu_driver_bug_list_arrays_and_structs_autogen.h",
"$target_gen_dir/gpu_driver_bug_list_autogen.cc", "$target_gen_dir/gpu_driver_bug_list_autogen.cc",
...@@ -85,6 +74,8 @@ source_set("config_sources") { ...@@ -85,6 +74,8 @@ source_set("config_sources") {
"gpu_blacklist.h", "gpu_blacklist.h",
"gpu_control_list.cc", "gpu_control_list.cc",
"gpu_control_list.h", "gpu_control_list.h",
"gpu_crash_keys.cc",
"gpu_crash_keys.h",
"gpu_driver_bug_list.cc", "gpu_driver_bug_list.cc",
"gpu_driver_bug_list.h", "gpu_driver_bug_list.h",
"gpu_driver_bug_workaround_type.h", "gpu_driver_bug_workaround_type.h",
...@@ -124,8 +115,11 @@ source_set("config_sources") { ...@@ -124,8 +115,11 @@ source_set("config_sources") {
configs += [ "//gpu:gpu_implementation" ] configs += [ "//gpu:gpu_implementation" ]
public_deps = [
"//components/crash/core/common:crash_key",
]
deps = [ deps = [
":crash_keys",
":process_json", ":process_json",
"//base", "//base",
"//third_party/re2", "//third_party/re2",
......
...@@ -7,25 +7,26 @@ ...@@ -7,25 +7,26 @@
#include "build/build_config.h" #include "build/build_config.h"
#include "components/crash/core/common/crash_key.h" #include "components/crash/core/common/crash_key.h"
#include "gpu/gpu_export.h"
namespace gpu { namespace gpu {
namespace crash_keys { namespace crash_keys {
// Keys that can be used for crash reporting. // Keys that can be used for crash reporting.
#if !defined(OS_ANDROID) #if !defined(OS_ANDROID)
extern crash_reporter::CrashKeyString<16> gpu_vendor_id; extern GPU_EXPORT crash_reporter::CrashKeyString<16> gpu_vendor_id;
extern crash_reporter::CrashKeyString<16> gpu_device_id; extern GPU_EXPORT crash_reporter::CrashKeyString<16> gpu_device_id;
#endif #endif
extern crash_reporter::CrashKeyString<64> gpu_driver_version; extern GPU_EXPORT crash_reporter::CrashKeyString<64> gpu_driver_version;
extern crash_reporter::CrashKeyString<16> gpu_pixel_shader_version; extern GPU_EXPORT crash_reporter::CrashKeyString<16> gpu_pixel_shader_version;
extern crash_reporter::CrashKeyString<16> gpu_vertex_shader_version; extern GPU_EXPORT crash_reporter::CrashKeyString<16> gpu_vertex_shader_version;
#if defined(OS_MACOSX) #if defined(OS_MACOSX)
extern crash_reporter::CrashKeyString<64> gpu_gl_version; extern GPU_EXPORT crash_reporter::CrashKeyString<64> gpu_gl_version;
#elif defined(OS_POSIX) #elif defined(OS_POSIX)
extern crash_reporter::CrashKeyString<256> gpu_vendor; extern GPU_EXPORT crash_reporter::CrashKeyString<256> gpu_vendor;
extern crash_reporter::CrashKeyString<128> gpu_renderer; extern GPU_EXPORT crash_reporter::CrashKeyString<128> gpu_renderer;
#endif #endif
extern crash_reporter::CrashKeyString<4> gpu_gl_context_is_virtual; extern GPU_EXPORT crash_reporter::CrashKeyString<4> gpu_gl_context_is_virtual;
} // namespace crash_keys } // namespace crash_keys
} // namespace gpu } // namespace gpu
......
...@@ -40,7 +40,6 @@ target(link_target_type, "command_buffer_sources") { ...@@ -40,7 +40,6 @@ target(link_target_type, "command_buffer_sources") {
"//gpu/command_buffer/service:gles2_sources", "//gpu/command_buffer/service:gles2_sources",
"//gpu/command_buffer/service:service_sources", "//gpu/command_buffer/service:service_sources",
"//gpu/config:config_sources", "//gpu/config:config_sources",
"//gpu/config:crash_keys",
"//gpu/ipc/client:ipc_client_sources", "//gpu/ipc/client:ipc_client_sources",
"//gpu/ipc/service:ipc_service_sources", "//gpu/ipc/service:ipc_service_sources",
"//ui/gfx", "//ui/gfx",
......
...@@ -78,7 +78,6 @@ target(link_target_type, "ipc_service_sources") { ...@@ -78,7 +78,6 @@ target(link_target_type, "ipc_service_sources") {
"//gpu/command_buffer/service:gles2_sources", "//gpu/command_buffer/service:gles2_sources",
"//gpu/command_buffer/service:service_sources", "//gpu/command_buffer/service:service_sources",
"//gpu/config:config_sources", "//gpu/config:config_sources",
"//gpu/config:crash_keys",
"//gpu/ipc/common:ipc_common_sources", "//gpu/ipc/common:ipc_common_sources",
] ]
libs = [] libs = []
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment