Add optimization for CHROMIUM_subscribe_uniform extension.

gpu/command_buffer/service/context_group_unittest.cc

Index: gpu/command_buffer/service/context_group_unittest.cc
diff --git a/gpu/command_buffer/service/context_group_unittest.cc b/gpu/command_buffer/service/context_group_unittest.cc
index 98e8e3c8108f1d7a9f8f1bc45dd46501e4b016b9..afd343d169de931df47805ecd11c474176780172 100644
--- a/gpu/command_buffer/service/context_group_unittest.cc
+++ b/gpu/command_buffer/service/context_group_unittest.cc
@@ -11,6 +11,7 @@
 #include "gpu/command_buffer/service/mailbox_manager.h"
 #include "gpu/command_buffer/service/test_helper.h"
 #include "gpu/command_buffer/service/texture_manager.h"
+#include "gpu/command_buffer/service/valuebuffer_manager.h"

[bajones, 2014/12/04 23:23:14]

Is this include necessary?

[orglofch, 2014/12/04 23:37:10]

Yep, it's passing a SubscriptionRefset (albeit a NULL one) to the ContextGroup,
which is defined in valuebuffer_manager.h and requires the definition since it's
being passed as a scoped_refptr.

 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gl/gl_mock.h"
 
@@ -39,8 +40,8 @@ class ContextGroupTest : public GpuServiceTest {
   void SetUp() override {
     GpuServiceTest::SetUp();
     decoder_.reset(new MockGLES2Decoder());
-    group_ = scoped_refptr<ContextGroup>(
-        new ContextGroup(NULL, NULL, NULL, NULL, NULL, kBindGeneratesResource));
+    group_ = scoped_refptr<ContextGroup>(new ContextGroup(
+        NULL, NULL, NULL, NULL, NULL, NULL, kBindGeneratesResource));
   }
 
   scoped_ptr<MockGLES2Decoder> decoder_;