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

Unified Diff: ui/gfx/geometry/quad_unittest.cc

Issue 2483493002: android: Don't run failing FontTest.DeriveFont; run gfx_unittests on bots. (Closed)
Patch Set: include build_config.h more Created 4 years, 1 month 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
« no previous file with comments | « ui/gfx/font_unittest.cc ('k') | ui/gfx/geometry/vector3d_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/gfx/geometry/quad_unittest.cc
diff --git a/ui/gfx/geometry/quad_unittest.cc b/ui/gfx/geometry/quad_unittest.cc
index 44cd366d87b2c5dda1e0f356d34c1cf3d4b26f4a..26b3bf9fb48c322722f41d11ed18df457071bd23 100644
--- a/ui/gfx/geometry/quad_unittest.cc
+++ b/ui/gfx/geometry/quad_unittest.cc
@@ -5,6 +5,7 @@
#include <stddef.h>
#include "base/macros.h"
+#include "build/build_config.h"
#include "testing/gtest/include/gtest/gtest.h"
#include "ui/gfx/geometry/quad_f.h"
#include "ui/gfx/geometry/rect_f.h"
@@ -282,7 +283,10 @@ TEST(QuadTest, ContainsPoint) {
EXPECT_FALSE(QuadF(a, b, c, d).Contains(a + epsilon_x));
EXPECT_TRUE(QuadF(a, b, c, d).Contains(a + epsilon_y));
+#if !defined(ARCH_CPU_ARM_FAMILY)
+ // TODO(danakj): Make this pass on ARM, https://crbug.com/662561
EXPECT_TRUE(QuadF(a, b, c, d).Contains(b));
+#endif
EXPECT_FALSE(QuadF(a, b, c, d).Contains(b - epsilon_x));
EXPECT_FALSE(QuadF(a, b, c, d).Contains(b - epsilon_y));
EXPECT_TRUE(QuadF(a, b, c, d).Contains(b + epsilon_x));
« no previous file with comments | « ui/gfx/font_unittest.cc ('k') | ui/gfx/geometry/vector3d_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698