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

Side by Side Diff: cc/layers/texture_layer_impl.h

Issue 139233002: [#4] Pass gfx structs by const ref (gfx::PointF) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Corrected as per review comments! 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/layers/texture_layer.cc ('k') | cc/layers/ui_resource_layer.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 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 #ifndef CC_LAYERS_TEXTURE_LAYER_IMPL_H_ 5 #ifndef CC_LAYERS_TEXTURE_LAYER_IMPL_H_
6 #define CC_LAYERS_TEXTURE_LAYER_IMPL_H_ 6 #define CC_LAYERS_TEXTURE_LAYER_IMPL_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 27 matching lines...) Expand all
38 38
39 unsigned texture_id() const { return texture_id_; } 39 unsigned texture_id() const { return texture_id_; }
40 void set_texture_id(unsigned id) { texture_id_ = id; } 40 void set_texture_id(unsigned id) { texture_id_ = id; }
41 void set_premultiplied_alpha(bool premultiplied_alpha) { 41 void set_premultiplied_alpha(bool premultiplied_alpha) {
42 premultiplied_alpha_ = premultiplied_alpha; 42 premultiplied_alpha_ = premultiplied_alpha;
43 } 43 }
44 void set_blend_background_color(bool blend) { 44 void set_blend_background_color(bool blend) {
45 blend_background_color_ = blend; 45 blend_background_color_ = blend;
46 } 46 }
47 void set_flipped(bool flipped) { flipped_ = flipped; } 47 void set_flipped(bool flipped) { flipped_ = flipped; }
48 void set_uv_top_left(gfx::PointF top_left) { uv_top_left_ = top_left; } 48 void set_uv_top_left(const gfx::PointF& top_left) { uv_top_left_ = top_left; }
49 void set_uv_bottom_right(gfx::PointF bottom_right) { 49 void set_uv_bottom_right(const gfx::PointF& bottom_right) {
50 uv_bottom_right_ = bottom_right; 50 uv_bottom_right_ = bottom_right;
51 } 51 }
52 52
53 // 1--2 53 // 1--2
54 // | | 54 // | |
55 // 0--3 55 // 0--3
56 void set_vertex_opacity(const float vertex_opacity[4]) { 56 void set_vertex_opacity(const float vertex_opacity[4]) {
57 vertex_opacity_[0] = vertex_opacity[0]; 57 vertex_opacity_[0] = vertex_opacity[0];
58 vertex_opacity_[1] = vertex_opacity[1]; 58 vertex_opacity_[1] = vertex_opacity[1];
59 vertex_opacity_[2] = vertex_opacity[2]; 59 vertex_opacity_[2] = vertex_opacity[2];
(...skipping 25 matching lines...) Expand all
85 bool uses_mailbox_; 85 bool uses_mailbox_;
86 bool own_mailbox_; 86 bool own_mailbox_;
87 bool valid_texture_copy_; 87 bool valid_texture_copy_;
88 88
89 DISALLOW_COPY_AND_ASSIGN(TextureLayerImpl); 89 DISALLOW_COPY_AND_ASSIGN(TextureLayerImpl);
90 }; 90 };
91 91
92 } // namespace cc 92 } // namespace cc
93 93
94 #endif // CC_LAYERS_TEXTURE_LAYER_IMPL_H_ 94 #endif // CC_LAYERS_TEXTURE_LAYER_IMPL_H_
OLDNEW
« no previous file with comments | « cc/layers/texture_layer.cc ('k') | cc/layers/ui_resource_layer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698