Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(334)

Side by Side Diff: cc/test/fake_tile_manager.cc

Issue 1866043006: cc: Remove ScheduleOnOriginThread() and CompleteOnOriginThread(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: wip Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/test/fake_tile_manager.h" 5 #include "cc/test/fake_tile_manager.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <deque> 10 #include <deque>
11 #include <limits> 11 #include <limits>
12 12
13 #include "base/lazy_instance.h" 13 #include "base/lazy_instance.h"
14 #include "base/thread_task_runner_handle.h" 14 #include "base/thread_task_runner_handle.h"
15 #include "cc/raster/raster_buffer.h" 15 #include "cc/raster/raster_buffer.h"
16 #include "cc/raster/tile_task_runner.h" 16 #include "cc/raster/tile_task_runner.h"
17 17
18 namespace cc { 18 namespace cc {
19 19
20 namespace { 20 namespace {
21 21
22 class FakeTileTaskRunnerImpl : public TileTaskRunner, public TileTaskClient { 22 class FakeTileTaskRunnerImpl : public TileTaskRunner, public TileTaskClient {
23 public: 23 public:
24 // Overridden from TileTaskRunner: 24 // Overridden from TileTaskRunner:
25 void Shutdown() override {} 25 void Shutdown() override {}
26 void ScheduleTasks(TaskGraph* graph) override { 26 void ScheduleTasks(TaskGraph* graph) override {
27 for (const auto& node : graph->nodes) { 27 for (const auto& node : graph->nodes) {
28 RasterTask* task = static_cast<RasterTask*>(node.task); 28 RasterTask* task = static_cast<RasterTask*>(node.task);
29
30 task->WillSchedule();
31 task->ScheduleOnOriginThread(this);
32 task->DidSchedule();
33
34 completed_tasks_.push_back(task); 29 completed_tasks_.push_back(task);
35 } 30 }
36 } 31 }
37 void CheckForCompletedTasks() override { 32 void CollectCompletedTasks(Task::Vector* completed_tasks) override {
33 // TODO(prashant.n): Modify the function definition to make it analogous
34 // with other overridden definitions of CollectCompletedTasks.
38 for (RasterTask::Vector::iterator it = completed_tasks_.begin(); 35 for (RasterTask::Vector::iterator it = completed_tasks_.begin();
39 it != completed_tasks_.end(); 36 it != completed_tasks_.end();
40 ++it) { 37 ++it) {
41 RasterTask* task = it->get(); 38 RasterTask* task = it->get();
42
43 task->WillComplete();
44 task->CompleteOnOriginThread(this);
45 task->DidComplete(); 39 task->DidComplete();
46 } 40 }
47 completed_tasks_.clear(); 41 completed_tasks_.clear();
48 } 42 }
49 ResourceFormat GetResourceFormat(bool must_support_alpha) const override { 43 ResourceFormat GetResourceFormat(bool must_support_alpha) const override {
50 return RGBA_8888; 44 return RGBA_8888;
51 } 45 }
52 bool GetResourceRequiresSwizzle(bool must_support_alpha) const override { 46 bool GetResourceRequiresSwizzle(bool must_support_alpha) const override {
53 return ResourceFormatRequiresSwizzle(GetResourceFormat(must_support_alpha)); 47 return ResourceFormatRequiresSwizzle(GetResourceFormat(must_support_alpha));
54 } 48 }
55 49
50 TileTaskClient* AsTileTaskClient() override { return this; }
51
56 // Overridden from TileTaskClient: 52 // Overridden from TileTaskClient:
57 scoped_ptr<RasterBuffer> AcquireBufferForRaster( 53 scoped_ptr<RasterBuffer> AcquireBufferForRaster(
58 const Resource* resource, 54 const Resource* resource,
59 uint64_t resource_content_id, 55 uint64_t resource_content_id,
60 uint64_t previous_content_id) override { 56 uint64_t previous_content_id) override {
61 return nullptr; 57 return nullptr;
62 } 58 }
63 void ReleaseBufferForRaster(scoped_ptr<RasterBuffer> buffer) override {} 59 void ReleaseBufferForRaster(scoped_ptr<RasterBuffer> buffer) override {}
64 60
65 private: 61 private:
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 } 96 }
101 97
102 void FakeTileManager::Release(Tile* tile) { 98 void FakeTileManager::Release(Tile* tile) {
103 TileManager::Release(tile); 99 TileManager::Release(tile);
104 100
105 FreeResourcesForReleasedTiles(); 101 FreeResourcesForReleasedTiles();
106 CleanUpReleasedTiles(); 102 CleanUpReleasedTiles();
107 } 103 }
108 104
109 } // namespace cc 105 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698