Use RE string pattern matching for blacklist strings.

gpu/config/gpu_control_list_format.txt

Index: gpu/config/gpu_control_list_format.txt
diff --git a/gpu/config/gpu_control_list_format.txt b/gpu/config/gpu_control_list_format.txt
index 71c569c8397eb30fcd6cd31b0f3ffb67e0b4e85a..c897cb6fd37c51738050f96d88d639bd463bdc4e 100644
--- a/gpu/config/gpu_control_list_format.txt
+++ b/gpu/config/gpu_control_list_format.txt
@@ -32,7 +32,7 @@
 // 6. "multi_gpu_category" is a string, valid values include "any", "primary",
 //    "secondary", and "active". If unspecified, the default value is "primary".
 //    See gpu_control_list.h for more details on the meanings of the strings.
-// 7. "driver_vendor" is a STRING structure (defined below).
+// 7. "driver_vendor" is a string pattern.
 // 8. "driver_version" is a VERSION structure (defined below).
 // 9. "driver_date" is a VERSION structure (defined below).
 //    The version is interpreted as "year.month.day".
@@ -41,17 +41,16 @@
 //    The default value on Android is "gles", on Windows is "angle", on other
 //    platforms is "gl".
 // 11. "gl_version" is a VERSION structure (defined below).
-// 12. "gl_vendor" is a STRING structure (defined below).
-// 13. "gl_renderer" is a STRING structure (defined below).
-// 14. "gl_extensions" is a STRING structure (defined below).
+// 12. "gl_vendor" is a string pattern.
+// 13. "gl_renderer" is a string pattern.
+// 14. "gl_extensions" is a string pattern.
 // 15. "perf_graphics" is a FLOAT structure (defined below).
 // 16. "perf_gaming" is a FLOAT structure (defined below).
 // 17. "perf_overall" is a FLOAT structure (defined below).
-// 18. "machine_model_name" is an array of strings.  The strings can contain
-//     any characters.
+// 18. "machine_model_name" is an array of string patterns.
 // 19. "machine_model_version" is a VERSION structure (defined below).
 // 20. "gpu_count" is a INT structure (defined below).
-// 21  "cpu_info" is a STRING structure (defined below).
+// 21  "cpu_info" is a string pattern.
 // 22. "exceptions" is a list of entries.
 // 23. "features" is a list of gpu control list options, which can be
 //     configured by a specific list. See its *_json.cc file for a list of
@@ -74,12 +73,11 @@
 // Only "driver_version" supports lexical style if the format is major.minor;
 // in that case, major is still numerical, but minor is lexical. 
 //
-// STRING includes "op" and "value".  "op" can be any of the following values:
-// "contains", "beginwith", "endwith", "=".  "value" is a string.
-//
 // FLOAT includes "op" "value", and "value2".  "op" can be any of the
 // following values: "=", "<", "<=", ">", ">=", "any", "between".  "value2" is
 // only used if "op" is "between".  "value" is used for all "op" values except
 // "any". "value" and "value2" are valid float numbers.
 // INT is very much like FLOAT, except that the values need to be integers.
-
+//
+// String pattern syntax can be found at
+// https://code.google.com/p/re2/wiki/Syntax.

[Ken Russell (switch to Gerrit), 2014/08/09 00:57:02]

As mentioned above, a tighter definition of how the matching is done is needed.
I'm pretty sure it isn't correct in all cases to simply convert a regexp pattern
to lower case via ASCII rules. At a minimum, it needs to be documented that both
the string pattern, and the value it matches against, are converted to lower
case. Perhaps it would be better to just use re's functionality for doing
case-insensitive matches. This would mean that all of the regexps would need to
be prefixed with (I think)

  (?i:re)

This would certainly make the various blacklists more verbose, so it's your
decision what to do here.

[Zhenyao Mo, 2014/08/09 01:23:59]

I think allow the full power of RE is a better option. However, we don't need to
set all entries to be case insensitive.  I think many of them should be
case-sensitive matching.  Only one needs to be case insensitive.