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

Unified Diff: android_webview/javatests/src/org/chromium/android_webview/test/AwLegacyQuirksTest.java

Issue 689473005: aw: Do not merge kModeProcess into kModeDraw (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: style Created 6 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
Index: android_webview/javatests/src/org/chromium/android_webview/test/AwLegacyQuirksTest.java
diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/AwLegacyQuirksTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/AwLegacyQuirksTest.java
index fdac91d183d1ad51cf6292def91969b0f3d1584d..41f378bfb44c5a064d798071f41e9707a56517eb 100644
--- a/android_webview/javatests/src/org/chromium/android_webview/test/AwLegacyQuirksTest.java
+++ b/android_webview/javatests/src/org/chromium/android_webview/test/AwLegacyQuirksTest.java
@@ -6,7 +6,6 @@ package org.chromium.android_webview.test;
import android.test.suitebuilder.annotation.MediumTest;
-import org.chromium.base.test.util.DisabledTest;
import org.chromium.android_webview.AwContents;
import org.chromium.android_webview.AwContentsClient;
import org.chromium.android_webview.AwSettings;
@@ -32,9 +31,9 @@ public class AwLegacyQuirksTest extends AwTestBase {
AwSettings settings = getAwSettingsOnUiThread(awContents);
CallbackHelper onPageFinishedHelper = contentClient.getOnPageFinishedHelper();
- final String pageTemplate = "<html><head>" +
- "<meta name='viewport' content='width=device-width, target-densityDpi=%s' />" +
- "</head><body onload='document.title=document.body.clientWidth'></body></html>";
+ final String pageTemplate = "<html><head>"
+ + "<meta name='viewport' content='width=device-width, target-densityDpi=%s' />"
+ + "</head><body onload='document.title=document.body.clientWidth'></body></html>";
final String pageDeviceDpi = String.format((Locale) null, pageTemplate, "device-dpi");
final String pageHighDpi = String.format((Locale) null, pageTemplate, "high-dpi");
final String pageDpi100 = String.format((Locale) null, pageTemplate, "100");
@@ -69,9 +68,9 @@ public class AwLegacyQuirksTest extends AwTestBase {
AwSettings settings = getAwSettingsOnUiThread(awContents);
CallbackHelper onPageFinishedHelper = contentClient.getOnPageFinishedHelper();
- final String page = "<html><head>" +
- "<meta name='viewport' content='width=device-width, initial-scale=0.5' />" +
- "</head><body onload='document.title=document.body.clientWidth'></body></html>";
+ final String page = "<html><head>"
+ + "<meta name='viewport' content='width=device-width, initial-scale=0.5' />"
+ + "</head><body onload='document.title=document.body.clientWidth'></body></html>";
settings.setJavaScriptEnabled(true);
settings.setUseWideViewPort(true);
@@ -95,10 +94,10 @@ public class AwLegacyQuirksTest extends AwTestBase {
AwSettings settings = getAwSettingsOnUiThread(awContents);
CallbackHelper onPageFinishedHelper = contentClient.getOnPageFinishedHelper();
- final String page = "<html><head>" +
- "<meta name='viewport' content='width=0, height=0, initial-scale=0.0, " +
- " minimum-scale=0.0, maximum-scale=0.0' />" +
- "</head><body onload='document.title=document.body.clientWidth'></body></html>";
+ final String page = "<html><head>"
+ + "<meta name='viewport' content='width=0, height=0, initial-scale=0.0, "
+ + " minimum-scale=0.0, maximum-scale=0.0' />"
+ + "</head><body onload='document.title=document.body.clientWidth'></body></html>";
settings.setJavaScriptEnabled(true);
@@ -134,13 +133,13 @@ public class AwLegacyQuirksTest extends AwTestBase {
DeviceDisplayInfo deviceInfo =
DeviceDisplayInfo.create(getInstrumentation().getTargetContext());
float dipScale = (float) deviceInfo.getDIPScale();
- float physicalDisplayWidth = deviceInfo.getPhysicalDisplayWidth() != 0 ?
- deviceInfo.getPhysicalDisplayWidth() :
- deviceInfo.getDisplayWidth();
+ float physicalDisplayWidth = deviceInfo.getPhysicalDisplayWidth() != 0
+ ? deviceInfo.getPhysicalDisplayWidth()
+ : deviceInfo.getDisplayWidth();
float cssDisplayWidth = physicalDisplayWidth / dipScale;
- float physicalDisplayHeight = deviceInfo.getPhysicalDisplayHeight() != 0 ?
- deviceInfo.getPhysicalDisplayHeight() :
- deviceInfo.getDisplayHeight();
+ float physicalDisplayHeight = deviceInfo.getPhysicalDisplayHeight() != 0
+ ? deviceInfo.getPhysicalDisplayHeight()
+ : deviceInfo.getDisplayHeight();
float cssDisplayHeight = physicalDisplayHeight / dipScale;
float screenWidth = Integer.parseInt(
@@ -188,11 +187,11 @@ public class AwLegacyQuirksTest extends AwTestBase {
final int pageWidth = 3000;
final float pageScale = 1.0f;
- final String page = String.format((Locale) null, "<html><head>" +
- "<meta name='viewport' content='width=%d' />" +
- "<meta name='viewport' content='initial-scale=%.1f' />" +
- "<meta name='viewport' content='user-scalable=0' />" +
- "</head><body onload='document.title=document.body.clientWidth'></body></html>",
+ final String page = String.format((Locale) null, "<html><head>"
+ + "<meta name='viewport' content='width=%d' />"
+ + "<meta name='viewport' content='initial-scale=%.1f' />"
+ + "<meta name='viewport' content='user-scalable=0' />"
+ + "</head><body onload='document.title=document.body.clientWidth'></body></html>",
pageWidth, pageScale);
settings.setJavaScriptEnabled(true);
@@ -225,10 +224,10 @@ public class AwLegacyQuirksTest extends AwTestBase {
CallbackHelper onPageFinishedHelper = contentClient.getOnPageFinishedHelper();
final int pageWidth = 3000;
- final String page = String.format((Locale) null, "<html><head>" +
- "<meta name='viewport' content='width=device-width' />" +
- "<meta name='viewport' content='width=%d' />" +
- "</head><body onload='document.title=document.body.clientWidth'></body></html>",
+ final String page = String.format((Locale) null, "<html><head>"
+ + "<meta name='viewport' content='width=device-width' />"
+ + "<meta name='viewport' content='width=%d' />"
+ + "</head><body onload='document.title=document.body.clientWidth'></body></html>",
pageWidth);
settings.setJavaScriptEnabled(true);
@@ -249,11 +248,11 @@ public class AwLegacyQuirksTest extends AwTestBase {
AwSettings settings = getAwSettingsOnUiThread(awContents);
CallbackHelper onPageFinishedHelper = contentClient.getOnPageFinishedHelper();
- final String pageTemplate = "<html><head>" +
- "<meta name='viewport' content='initial-scale=%d' />" +
- "</head><body>" +
- "<div style='width:10000px;height:200px'>A big div</div>" +
- "</body></html>";
+ final String pageTemplate = "<html><head>"
+ + "<meta name='viewport' content='initial-scale=%d' />"
+ + "</head><body>"
+ + "<div style='width:10000px;height:200px'>A big div</div>"
+ + "</body></html>";
final String pageScale4 = String.format((Locale) null, pageTemplate, 4);
final String page = String.format((Locale) null, pageTemplate, 1);
@@ -273,8 +272,6 @@ public class AwLegacyQuirksTest extends AwTestBase {
ensureScaleBecomes(1.0f, awContents);
}
- // This should not be run in software mode. crbug.com/428355.
- @DisableHardwareAccelerationForTest
@MediumTest
@Feature({"AndroidWebView"})
public void testNoUserScalableQuirk() throws Throwable {
@@ -284,17 +281,17 @@ public class AwLegacyQuirksTest extends AwTestBase {
final AwContents awContents = testContainerView.getAwContents();
CallbackHelper onPageFinishedHelper = contentClient.getOnPageFinishedHelper();
- final String pageScale4 = "<html><head>" +
- "<meta name='viewport' content='initial-scale=4' />" +
- "</head><body>" +
- "<div style='width:10000px;height:200px'>A big div</div>" +
- "</body></html>";
- final String page = "<html><head>" +
- "<meta name='viewport' " +
- "content='width=device-width,initial-scale=2,user-scalable=no' />" +
- "</head><body>" +
- "<div style='width:10000px;height:200px'>A big div</div>" +
- "</body></html>";
+ final String pageScale4 = "<html><head>"
+ + "<meta name='viewport' content='initial-scale=4' />"
+ + "</head><body>"
+ + "<div style='width:10000px;height:200px'>A big div</div>"
+ + "</body></html>";
+ final String page = "<html><head>"
+ + "<meta name='viewport' "
+ + "content='width=device-width,initial-scale=2,user-scalable=no' />"
+ + "</head><body>"
+ + "<div style='width:10000px;height:200px'>A big div</div>"
+ + "</body></html>";
// Page scale updates are asynchronous. There is an issue that we can't
// reliably check, whether the scale as NOT changed (i.e. remains to be 1.0).
@@ -322,18 +319,18 @@ public class AwLegacyQuirksTest extends AwTestBase {
AwSettings settings = getAwSettingsOnUiThread(awContents);
CallbackHelper onPageFinishedHelper = contentClient.getOnPageFinishedHelper();
final String expectedBackgroundSize = "cover";
- final String page = "<html><head>" +
- "<script>" +
- "function getBackgroundSize() {" +
- " var e = document.getElementById('test'); " +
- " e.style.backgroundSize = '" + expectedBackgroundSize + "';" +
- " e.style.background = 'center red url(dummy://test.png) no-repeat border-box'; " +
- " return e.style.backgroundSize; " +
- "}" +
- "</script></head>" +
- "<body onload='document.title=getBackgroundSize()'>" +
- " <div id='test'> </div>" +
- "</body></html>";
+ final String page = "<html><head>"
+ + "<script>"
+ + "function getBackgroundSize() {"
+ + " var e = document.getElementById('test'); "
+ + " e.style.backgroundSize = '" + expectedBackgroundSize + "';"
+ + " e.style.background = 'center red url(dummy://test.png) no-repeat border-box'; "
+ + " return e.style.backgroundSize; "
+ + "}"
+ + "</script></head>"
+ + "<body onload='document.title=getBackgroundSize()'>"
+ + " <div id='test'> </div>"
+ + "</body></html>";
settings.setJavaScriptEnabled(true);
loadDataSync(awContents, onPageFinishedHelper, page, "text/html", false);
String actualBackgroundSize = getTitleOnUiThread(awContents);

Powered by Google App Engine
This is Rietveld 408576698