ui: Allow individual test suites to opt-out of the TestCompositor.

ui/gl/gl_surface.h

Index: ui/gl/gl_surface.h
diff --git a/ui/gl/gl_surface.h b/ui/gl/gl_surface.h
index f7a02f042834247f9e125dc263db6d221029423b..ffdc3ce230b6c5c508b4cdfbc03c8817182f0b18 100644
--- a/ui/gl/gl_surface.h
+++ b/ui/gl/gl_surface.h
@@ -75,7 +75,7 @@ class GL_EXPORT GLSurface : public base::RefCounted<GLSurface> {
 
   // Unit tests should call these instead of InitializeOneOff() to set up
   // GL bindings appropriate for tests.
-  static void InitializeOneOffForTests();
+  static void InitializeOneOffForTests(bool disable_drawing = false);

[no sievers, 2014/02/21 19:51:49]

I thought the default param was against the style guide.

[danakj, 2014/02/21 19:56:58]

Ya kinda. But it's just for a bit, in test suites I'll
InitializeOneOffForTests(true) to opt them in. Once everyone is (true) I can
remove the argument.

The alternative is add another method InitializeOneOffForTestsWithNullContexts,
and then go back and delete the method. I was trying to keep the churn down for
a temporary hack for our tests, but it's going to hit all the same lines of code
either way. But makes more more churn here in gl_surface, as I would have to
split this method up and have both methods call a 3rd internal method which then
just calls a 4th internal method. Meh, defer to your judgement on it :)

[Ken Russell (switch to Gerrit), 2014/02/21 20:03:37]

It's fine as a temporary workaround, but please add a TODO about removing it.

[danakj, 2014/02/21 20:20:54]

Thanks, good plan. Done!

   static void InitializeOneOffWithMockBindingsForTests();
   static void InitializeDynamicMockBindingsForTests(GLContext* context);