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

Side by Side Diff: Source/platform/LengthBox.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/core/rendering/style/RenderStyle.cpp ('k') | Source/platform/LengthPoint.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 Copyright (c) 2012, Google Inc. All rights reserved. 4 Copyright (c) 2012, Google Inc. All rights reserved.
5 5
6 This library is free software; you can redistribute it and/or 6 This library is free software; you can redistribute it and/or
7 modify it under the terms of the GNU Library General Public 7 modify it under the terms of the GNU Library General Public
8 License as published by the Free Software Foundation; either 8 License as published by the Free Software Foundation; either
9 version 2 of the License, or (at your option) any later version. 9 version 2 of the License, or (at your option) any later version.
10 10
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 } 46 }
47 47
48 LengthBox(int v) 48 LengthBox(int v)
49 : m_left(Length(v, Fixed)) 49 : m_left(Length(v, Fixed))
50 , m_right(Length(v, Fixed)) 50 , m_right(Length(v, Fixed))
51 , m_top(Length(v, Fixed)) 51 , m_top(Length(v, Fixed))
52 , m_bottom(Length(v, Fixed)) 52 , m_bottom(Length(v, Fixed))
53 { 53 {
54 } 54 }
55 55
56 LengthBox(Length t, Length r, Length b, Length l) 56 LengthBox(const Length& t, const Length& r, const Length& b, const Length& l )
57 : m_left(l) 57 : m_left(l)
58 , m_right(r) 58 , m_right(r)
59 , m_top(t) 59 , m_top(t)
60 , m_bottom(b) 60 , m_bottom(b)
61 { 61 {
62 } 62 }
63 63
64 LengthBox(int t, int r, int b, int l) 64 LengthBox(int t, int r, int b, int l)
65 : m_left(Length(l, Fixed)) 65 : m_left(Length(l, Fixed))
66 , m_right(Length(r, Fixed)) 66 , m_right(Length(r, Fixed))
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 // Must be public for SET_VAR in RenderStyle.h 100 // Must be public for SET_VAR in RenderStyle.h
101 Length m_left; 101 Length m_left;
102 Length m_right; 102 Length m_right;
103 Length m_top; 103 Length m_top;
104 Length m_bottom; 104 Length m_bottom;
105 }; 105 };
106 106
107 } // namespace WebCore 107 } // namespace WebCore
108 108
109 #endif // LengthBox_h 109 #endif // LengthBox_h
OLDNEW
« no previous file with comments | « Source/core/rendering/style/RenderStyle.cpp ('k') | Source/platform/LengthPoint.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698