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

Unified Diff: mojo/public/cpp/bindings/tests/sample_service_unittest.cc

Issue 291173010: Mojo: Mojom: Remove support for octal. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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: mojo/public/cpp/bindings/tests/sample_service_unittest.cc
diff --git a/mojo/public/cpp/bindings/tests/sample_service_unittest.cc b/mojo/public/cpp/bindings/tests/sample_service_unittest.cc
index 1cb1af42f7bb579b570d7239bdbc437713807f26..ea32516b908bb4b6c42ad2db791607448e198143 100644
--- a/mojo/public/cpp/bindings/tests/sample_service_unittest.cc
+++ b/mojo/public/cpp/bindings/tests/sample_service_unittest.cc
@@ -374,20 +374,17 @@ TEST(BindingsSampleTest, DefaultValues) {
// EXPECT_EQ(9999999999999999999ULL, full.a7());
EXPECT_EQ(0x12345, full.a8());
EXPECT_EQ(-0x12345, full.a9());
- // TODO(vtl): crbug.com/375829
- // EXPECT_EQ(8, full.a10());
- // EXPECT_EQ(-8, full.a11());
- EXPECT_EQ(1234, full.a12());
- EXPECT_TRUE(full.a13());
- EXPECT_FALSE(full.a14());
- EXPECT_FLOAT_EQ(123.25f, full.a15());
- EXPECT_DOUBLE_EQ(1234567890.123, full.a16());
- EXPECT_DOUBLE_EQ(1E10, full.a17());
- EXPECT_DOUBLE_EQ(-1.2E+20, full.a18());
- EXPECT_DOUBLE_EQ(1.23E-20, full.a19());
+ EXPECT_EQ(1234, full.a10());
+ EXPECT_TRUE(full.a11());
+ EXPECT_FALSE(full.a12());
+ EXPECT_FLOAT_EQ(123.25f, full.a13());
+ EXPECT_DOUBLE_EQ(1234567890.123, full.a14());
+ EXPECT_DOUBLE_EQ(1E10, full.a15());
+ EXPECT_DOUBLE_EQ(-1.2E+20, full.a16());
+ EXPECT_DOUBLE_EQ(1.23E-20, full.a17());
+ EXPECT_TRUE(full.a18().is_null());
+ EXPECT_TRUE(full.a19().is_null());
EXPECT_TRUE(full.a20().is_null());
- EXPECT_TRUE(full.a21().is_null());
- EXPECT_TRUE(full.a22().is_null());
}
} // namespace
« no previous file with comments | « mojo/apps/js/bindings/sample_service_unittests.js ('k') | mojo/public/interfaces/bindings/tests/sample_service.mojom » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698