On Linux rework driver_version/vendor extraction from gl version

gpu/config/gpu_info_collector_linux.cc

Index: gpu/config/gpu_info_collector_linux.cc
diff --git a/gpu/config/gpu_info_collector_linux.cc b/gpu/config/gpu_info_collector_linux.cc
index 050c33d246d391afb8edea0fbd0ed753fdd4c8ee..6c4f82c17df05616a01ad4242d9adf2024f4d2dd 100644
--- a/gpu/config/gpu_info_collector_linux.cc
+++ b/gpu/config/gpu_info_collector_linux.cc
@@ -247,8 +247,15 @@ CollectInfoResult CollectDriverInfoGL(GPUInfo* gpu_info) {
   DCHECK(gpu_info);
 
   std::string gl_version = gpu_info->gl_version;
-  if (base::StartsWith(gl_version, "OpenGL ES", base::CompareCase::SENSITIVE))
+  if (base::StartsWith(gl_version, "OpenGL ES", base::CompareCase::SENSITIVE)) {
     gl_version = gl_version.substr(10);
+  } else {
+    std::string core_profile = "(Core Profile)";

[Zhenyao Mo, 2016/07/21 13:30:44]

I guess this is ok, but a more generic fix is you always use the last piece
after SplitString for driver version instead of pieces[2].

[Julien Isorce Samsung, 2016/07/21 13:56:36]

Good idea, thx, and driver_vendor would be the last-1 piece, right ?

[Zhenyao Mo, 2016/07/21 13:59:47]

right

+    size_t position = gl_version.find(core_profile);
+    if (position != std::string::npos)
+      gl_version.erase(position, core_profile.length());
+  }
+
   std::vector<std::string> pieces = base::SplitString(
       gl_version, base::kWhitespaceASCII, base::KEEP_WHITESPACE,
       base::SPLIT_WANT_NONEMPTY);