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

Side by Side Diff: chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java

Issue 2898373002: Redirects _blank and window.open() off-origin navigation from PWA to CCT. (Closed)
Patch Set: Merge Created 3 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.webapps; 5 package org.chromium.chrome.browser.webapps;
6 6
7 import android.content.Intent; 7 import android.content.Intent;
8 import android.graphics.Bitmap; 8 import android.graphics.Bitmap;
9 import android.graphics.Color; 9 import android.graphics.Color;
10 import android.graphics.drawable.Drawable; 10 import android.graphics.drawable.Drawable;
(...skipping 21 matching lines...) Expand all
32 import org.chromium.chrome.R; 32 import org.chromium.chrome.R;
33 import org.chromium.chrome.browser.IntentHandler; 33 import org.chromium.chrome.browser.IntentHandler;
34 import org.chromium.chrome.browser.TabState; 34 import org.chromium.chrome.browser.TabState;
35 import org.chromium.chrome.browser.document.DocumentUtils; 35 import org.chromium.chrome.browser.document.DocumentUtils;
36 import org.chromium.chrome.browser.fullscreen.ChromeFullscreenManager; 36 import org.chromium.chrome.browser.fullscreen.ChromeFullscreenManager;
37 import org.chromium.chrome.browser.metrics.WebappUma; 37 import org.chromium.chrome.browser.metrics.WebappUma;
38 import org.chromium.chrome.browser.tab.EmptyTabObserver; 38 import org.chromium.chrome.browser.tab.EmptyTabObserver;
39 import org.chromium.chrome.browser.tab.Tab; 39 import org.chromium.chrome.browser.tab.Tab;
40 import org.chromium.chrome.browser.tab.TabDelegateFactory; 40 import org.chromium.chrome.browser.tab.TabDelegateFactory;
41 import org.chromium.chrome.browser.tab.TabObserver; 41 import org.chromium.chrome.browser.tab.TabObserver;
42 import org.chromium.chrome.browser.tabmodel.document.TabDelegate;
42 import org.chromium.chrome.browser.util.ColorUtils; 43 import org.chromium.chrome.browser.util.ColorUtils;
43 import org.chromium.chrome.browser.util.UrlUtilities; 44 import org.chromium.chrome.browser.util.UrlUtilities;
44 import org.chromium.content.browser.ScreenOrientationProvider; 45 import org.chromium.content.browser.ScreenOrientationProvider;
45 import org.chromium.content_public.browser.LoadUrlParams; 46 import org.chromium.content_public.browser.LoadUrlParams;
46 import org.chromium.net.NetworkChangeNotifier; 47 import org.chromium.net.NetworkChangeNotifier;
47 import org.chromium.ui.base.PageTransition; 48 import org.chromium.ui.base.PageTransition;
48 49
49 import java.io.File; 50 import java.io.File;
50 import java.util.concurrent.TimeUnit; 51 import java.util.concurrent.TimeUnit;
51 52
(...skipping 655 matching lines...) Expand 10 before | Expand all | Expand 10 after
707 @Override 708 @Override
708 protected Drawable getBackgroundDrawable() { 709 protected Drawable getBackgroundDrawable() {
709 return null; 710 return null;
710 } 711 }
711 712
712 @Override 713 @Override
713 protected TabDelegateFactory createTabDelegateFactory() { 714 protected TabDelegateFactory createTabDelegateFactory() {
714 return new WebappDelegateFactory(this); 715 return new WebappDelegateFactory(this);
715 } 716 }
716 717
718 @Override
719 protected TabDelegate createTabDelegate(boolean incognito) {
720 return new WebappTabDelegate(this, incognito);
721 }
722
717 // We're temporarily disable CS on webapp since there are some issues. (http ://crbug.com/471950) 723 // We're temporarily disable CS on webapp since there are some issues. (http ://crbug.com/471950)
718 // TODO(changwan): re-enable it once the issues are resolved. 724 // TODO(changwan): re-enable it once the issues are resolved.
719 @Override 725 @Override
720 protected boolean isContextualSearchAllowed() { 726 protected boolean isContextualSearchAllowed() {
721 return false; 727 return false;
722 } 728 }
723 } 729 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698