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

Side by Side Diff: Source/platform/LengthSize.h

Issue 305883002: Switch arguments to const Length references for performance (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 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 | « Source/platform/LengthPoint.h ('k') | Source/platform/graphics/filters/FilterOperation.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 /* 1 /*
2 Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 Copyright (C) 2006, 2008 Apple Inc. All rights reserved. 3 Copyright (C) 2006, 2008 Apple Inc. All rights reserved.
4 4
5 This library is free software; you can redistribute it and/or 5 This library is free software; you can redistribute it and/or
6 modify it under the terms of the GNU Library General Public 6 modify it under the terms of the GNU Library General Public
7 License as published by the Free Software Foundation; either 7 License as published by the Free Software Foundation; either
8 version 2 of the License, or (at your option) any later version. 8 version 2 of the License, or (at your option) any later version.
9 9
10 This library is distributed in the hope that it will be useful, 10 This library is distributed in the hope that it will be useful,
(...skipping 13 matching lines...) Expand all
24 #include "platform/Length.h" 24 #include "platform/Length.h"
25 25
26 namespace WebCore { 26 namespace WebCore {
27 27
28 class LengthSize { 28 class LengthSize {
29 public: 29 public:
30 LengthSize() 30 LengthSize()
31 { 31 {
32 } 32 }
33 33
34 LengthSize(Length width, Length height) 34 LengthSize(const Length& width, const Length& height)
35 : m_width(width) 35 : m_width(width)
36 , m_height(height) 36 , m_height(height)
37 { 37 {
38 } 38 }
39 39
40 bool operator==(const LengthSize& o) const 40 bool operator==(const LengthSize& o) const
41 { 41 {
42 return m_width == o.m_width && m_height == o.m_height; 42 return m_width == o.m_width && m_height == o.m_height;
43 } 43 }
44 44
45 void setWidth(Length width) { m_width = width; } 45 void setWidth(const Length& width) { m_width = width; }
46 Length width() const { return m_width; } 46 const Length& width() const { return m_width; }
47 47
48 void setHeight(Length height) { m_height = height; } 48 void setHeight(const Length& height) { m_height = height; }
49 Length height() const { return m_height; } 49 const Length& height() const { return m_height; }
50 50
51 private: 51 private:
52 Length m_width; 52 Length m_width;
53 Length m_height; 53 Length m_height;
54 }; 54 };
55 55
56 } // namespace WebCore 56 } // namespace WebCore
57 57
58 #endif // LengthSize_h 58 #endif // LengthSize_h
OLDNEW
« no previous file with comments | « Source/platform/LengthPoint.h ('k') | Source/platform/graphics/filters/FilterOperation.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698