Move GPU blacklist calculation to GPU proc

gpu/config/gpu_feature_info.h

Index: gpu/config/gpu_feature_info.h
diff --git a/gpu/config/gpu_feature_info.h b/gpu/config/gpu_feature_info.h
new file mode 100644
index 0000000000000000000000000000000000000000..a48679d65c4638cd1b1d359af5aa5bc7c4182027
--- /dev/null
+++ b/gpu/config/gpu_feature_info.h
@@ -0,0 +1,34 @@
+// Copyright (c) 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef GPU_CONFIG_GPU_FEATURE_INFO_H_
+#define GPU_CONFIG_GPU_FEATURE_INFO_H_
+
+#include <vector>
+
+#include "gpu/config/gpu_feature_type.h"
+
+namespace gpu {
+
+// Flags indicating the status of a GPU feature (see gpu_feature_type.h).
+enum GpuFeatureStatus {
+  kGpuFeatureStatusEnabled,
+  kGpuFeatureStatusBlacklisted,
+  kGpuFeatureStatusDisabled,
+  kGpuFeatureStatusUndefined,
+  kGpuFeatureStatusMax

[dcheng, 2017/02/07 00:58:22]

kGpuFeatureStatusMax = kGpuFeatureStatusUndefined

How many features does this thing have? Does it make sense to just skip the
vector and just make this an inline array to avoid the indirection through the
heap allocation?

[ericrk, 2017/02/08 21:29:46]

Makes sense - switched this to a fixed-size array.

+};
+
+// A vector of GpuFeatureStatus values, one per GpuFeatureType. By default, all
+// features are disabled.
+struct GpuFeatureInfo {
+  GpuFeatureInfo()
+      : status_values(NUMBER_OF_GPU_FEATURE_TYPES, kGpuFeatureStatusUndefined) {
+  }
+  std::vector<GpuFeatureStatus> status_values;
+};
+
+}  // namespace gpu
+
+#endif  // GPU_CONFIG_GPU_FEATURE_INFO_H_