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

Side by Side Diff: cc/resources/raster_worker_pool.cc

Issue 93663004: [#2] Pass gfx structs by const ref (gfx::Rect, gfx::RectF) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase to ToT, fix builds on non-linux platforms! Created 6 years, 11 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
« no previous file with comments | « cc/resources/raster_worker_pool.h ('k') | cc/resources/resource_provider.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/resources/raster_worker_pool.h" 5 #include "cc/resources/raster_worker_pool.h"
6 6
7 #include "base/json/json_writer.h" 7 #include "base/json/json_writer.h"
8 #include "base/metrics/histogram.h" 8 #include "base/metrics/histogram.h"
9 #include "base/values.h" 9 #include "base/values.h"
10 #include "cc/debug/devtools_instrumentation.h" 10 #include "cc/debug/devtools_instrumentation.h"
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 47
48 paint->setLCDRenderText(false); 48 paint->setLCDRenderText(false);
49 return true; 49 return true;
50 } 50 }
51 }; 51 };
52 52
53 class RasterWorkerPoolTaskImpl : public internal::RasterWorkerPoolTask { 53 class RasterWorkerPoolTaskImpl : public internal::RasterWorkerPoolTask {
54 public: 54 public:
55 RasterWorkerPoolTaskImpl(const Resource* resource, 55 RasterWorkerPoolTaskImpl(const Resource* resource,
56 PicturePileImpl* picture_pile, 56 PicturePileImpl* picture_pile,
57 gfx::Rect content_rect, 57 const gfx::Rect& content_rect,
58 float contents_scale, 58 float contents_scale,
59 RasterMode raster_mode, 59 RasterMode raster_mode,
60 TileResolution tile_resolution, 60 TileResolution tile_resolution,
61 int layer_id, 61 int layer_id,
62 const void* tile_id, 62 const void* tile_id,
63 int source_frame_number, 63 int source_frame_number,
64 bool use_gpu_rasterization, 64 bool use_gpu_rasterization,
65 RenderingStatsInstrumentation* rendering_stats, 65 RenderingStatsInstrumentation* rendering_stats,
66 const RasterWorkerPool::RasterTask::Reply& reply, 66 const RasterWorkerPool::RasterTask::Reply& reply,
67 TaskVector* dependencies) 67 TaskVector* dependencies)
(...skipping 393 matching lines...) Expand 10 before | Expand all | Expand 10 after
461 internal_ = NULL; 461 internal_ = NULL;
462 } 462 }
463 463
464 RasterWorkerPool::RasterTask::~RasterTask() { 464 RasterWorkerPool::RasterTask::~RasterTask() {
465 } 465 }
466 466
467 // static 467 // static
468 RasterWorkerPool::RasterTask RasterWorkerPool::CreateRasterTask( 468 RasterWorkerPool::RasterTask RasterWorkerPool::CreateRasterTask(
469 const Resource* resource, 469 const Resource* resource,
470 PicturePileImpl* picture_pile, 470 PicturePileImpl* picture_pile,
471 gfx::Rect content_rect, 471 const gfx::Rect& content_rect,
472 float contents_scale, 472 float contents_scale,
473 RasterMode raster_mode, 473 RasterMode raster_mode,
474 TileResolution tile_resolution, 474 TileResolution tile_resolution,
475 int layer_id, 475 int layer_id,
476 const void* tile_id, 476 const void* tile_id,
477 int source_frame_number, 477 int source_frame_number,
478 bool use_gpu_rasterization, 478 bool use_gpu_rasterization,
479 RenderingStatsInstrumentation* rendering_stats, 479 RenderingStatsInstrumentation* rendering_stats,
480 const RasterTask::Reply& reply, 480 const RasterTask::Reply& reply,
481 Task::Set* dependencies) { 481 Task::Set* dependencies) {
(...skipping 197 matching lines...) Expand 10 before | Expand all | Expand 10 after
679 679
680 internal::GraphNode* decode_node = CreateGraphNodeForTask( 680 internal::GraphNode* decode_node = CreateGraphNodeForTask(
681 decode_task, priority, graph); 681 decode_task, priority, graph);
682 decode_node->add_dependent(raster_node); 682 decode_node->add_dependent(raster_node);
683 } 683 }
684 684
685 return raster_node; 685 return raster_node;
686 } 686 }
687 687
688 } // namespace cc 688 } // namespace cc
OLDNEW
« no previous file with comments | « cc/resources/raster_worker_pool.h ('k') | cc/resources/resource_provider.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698