cc/layer_quad.h - Issue 11264056: cc: Use gfx:: Geometry types for positions, bounds, and related things.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: cc/layer_quad.h

Issue 11264056: cc: Use gfx:: Geometry types for positions, bounds, and related things. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: some missed intstuff Created 8 years, 2 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 side-by-side diff with in-line comments

Download patch

Index: cc/layer_quad.h

diff --git a/cc/layer_quad.h b/cc/layer_quad.h

index 9f4dfcf97e24ab8e23a4625cced154ae08f568dc..9c760611f57b44bd14c95766fba2369c961949c1 100644

--- a/cc/layer_quad.h

+++ b/cc/layer_quad.h

@@ -23,7 +23,7 @@ public:

, m_z(0)

{

}

- Edge(const FloatPoint&, const FloatPoint&);

+ Edge(const gfx::PointF&, const gfx::PointF&);

float x() const { return m_x; }

float y() const { return m_y; }

@@ -60,7 +60,7 @@ public:

}

void scale(float s) { scale(s, s, s); }

- FloatPoint intersect(const Edge& e) const

+ gfx::PointF intersect(const Edge& e) const

{

return FloatPoint(

enne (OOO) 2012/10/29 22:23:47 Don't rely on implicit conversion here.

danakj 2012/10/31 19:45:43 Oh, thanks. I just missed that one.

(y() * e.z() - e.y() * z()) / (x() * e.y() - e.x() * y()),

« no previous file with comments | « cc/layer_iterator_unittest.cc ('k') | cc/layer_quad.cc » ('j') | cc/layer_tiling_data.cc » ('J')