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

Side by Side Diff: chrome/android/java/src/org/chromium/chrome/browser/tab/ChromeTab.java

Issue 1243253004: Pass user gesture bit when chrome handles an intent fired by itself (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix findbugs warning Created 5 years, 4 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.tab; 5 package org.chromium.chrome.browser.tab;
6 6
7 import android.annotation.TargetApi; 7 import android.annotation.TargetApi;
8 import android.app.ActivityManager; 8 import android.app.ActivityManager;
9 import android.content.Context; 9 import android.content.Context;
10 import android.content.Intent; 10 import android.content.Intent;
(...skipping 1186 matching lines...) Expand 10 before | Expand all | Expand 10 after
1197 ExternalNavigationParams params = new ExternalNavigationParams.Build er( 1197 ExternalNavigationParams params = new ExternalNavigationParams.Build er(
1198 url, isIncognito(), navigationParams.referrer, 1198 url, isIncognito(), navigationParams.referrer,
1199 navigationParams.pageTransitionType, 1199 navigationParams.pageTransitionType,
1200 navigationParams.isRedirect) 1200 navigationParams.isRedirect)
1201 .setTab(ChromeTab.this) 1201 .setTab(ChromeTab.this)
1202 .setApplicationMustBeInForeground(true) 1202 .setApplicationMustBeInForeground(true)
1203 .setRedirectHandler(mTabRedirectHandler) 1203 .setRedirectHandler(mTabRedirectHandler)
1204 .setOpenInNewTab(shouldCloseTab) 1204 .setOpenInNewTab(shouldCloseTab)
1205 .setIsBackgroundTabNavigation(isHidden() && !isInitialTabLau nchInBackground) 1205 .setIsBackgroundTabNavigation(isHidden() && !isInitialTabLau nchInBackground)
1206 .setIsMainFrame(navigationParams.isMainFrame) 1206 .setIsMainFrame(navigationParams.isMainFrame)
1207 .setHasUserGesture(navigationParams.hasUserGesture)
1207 .setShouldCloseContentsOnOverrideUrlLoadingAndLaunchIntent(s houldCloseTab 1208 .setShouldCloseContentsOnOverrideUrlLoadingAndLaunchIntent(s houldCloseTab
1208 && navigationParams.isMainFrame) 1209 && navigationParams.isMainFrame)
1209 .build(); 1210 .build();
1210 ExternalNavigationHandler.OverrideUrlLoadingResult result = 1211 ExternalNavigationHandler.OverrideUrlLoadingResult result =
1211 mExternalNavHandler.shouldOverrideUrlLoading(params); 1212 mExternalNavHandler.shouldOverrideUrlLoading(params);
1212 mLastOverrideUrlLoadingResult = result; 1213 mLastOverrideUrlLoadingResult = result;
1213 switch (result) { 1214 switch (result) {
1214 case OVERRIDE_WITH_EXTERNAL_INTENT: 1215 case OVERRIDE_WITH_EXTERNAL_INTENT:
1215 assert mExternalNavHandler.canExternalAppHandleUrl(url); 1216 assert mExternalNavHandler.canExternalAppHandleUrl(url);
1216 if (navigationParams.isMainFrame) { 1217 if (navigationParams.isMainFrame) {
(...skipping 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
1374 didFinishPageLoad(); 1375 didFinishPageLoad();
1375 } 1376 }
1376 } 1377 }
1377 }; 1378 };
1378 1379
1379 @VisibleForTesting 1380 @VisibleForTesting
1380 public OverrideUrlLoadingResult getLastOverrideUrlLoadingResultForTests() { 1381 public OverrideUrlLoadingResult getLastOverrideUrlLoadingResultForTests() {
1381 return mLastOverrideUrlLoadingResult; 1382 return mLastOverrideUrlLoadingResult;
1382 } 1383 }
1383 } 1384 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698