| Index: cc/raster/raster_buffer_provider_unittest.cc
|
| diff --git a/cc/raster/raster_buffer_provider_unittest.cc b/cc/raster/raster_buffer_provider_unittest.cc
|
| index a2aeeebe62e1417e7437a2b8b1a84e63b0d017a9..0c18dde9ab9963073d956aa316784a2b9a6bfff7 100644
|
| --- a/cc/raster/raster_buffer_provider_unittest.cc
|
| +++ b/cc/raster/raster_buffer_provider_unittest.cc
|
| @@ -27,10 +27,9 @@
|
| #include "cc/resources/resource_pool.h"
|
| #include "cc/resources/resource_provider.h"
|
| #include "cc/resources/scoped_resource.h"
|
| -#include "cc/test/fake_output_surface.h"
|
| -#include "cc/test/fake_output_surface_client.h"
|
| #include "cc/test/fake_raster_source.h"
|
| #include "cc/test/fake_resource_provider.h"
|
| +#include "cc/test/test_context_provider.h"
|
| #include "cc/test/test_gpu_memory_buffer_manager.h"
|
| #include "cc/test/test_shared_bitmap_manager.h"
|
| #include "cc/test/test_web_graphics_context_3d.h"
|
| @@ -149,9 +148,7 @@ class RasterBufferProviderTest
|
| enum NamedTaskSet { REQUIRED_FOR_ACTIVATION, REQUIRED_FOR_DRAW, ALL };
|
|
|
| RasterBufferProviderTest()
|
| - : context_provider_(TestContextProvider::Create()),
|
| - worker_context_provider_(TestContextProvider::CreateWorker()),
|
| - all_tile_tasks_finished_(
|
| + : all_tile_tasks_finished_(
|
| base::ThreadTaskRunnerHandle::Get().get(),
|
| base::Bind(&RasterBufferProviderTest::AllTileTasksFinished,
|
| base::Unretained(this))),
|
| @@ -162,12 +159,12 @@ class RasterBufferProviderTest
|
| void SetUp() override {
|
| switch (GetParam()) {
|
| case RASTER_BUFFER_PROVIDER_TYPE_ZERO_COPY:
|
| - Create3dOutputSurfaceAndResourceProvider();
|
| + Create3dResourceProvider();
|
| raster_buffer_provider_ = ZeroCopyRasterBufferProvider::Create(
|
| resource_provider_.get(), PlatformColor::BestTextureFormat());
|
| break;
|
| case RASTER_BUFFER_PROVIDER_TYPE_ONE_COPY:
|
| - Create3dOutputSurfaceAndResourceProvider();
|
| + Create3dResourceProvider();
|
| raster_buffer_provider_ = base::MakeUnique<OneCopyRasterBufferProvider>(
|
| base::ThreadTaskRunnerHandle::Get().get(), context_provider_.get(),
|
| worker_context_provider_.get(), resource_provider_.get(),
|
| @@ -175,13 +172,13 @@ class RasterBufferProviderTest
|
| PlatformColor::BestTextureFormat(), false);
|
| break;
|
| case RASTER_BUFFER_PROVIDER_TYPE_GPU:
|
| - Create3dOutputSurfaceAndResourceProvider();
|
| + Create3dResourceProvider();
|
| raster_buffer_provider_ = base::MakeUnique<GpuRasterBufferProvider>(
|
| context_provider_.get(), worker_context_provider_.get(),
|
| resource_provider_.get(), false, 0, false);
|
| break;
|
| case RASTER_BUFFER_PROVIDER_TYPE_BITMAP:
|
| - CreateSoftwareOutputSurfaceAndResourceProvider();
|
| + CreateSoftwareResourceProvider();
|
| raster_buffer_provider_ =
|
| BitmapRasterBufferProvider::Create(resource_provider_.get());
|
| break;
|
| @@ -279,22 +276,20 @@ class RasterBufferProviderTest
|
| }
|
|
|
| private:
|
| - void Create3dOutputSurfaceAndResourceProvider() {
|
| - output_surface_ = FakeOutputSurface::Create3d(context_provider_,
|
| - worker_context_provider_);
|
| - CHECK(output_surface_->BindToClient(&output_surface_client_));
|
| + void Create3dResourceProvider() {
|
| + context_provider_ = TestContextProvider::Create();
|
| + context_provider_->BindToCurrentThread();
|
| + worker_context_provider_ = TestContextProvider::CreateWorker();
|
| TestWebGraphicsContext3D* context3d = context_provider_->TestContext3d();
|
| context3d->set_support_sync_query(true);
|
| resource_provider_ = FakeResourceProvider::Create(
|
| - output_surface_.get(), nullptr, &gpu_memory_buffer_manager_);
|
| + context_provider_.get(), &shared_bitmap_manager_,
|
| + &gpu_memory_buffer_manager_);
|
| }
|
|
|
| - void CreateSoftwareOutputSurfaceAndResourceProvider() {
|
| - output_surface_ = FakeOutputSurface::CreateSoftware(
|
| - base::WrapUnique(new SoftwareOutputDevice));
|
| - CHECK(output_surface_->BindToClient(&output_surface_client_));
|
| + void CreateSoftwareResourceProvider() {
|
| resource_provider_ = FakeResourceProvider::Create(
|
| - output_surface_.get(), &shared_bitmap_manager_, nullptr);
|
| + nullptr, &shared_bitmap_manager_, &gpu_memory_buffer_manager_);
|
| }
|
|
|
| void OnTimeout() {
|
| @@ -305,8 +300,6 @@ class RasterBufferProviderTest
|
| protected:
|
| scoped_refptr<TestContextProvider> context_provider_;
|
| scoped_refptr<TestContextProvider> worker_context_provider_;
|
| - FakeOutputSurfaceClient output_surface_client_;
|
| - std::unique_ptr<FakeOutputSurface> output_surface_;
|
| std::unique_ptr<ResourceProvider> resource_provider_;
|
| std::unique_ptr<TileTaskManager> tile_task_manager_;
|
| std::unique_ptr<RasterBufferProvider> raster_buffer_provider_;
|
| @@ -374,8 +367,8 @@ TEST_P(RasterBufferProviderTest, FalseThrottling) {
|
| }
|
|
|
| TEST_P(RasterBufferProviderTest, LostContext) {
|
| - LoseContext(output_surface_->context_provider());
|
| - LoseContext(output_surface_->worker_context_provider());
|
| + LoseContext(context_provider_.get());
|
| + LoseContext(worker_context_provider_.get());
|
|
|
| AppendTask(0u);
|
| AppendTask(1u);
|
|
|