[Blink] Don't center images in Android WebViews

third_party/WebKit/Source/core/html/ImageDocumentTest.cpp

Index: third_party/WebKit/Source/core/html/ImageDocumentTest.cpp
diff --git a/third_party/WebKit/Source/core/html/ImageDocumentTest.cpp b/third_party/WebKit/Source/core/html/ImageDocumentTest.cpp
index 28b344e0616a10e82a6a15261ac6fd75f78d441b..82e46a0a0e7e74549d8c6d4c618ed998c59d783d 100644
--- a/third_party/WebKit/Source/core/html/ImageDocumentTest.cpp
+++ b/third_party/WebKit/Source/core/html/ImageDocumentTest.cpp
@@ -182,4 +182,16 @@ TEST_F(ImageDocumentTest, ImageScalesDownWithDsf) {
   EXPECT_EQ(10, imageHeight());
 }
 
+TEST_F(ImageDocumentTest, ImagePositionedCorrectly) {
+  createDocument(80, 70);
+
+  if (document().shouldShrinkToFit()) {

[aelias_OOO_until_Jul13, 2017/01/19 04:08:22]

This test can never go red, even without your change.  It's not an integration
test, so there's no concept of running in a different environment with a
different setting.  Rather, the test itself determines its settings and it's
uniform across all platforms.  So please set the forceZeroLayoutHeight setting
during test initialization instead of having an if statement (or simply remove
the test if you don't have time, I'd rather have no test than a fake test).

[gone, 2017/01/19 18:56:29]

Yeah, I was conking out last night before I could figure out how to set up the
setting and uploaded this to see if I was on the right track with the bots. 
I've split it into two tests and updated the settings now.

+    EXPECT_EQ(15, document().imageElement()->offsetLeft());
+    EXPECT_EQ(10, document().imageElement()->offsetTop());
+  } else {
+    EXPECT_EQ(0, document().imageElement()->offsetLeft());
+    EXPECT_EQ(0, document().imageElement()->offsetTop());
+  }
+}
+
 }  // namespace blink