Move paint aggregator and the completion callback factory.

ppapi/tests/test_flash.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ppapi/tests/test_flash.h"
 
 #include "ppapi/c/pp_macros.h"
 #include "ppapi/c/private/ppb_flash.h"
+#include "ppapi/cpp/completion_callback.h"

[noelallen1, 2012/01/03 23:44:34]

completion_callback_factory?

I think this compiles because completion_callback_factory is included by
test_flash.h.   Shouldn't this be completion_callback_factory since you are
using a factory bellow?

 #include "ppapi/cpp/instance.h"
 #include "ppapi/cpp/module.h"
 #include "ppapi/cpp/var.h"
 #include "ppapi/tests/testing_instance.h"
 
 REGISTER_TEST_CASE(Flash);
 
 using pp::Var;
 
 TestFlash::TestFlash(TestingInstance* instance)
@@ skipping 76 matching lines @@
              flash_interface_->GetCommandLineArgs(
                  pp::Module::Get()->pp_module()));
   ASSERT_TRUE(result.is_string());
 
   PASS();
 }
 
 void TestFlash::QuitMessageLoopTask(int32_t) {
   flash_interface_->QuitMessageLoop(instance_->pp_instance());
 }