chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebappModeTest.java - Issue 1190103002: Make lint fail builds.

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebappModeTest.java

Issue 1190103002: Make lint fail builds. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Fix exported crash for ChromeShellTest Created 5 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

« no previous file with comments | « chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/document/DocumentTabModelImplTest.java ('k') | chrome/android/javatests_shell/src/org/chromium/chrome/browser/notifications/NotificationUIManagerTest.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebappModeTest.java

diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebappModeTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebappModeTest.java

index 3d6c396881686be42aed31bcd56a10fcc903829d..29f9b7ae235cd252e2c28ecde6346fdab599743b 100644

--- a/chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebappModeTest.java

+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebappModeTest.java

@@ -4,7 +4,6 @@

package org.chromium.chrome.browser.webapps;

-import android.R;

import android.app.Activity;

import android.content.Context;

import android.content.Intent;

@@ -112,7 +111,7 @@ public class WebappModeTest extends MultiActivityTestBase {

public boolean isSatisfied() {

Activity lastActivity = ApplicationStatus.getLastTrackedFocusedActivity();

return lastActivity instanceof WebappActivity

- && lastActivity.findViewById(R.id.content).hasWindowFocus();

+ && lastActivity.findViewById(android.R.id.content).hasWindowFocus();

}

}));

assertTrue(isNumberOfRunningActivitiesCorrect(1));

@@ -125,7 +124,7 @@ public class WebappModeTest extends MultiActivityTestBase {

public boolean isSatisfied() {

Activity lastActivity = ApplicationStatus.getLastTrackedFocusedActivity();

return lastActivity instanceof WebappActivity && lastActivity != firstActivity

- && lastActivity.findViewById(R.id.content).hasWindowFocus();

+ && lastActivity.findViewById(android.R.id.content).hasWindowFocus();

}

}));

assertTrue(isNumberOfRunningActivitiesCorrect(2));

@@ -137,7 +136,7 @@ public class WebappModeTest extends MultiActivityTestBase {

public boolean isSatisfied() {

Activity lastActivity = ApplicationStatus.getLastTrackedFocusedActivity();

return lastActivity instanceof WebappActivity && lastActivity == firstActivity

- && lastActivity.findViewById(R.id.content).hasWindowFocus();

+ && lastActivity.findViewById(android.R.id.content).hasWindowFocus();

}

}));

assertTrue(isNumberOfRunningActivitiesCorrect(2));

@@ -157,7 +156,7 @@ public class WebappModeTest extends MultiActivityTestBase {

@Override

public boolean isSatisfied() {

Activity lastActivity = ApplicationStatus.getLastTrackedFocusedActivity();

- View rootView = lastActivity.findViewById(R.id.content);

+ View rootView = lastActivity.findViewById(android.R.id.content);

return lastActivity instanceof WebappActivity && rootView.hasWindowFocus();

}

}));

@@ -197,7 +196,7 @@ public class WebappModeTest extends MultiActivityTestBase {

@Override

public boolean isSatisfied() {

Activity lastActivity = ApplicationStatus.getLastTrackedFocusedActivity();

- if (!lastActivity.findViewById(R.id.content).hasWindowFocus()) return false;

+ if (!lastActivity.findViewById(android.R.id.content).hasWindowFocus()) return false;

return lastActivity instanceof ChromeTabbedActivity

|| lastActivity instanceof DocumentActivity;

}

@@ -211,7 +210,7 @@ public class WebappModeTest extends MultiActivityTestBase {

public boolean isSatisfied() {

Activity lastActivity = ApplicationStatus.getLastTrackedFocusedActivity();

return lastActivity instanceof WebappActivity && lastActivity != firstActivity

- && lastActivity.findViewById(R.id.content).hasWindowFocus();

+ && lastActivity.findViewById(android.R.id.content).hasWindowFocus();

}

}));

}