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

Side by Side Diff: third_party/WebKit/Source/platform/DecimalTest.cpp

Issue 2388303002: reflow comments in platform/ (Closed)
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 fromString(step)); 94 fromString(step));
95 Decimal value = fromString(valueString); 95 Decimal value = fromString(valueString);
96 for (int i = 0; i < numberOfStepTimes; ++i) { 96 for (int i = 0; i < numberOfStepTimes; ++i) {
97 value += stepRange.step; 97 value += stepRange.step;
98 value = stepRange.clampValue(value); 98 value = stepRange.clampValue(value);
99 } 99 }
100 return value; 100 return value;
101 } 101 }
102 }; 102 };
103 103
104 // FIXME: We should use expectedSign without "Decimal::", however, g++ causes un defined references for DecimalTest::Positive and Negative. 104 // FIXME: We should use expectedSign without "Decimal::", however, g++ causes
105 // undefined references for DecimalTest::Positive and Negative.
105 #define EXPECT_DECIMAL_ENCODED_DATA_EQ(expectedCoefficient, expectedExponent, \ 106 #define EXPECT_DECIMAL_ENCODED_DATA_EQ(expectedCoefficient, expectedExponent, \
106 expectedSign, decimal) \ 107 expectedSign, decimal) \
107 EXPECT_EQ((expectedCoefficient), (decimal).value().coefficient()); \ 108 EXPECT_EQ((expectedCoefficient), (decimal).value().coefficient()); \
108 EXPECT_EQ((expectedExponent), (decimal).value().exponent()); \ 109 EXPECT_EQ((expectedExponent), (decimal).value().exponent()); \
109 EXPECT_EQ(Decimal::expectedSign, (decimal).value().getSign()); 110 EXPECT_EQ(Decimal::expectedSign, (decimal).value().getSign());
110 111
111 #define EXPECT_DECIMAL_STREQ(expected, decimal) \ 112 #define EXPECT_DECIMAL_STREQ(expected, decimal) \
112 EXPECT_STREQ((expected), (decimal).toString().ascii().data()) 113 EXPECT_STREQ((expected), (decimal).toString().ascii().data())
113 114
114 TEST_F(DecimalTest, Abs) { 115 TEST_F(DecimalTest, Abs) {
(...skipping 1031 matching lines...) Expand 10 before | Expand all | Expand 10 after
1146 encode(UINT64_C(99999999999999999), -100, Positive)); 1147 encode(UINT64_C(99999999999999999), -100, Positive));
1147 } 1148 }
1148 1149
1149 TEST_F(DecimalTest, ToStringSpecialValues) { 1150 TEST_F(DecimalTest, ToStringSpecialValues) {
1150 EXPECT_DECIMAL_STREQ("Infinity", Decimal::infinity(Positive)); 1151 EXPECT_DECIMAL_STREQ("Infinity", Decimal::infinity(Positive));
1151 EXPECT_DECIMAL_STREQ("-Infinity", Decimal::infinity(Negative)); 1152 EXPECT_DECIMAL_STREQ("-Infinity", Decimal::infinity(Negative));
1152 EXPECT_DECIMAL_STREQ("NaN", Decimal::nan()); 1153 EXPECT_DECIMAL_STREQ("NaN", Decimal::nan());
1153 } 1154 }
1154 1155
1155 } // namespace blink 1156 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/platform/Decimal.h ('k') | third_party/WebKit/Source/platform/DragImageTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698