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

Side by Side Diff: chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationHandlerTest.java

Issue 2026953002: Disable external navigation for file downloads. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser.externalnav; 5 package org.chromium.chrome.browser.externalnav;
6 6
7 import android.annotation.SuppressLint; 7 import android.annotation.SuppressLint;
8 import android.content.Context; 8 import android.content.Context;
9 import android.content.Intent; 9 import android.content.Intent;
10 import android.content.pm.ActivityInfo; 10 import android.content.pm.ActivityInfo;
(...skipping 838 matching lines...) Expand 10 before | Expand all | Expand 10 after
849 @Override 849 @Override
850 public boolean isDocumentMode() { 850 public boolean isDocumentMode() {
851 return FeatureUtilities.isDocumentMode(mContext); 851 return FeatureUtilities.isDocumentMode(mContext);
852 } 852 }
853 853
854 @Override 854 @Override
855 public String getDefaultSmsPackageName() { 855 public String getDefaultSmsPackageName() {
856 return defaultSmsPackageName; 856 return defaultSmsPackageName;
857 } 857 }
858 858
859 @Override
860 public boolean isPdfDownload(String url) {
861 return false;
862 }
863
859 public void reset() { 864 public void reset() {
860 startActivityIntent = null; 865 startActivityIntent = null;
861 startIncognitoIntentCalled = false; 866 startIncognitoIntentCalled = false;
862 startFileIntentCalled = false; 867 startFileIntentCalled = false;
863 } 868 }
864 869
865 public void setCanResolveActivity(boolean value) { 870 public void setCanResolveActivity(boolean value) {
866 mQueryIntentOverride = value; 871 mQueryIntentOverride = value;
867 } 872 }
868 873
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
1020 return new TestPackageManager(); 1025 return new TestPackageManager();
1021 } 1026 }
1022 1027
1023 @Override 1028 @Override
1024 public String getPackageName() { 1029 public String getPackageName() {
1025 return "test.app.name"; 1030 return "test.app.name";
1026 } 1031 }
1027 1032
1028 } 1033 }
1029 } 1034 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698