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

Side by Side Diff: cc/layers/nine_patch_layer.cc

Issue 12967018: cc: Avoid pointless gfx::Point/Size contructors when making Rects. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: RectFs as well Created 7 years, 9 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 | Annotate | Revision Log
« no previous file with comments | « cc/layers/layer_iterator_unittest.cc ('k') | cc/layers/nine_patch_layer_impl_unittest.cc » ('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 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/layers/nine_patch_layer.h" 5 #include "cc/layers/nine_patch_layer.h"
6 6
7 #include "cc/layers/nine_patch_layer_impl.h" 7 #include "cc/layers/nine_patch_layer_impl.h"
8 #include "cc/resources/prioritized_resource.h" 8 #include "cc/resources/prioritized_resource.h"
9 #include "cc/resources/resource_update.h" 9 #include "cc/resources/resource_update.h"
10 #include "cc/resources/resource_update_queue.h" 10 #include "cc/resources/resource_update_queue.h"
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 SetNeedsDisplay(); 56 SetNeedsDisplay();
57 } 57 }
58 58
59 void NinePatchLayer::Update(ResourceUpdateQueue* queue, 59 void NinePatchLayer::Update(ResourceUpdateQueue* queue,
60 const OcclusionTracker* occlusion, 60 const OcclusionTracker* occlusion,
61 RenderingStats* stats) { 61 RenderingStats* stats) {
62 CreateUpdaterIfNeeded(); 62 CreateUpdaterIfNeeded();
63 63
64 if (resource_ && 64 if (resource_ &&
65 (bitmap_dirty_ || resource_->texture()->resource_id() == 0)) { 65 (bitmap_dirty_ || resource_->texture()->resource_id() == 0)) {
66 gfx::Rect content_rect(gfx::Point(), 66 gfx::Rect content_rect(0, 0, bitmap_.width(), bitmap_.height());
67 gfx::Size(bitmap_.width(), bitmap_.height()));
68 ResourceUpdate upload = ResourceUpdate::Create(resource_->texture(), 67 ResourceUpdate upload = ResourceUpdate::Create(resource_->texture(),
69 &bitmap_, 68 &bitmap_,
70 content_rect, 69 content_rect,
71 content_rect, 70 content_rect,
72 gfx::Vector2d()); 71 gfx::Vector2d());
73 queue->AppendFullUpload(upload); 72 queue->AppendFullUpload(upload);
74 bitmap_dirty_ = false; 73 bitmap_dirty_ = false;
75 } 74 }
76 } 75 }
77 76
(...skipping 30 matching lines...) Expand all
108 107
109 if (resource_) { 108 if (resource_) {
110 DCHECK(!bitmap_.isNull()); 109 DCHECK(!bitmap_.isNull());
111 layer_impl->SetResourceId(resource_->texture()->resource_id()); 110 layer_impl->SetResourceId(resource_->texture()->resource_id());
112 layer_impl->SetLayout( 111 layer_impl->SetLayout(
113 gfx::Size(bitmap_.width(), bitmap_.height()), image_aperture_); 112 gfx::Size(bitmap_.width(), bitmap_.height()), image_aperture_);
114 } 113 }
115 } 114 }
116 115
117 } 116 }
OLDNEW
« no previous file with comments | « cc/layers/layer_iterator_unittest.cc ('k') | cc/layers/nine_patch_layer_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698