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

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

Issue 16255010: Hookup android_webview scroll offset delegation to Java side. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix compile break Created 7 years, 6 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: android_webview/javatests/src/org/chromium/android_webview/test/AndroidViewIntegrationTest.java
diff --git a/android_webview/javatests/src/org/chromium/android_webview/test/AndroidViewIntegrationTest.java b/android_webview/javatests/src/org/chromium/android_webview/test/AndroidViewIntegrationTest.java
index 72e62a60a869ea688b699fa9e741f7433dc07afe..9ba94193d41e1a2320261bb88b0ad4a95cc3acf0 100644
--- a/android_webview/javatests/src/org/chromium/android_webview/test/AndroidViewIntegrationTest.java
+++ b/android_webview/javatests/src/org/chromium/android_webview/test/AndroidViewIntegrationTest.java
@@ -65,11 +65,14 @@ public class AndroidViewIntegrationTest extends AwTestBase {
}
}
- private class DependencyFactory extends TestDependencyFactory {
- @Override
- public AwLayoutSizer createLayoutSizer() {
- return new TestAwLayoutSizer();
- }
+ @Override
+ protected TestDependencyFactory createTestDependencyFactory() {
+ return new TestDependencyFactory() {
+ @Override
+ public AwLayoutSizer createLayoutSizer() {
+ return new TestAwLayoutSizer();
+ }
+ };
}
final LinearLayout.LayoutParams wrapContentLayoutParams =
@@ -82,8 +85,7 @@ public class AndroidViewIntegrationTest extends AwTestBase {
getInstrumentation().runOnMainSync(new Runnable() {
@Override
public void run() {
- testContainerView.set(createAwTestContainerView(awContentsClient,
- new DependencyFactory()));
+ testContainerView.set(createAwTestContainerView(awContentsClient));
testContainerView.get().setLayoutParams(wrapContentLayoutParams);
testContainerView.get().setVisibility(visibility);
}
@@ -98,8 +100,7 @@ public class AndroidViewIntegrationTest extends AwTestBase {
getInstrumentation().runOnMainSync(new Runnable() {
@Override
public void run() {
- testContainerView.set(createDetachedAwTestContainerView(awContentsClient,
- new DependencyFactory()));
+ testContainerView.set(createDetachedAwTestContainerView(awContentsClient));
}
});
return testContainerView.get();

Powered by Google App Engine
This is Rietveld 408576698