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

Unified Diff: ui/gl/gl_version_info.cc

Issue 895853003: Update from https://crrev.com/314320 (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ui/gl/gl_version_info.h ('k') | ui/gl/gl_wgl_api_implementation.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/gl/gl_version_info.cc
diff --git a/ui/gl/gl_version_info.cc b/ui/gl/gl_version_info.cc
index 0d052b58dc62a62766a903b7be1d4653f9190fe0..e2aa0deab7997c1cf585371a3052d26704c6b7bc 100644
--- a/ui/gl/gl_version_info.cc
+++ b/ui/gl/gl_version_info.cc
@@ -4,35 +4,35 @@
#include "ui/gl/gl_version_info.h"
+#include "base/strings/string_number_conversions.h"
+#include "base/strings/string_tokenizer.h"
#include "base/strings/string_util.h"
namespace gfx {
GLVersionInfo::GLVersionInfo(const char* version_str, const char* renderer_str)
: is_es(false),
- is_es1(false),
- is_es2(false),
- is_es3(false),
- is_gl1(false),
- is_gl2(false),
- is_gl3(false),
- is_gl4(false),
- is_angle(false) {
+ is_angle(false),
+ major_version(0),
+ minor_version(0),
+ is_es3(false) {
if (version_str) {
std::string lstr(base::StringToLowerASCII(std::string(version_str)));
- is_es = (lstr.substr(0, 9) == "opengl es");
- if (is_es) {
- is_es1 = (lstr.substr(9, 2) == "-c" && lstr.substr(13, 2) == "1.");
- is_es2 = (lstr.substr(9, 3) == " 2.");
- is_es3 = (lstr.substr(9, 3) == " 3.");
- } else {
- is_gl2 = (lstr.substr(0, 2) == "2.");
- is_gl3 = (lstr.substr(0, 2) == "3.");
- is_gl4 = (lstr.substr(0, 2) == "4.");
- // In early GL versions, GetString output format is implementation
- // dependent.
- is_gl1 = !is_gl2 && !is_gl3 && !is_gl4;
+ is_es = (lstr.length() > 12) && (lstr.substr(0, 9) == "opengl es");
+ if (is_es)
+ lstr = lstr.substr(10, 3);
+ base::StringTokenizer tokenizer(lstr.begin(), lstr.end(), ".");
+ unsigned major, minor;
+ if (tokenizer.GetNext() &&
+ base::StringToUint(tokenizer.token_piece(), &major)) {
+ major_version = major;
+ if (tokenizer.GetNext() &&
+ base::StringToUint(tokenizer.token_piece(), &minor)) {
+ minor_version = minor;
+ }
}
+ if (is_es && major_version == 3)
+ is_es3 = true;
}
if (renderer_str) {
is_angle = StartsWithASCII(renderer_str, "ANGLE", true);
« no previous file with comments | « ui/gl/gl_version_info.h ('k') | ui/gl/gl_wgl_api_implementation.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698