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

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: using token to verify intent from chrome 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 1178 matching lines...) Expand 10 before | Expand all | Expand 10 after
1189 ExternalNavigationParams params = new ExternalNavigationParams.Build er( 1189 ExternalNavigationParams params = new ExternalNavigationParams.Build er(
1190 url, isIncognito(), navigationParams.referrer, 1190 url, isIncognito(), navigationParams.referrer,
1191 navigationParams.pageTransitionType, 1191 navigationParams.pageTransitionType,
1192 navigationParams.isRedirect) 1192 navigationParams.isRedirect)
1193 .setTab(ChromeTab.this) 1193 .setTab(ChromeTab.this)
1194 .setApplicationMustBeInForeground(true) 1194 .setApplicationMustBeInForeground(true)
1195 .setRedirectHandler(mTabRedirectHandler) 1195 .setRedirectHandler(mTabRedirectHandler)
1196 .setOpenInNewTab(shouldCloseTab) 1196 .setOpenInNewTab(shouldCloseTab)
1197 .setIsBackgroundTabNavigation(isHidden() && !isInitialTabLau nchInBackground) 1197 .setIsBackgroundTabNavigation(isHidden() && !isInitialTabLau nchInBackground)
1198 .setIsMainFrame(navigationParams.isMainFrame) 1198 .setIsMainFrame(navigationParams.isMainFrame)
1199 .setHasUserGesture(navigationParams.hasUserGesture)
1199 .setShouldCloseContentsOnOverrideUrlLoadingAndLaunchIntent(s houldCloseTab 1200 .setShouldCloseContentsOnOverrideUrlLoadingAndLaunchIntent(s houldCloseTab
1200 && navigationParams.isMainFrame) 1201 && navigationParams.isMainFrame)
1201 .build(); 1202 .build();
1202 ExternalNavigationHandler.OverrideUrlLoadingResult result = 1203 ExternalNavigationHandler.OverrideUrlLoadingResult result =
1203 mExternalNavHandler.shouldOverrideUrlLoading(params); 1204 mExternalNavHandler.shouldOverrideUrlLoading(params);
1204 mLastOverrideUrlLoadingResult = result; 1205 mLastOverrideUrlLoadingResult = result;
1205 switch (result) { 1206 switch (result) {
1206 case OVERRIDE_WITH_EXTERNAL_INTENT: 1207 case OVERRIDE_WITH_EXTERNAL_INTENT:
1207 assert mExternalNavHandler.canExternalAppHandleUrl(url); 1208 assert mExternalNavHandler.canExternalAppHandleUrl(url);
1208 if (navigationParams.isMainFrame) { 1209 if (navigationParams.isMainFrame) {
(...skipping 152 matching lines...) Expand 10 before | Expand all | Expand 10 after
1361 didFinishPageLoad(); 1362 didFinishPageLoad();
1362 } 1363 }
1363 } 1364 }
1364 }; 1365 };
1365 1366
1366 @VisibleForTesting 1367 @VisibleForTesting
1367 public OverrideUrlLoadingResult getLastOverrideUrlLoadingResultForTests() { 1368 public OverrideUrlLoadingResult getLastOverrideUrlLoadingResultForTests() {
1368 return mLastOverrideUrlLoadingResult; 1369 return mLastOverrideUrlLoadingResult;
1369 } 1370 }
1370 } 1371 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698