[Blink] Don't center images in Android WebViews

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/html/ImageDocument.h"
 
 #include "core/dom/Document.h"
 #include "core/dom/DocumentParser.h"
 #include "core/loader/EmptyClients.h"
 #include "core/testing/DummyPageHolder.h"
@@ skipping 164 matching lines @@
 }
 
 TEST_F(ImageDocumentTest, ImageScalesDownWithDsf) {
   createDocumentWithoutLoadingImage(20, 30);
   setWindowToViewportScalingFactor(2.f);
   loadImage();
   EXPECT_EQ(10, imageWidth());
   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