app_shell: Shut down in response to power button.

ui/compositor/test/in_process_context_factory.cc

Index: ui/compositor/test/in_process_context_factory.cc
diff --git a/ui/compositor/test/in_process_context_factory.cc b/ui/compositor/test/in_process_context_factory.cc
index fbc4c813cc0eee2536ebce2ed57f2ef73788d7cd..63050bb9b7cd4343b84599ad0f5401fe0b2e8794 100644
--- a/ui/compositor/test/in_process_context_factory.cc
+++ b/ui/compositor/test/in_process_context_factory.cc
@@ -20,7 +20,14 @@ namespace ui {
 
 InProcessContextFactory::InProcessContextFactory()
     : shared_bitmap_manager_(new cc::TestSharedBitmapManager()) {
+#if defined(UNIT_TEST)

[James Cook, 2014/08/23 00:55:00]

I'm not sure this works. In particular, I think only *_unittest.cc files (and
maybe some test support files?) are compiled with this flag. Otherwise the
entire project would have to be rebuilt to build the unit_tests binary.

"git grep UNIT_TEST" shows it only used in headers, which could be compiled
differently for the unit test binary due to inlining.

[Daniel Erat, 2014/08/23 02:21:31]

what you write makes sense. it worked when i tested it, but probably i just
happened to build the test first. i've changed this back to a DCHECK and tweaked
the message.

+  CHECK_NE(gfx::GetGLImplementation(), gfx::kGLImplementationNone)
+      << "Ensure that the test binary's main() is calling "
+      << "gfx::GLSurface::InitializeOneOffForTests()";
+#else
   DCHECK_NE(gfx::GetGLImplementation(), gfx::kGLImplementationNone);
+#endif
+
 #if defined(OS_CHROMEOS)
   bool use_thread = !CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kUIDisableThreadedCompositing);