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

Side by Side Diff: cc/raster/tile_task_runner.cc

Issue 1854723002: cc: Simplify task and its derived classes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Corrected scope of dependencies. 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/raster/tile_task_runner.h" 5 #include "cc/raster/tile_task_runner.h"
6 #include "cc/resources/platform_color.h" 6 #include "cc/resources/platform_color.h"
7 7
8 namespace cc { 8 namespace cc {
9 9
10 TileTask::TileTask() : did_schedule_(false), did_complete_(false) {
11 }
12
13 TileTask::~TileTask() {
14 DCHECK(!did_schedule_);
15 DCHECK(!did_run_ || did_complete_);
16 }
17
18 void TileTask::WillSchedule() {
19 DCHECK(!did_schedule_);
20 }
21
22 void TileTask::DidSchedule() {
23 did_schedule_ = true;
24 did_complete_ = false;
25 }
26
27 bool TileTask::HasBeenScheduled() const {
28 return did_schedule_;
29 }
30
31 void TileTask::WillComplete() {
32 DCHECK(!did_complete_);
33 }
34
35 void TileTask::DidComplete() {
36 DCHECK(did_schedule_);
37 DCHECK(!did_complete_);
38 did_schedule_ = false;
39 did_complete_ = true;
40 }
41
42 bool TileTask::HasCompleted() const {
43 return did_complete_;
44 }
45
46 ImageDecodeTask::ImageDecodeTask() {
47 }
48
49 ImageDecodeTask::~ImageDecodeTask() {
50 }
51
52 RasterTask::RasterTask(ImageDecodeTask::Vector* dependencies) {
53 dependencies_.swap(*dependencies);
54 }
55
56 RasterTask::~RasterTask() {
57 }
58 10
59 bool TileTaskRunner::ResourceFormatRequiresSwizzle(ResourceFormat format) { 11 bool TileTaskRunner::ResourceFormatRequiresSwizzle(ResourceFormat format) {
60 switch (format) { 12 switch (format) {
61 case RGBA_8888: 13 case RGBA_8888:
62 case BGRA_8888: 14 case BGRA_8888:
63 // Initialize resource using the preferred PlatformColor component 15 // Initialize resource using the preferred PlatformColor component
64 // order and swizzle in the shader instead of in software. 16 // order and swizzle in the shader instead of in software.
65 return !PlatformColor::SameComponentOrder(format); 17 return !PlatformColor::SameComponentOrder(format);
66 case RGBA_4444: 18 case RGBA_4444:
67 case ETC1: 19 case ETC1:
68 case ALPHA_8: 20 case ALPHA_8:
69 case LUMINANCE_8: 21 case LUMINANCE_8:
70 case RGB_565: 22 case RGB_565:
71 case RED_8: 23 case RED_8:
72 case LUMINANCE_F16: 24 case LUMINANCE_F16:
73 return false; 25 return false;
74 } 26 }
75 NOTREACHED(); 27 NOTREACHED();
76 return false; 28 return false;
77 } 29 }
78 30
79 } // namespace cc 31 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698