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

Side by Side Diff: cc/tiles/gpu_image_decode_controller.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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/tiles/gpu_image_decode_controller.h" 5 #include "cc/tiles/gpu_image_decode_controller.h"
6 6
7 #include "cc/debug/devtools_instrumentation.h" 7 #include "cc/debug/devtools_instrumentation.h"
8 #include "cc/raster/tile_task_runner.h" 8 #include "cc/raster/tile_task_runner.h"
9 #include "skia/ext/refptr.h" 9 #include "skia/ext/refptr.h"
10 10
(...skipping 11 matching lines...) Expand all
22 22
23 // Overridden from Task: 23 // Overridden from Task:
24 void RunOnWorkerThread() override { 24 void RunOnWorkerThread() override {
25 TRACE_EVENT2("cc", "ImageDecodeTaskImpl::RunOnWorkerThread", "mode", "gpu", 25 TRACE_EVENT2("cc", "ImageDecodeTaskImpl::RunOnWorkerThread", "mode", "gpu",
26 "source_prepare_tiles_id", source_prepare_tiles_id_); 26 "source_prepare_tiles_id", source_prepare_tiles_id_);
27 devtools_instrumentation::ScopedImageDecodeTask image_decode_task( 27 devtools_instrumentation::ScopedImageDecodeTask image_decode_task(
28 image_ref_.get()); 28 image_ref_.get());
29 controller_->DecodeImage(image_); 29 controller_->DecodeImage(image_);
30 } 30 }
31 31
32 // Overridden from TileTask: 32 protected:
33 void ScheduleOnOriginThread(TileTaskClient* client) override {} 33 ~ImageDecodeTaskImpl() override {
34 void CompleteOnOriginThread(TileTaskClient* client) override {
35 controller_->RemovePendingTaskForImage(image_); 34 controller_->RemovePendingTaskForImage(image_);
36 } 35 }
37 36
38 protected:
39 ~ImageDecodeTaskImpl() override {}
40
41 private: 37 private:
42 GpuImageDecodeController* controller_; 38 GpuImageDecodeController* controller_;
43 DrawImage image_; 39 DrawImage image_;
44 skia::RefPtr<const SkImage> image_ref_; 40 skia::RefPtr<const SkImage> image_ref_;
45 uint64_t source_prepare_tiles_id_; 41 uint64_t source_prepare_tiles_id_;
46 42
47 DISALLOW_COPY_AND_ASSIGN(ImageDecodeTaskImpl); 43 DISALLOW_COPY_AND_ASSIGN(ImageDecodeTaskImpl);
48 }; 44 };
49 45
50 GpuImageDecodeController::GpuImageDecodeController() {} 46 GpuImageDecodeController::GpuImageDecodeController() {}
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 prerolled_images_.insert(image.image()->uniqueID()); 89 prerolled_images_.insert(image.image()->uniqueID());
94 } 90 }
95 91
96 void GpuImageDecodeController::RemovePendingTaskForImage( 92 void GpuImageDecodeController::RemovePendingTaskForImage(
97 const DrawImage& image) { 93 const DrawImage& image) {
98 base::AutoLock lock(lock_); 94 base::AutoLock lock(lock_);
99 pending_image_tasks_.erase(image.image()->uniqueID()); 95 pending_image_tasks_.erase(image.image()->uniqueID());
100 } 96 }
101 97
102 } // namespace cc 98 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698