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

Unified Diff: gpu/command_buffer/service/feature_info.cc

Issue 1750123002: Upgrade Tex{Sub}Image{2|3}D to handle ES3 unpack parameters. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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
Index: gpu/command_buffer/service/feature_info.cc
diff --git a/gpu/command_buffer/service/feature_info.cc b/gpu/command_buffer/service/feature_info.cc
index 73fb0eab56f3fa92067fc43f413084c80d019646..e0005e7ceb99465f6b96303834336b44da38cd32 100644
--- a/gpu/command_buffer/service/feature_info.cc
+++ b/gpu/command_buffer/service/feature_info.cc
@@ -388,10 +388,9 @@ void FeatureInfo::InitializeFeatures() {
// Pre es3, there are no PBOS and all unpack state is handled in client side.
// With es3, unpack state is needed in server side. We always mark these
// enums as valid and pass them to drivers only when a valid PBO is bound.
+ // UNPACK_ROW_LENGTH, UNPACK_SKIP_ROWS, and UNPACK_SKIP_PIXELS are enabled,
+ // but there is no need to add them to pixel_store validtor.
AddExtensionString("GL_EXT_unpack_subimage");
- validators_.pixel_store.AddValue(GL_UNPACK_ROW_LENGTH);
- validators_.pixel_store.AddValue(GL_UNPACK_SKIP_ROWS);
- validators_.pixel_store.AddValue(GL_UNPACK_SKIP_PIXELS);
if (feature_flags_.enable_subscribe_uniform) {
AddExtensionString("GL_CHROMIUM_subscribe_uniform");

Powered by Google App Engine
This is Rietveld 408576698