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

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

Issue 2785543003: Revert of Let ImeAdapterAndroid have the same lifecycle as its Java peer (Closed)
Patch Set: Created 3 years, 8 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
« no previous file with comments | « no previous file | chrome/android/javatests/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchTapEventTest.java » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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.SuppressLint; 7 import android.annotation.SuppressLint;
8 import android.app.Activity; 8 import android.app.Activity;
9 import android.app.Application; 9 import android.app.Application;
10 import android.content.Context; 10 import android.content.Context;
(...skipping 2291 matching lines...) Expand 10 before | Expand all | Expand 10 after
2302 // (zero) to the renderer process, although the new size will be set soo n. 2302 // (zero) to the renderer process, although the new size will be set soo n.
2303 // However, this size fluttering may confuse Blink and rendered result c an be broken 2303 // However, this size fluttering may confuse Blink and rendered result c an be broken
2304 // (see http://crbug.com/340987). 2304 // (see http://crbug.com/340987).
2305 newContentViewCore.onSizeChanged(originalWidth, originalHeight, 0, 0); 2305 newContentViewCore.onSizeChanged(originalWidth, originalHeight, 0, 0);
2306 if (!bounds.isEmpty()) { 2306 if (!bounds.isEmpty()) {
2307 newContentViewCore.onPhysicalBackingSizeChanged(bounds.right, bounds .bottom); 2307 newContentViewCore.onPhysicalBackingSizeChanged(bounds.right, bounds .bottom);
2308 } 2308 }
2309 newContentViewCore.onShow(); 2309 newContentViewCore.onShow();
2310 setContentViewCore(newContentViewCore); 2310 setContentViewCore(newContentViewCore);
2311 2311
2312 mContentViewCore.attachImeAdapter();
2313
2312 // If the URL has already committed (e.g. prerendering), tell process ma nagement logic that 2314 // If the URL has already committed (e.g. prerendering), tell process ma nagement logic that
2313 // it can rely on the process visibility signal for binding management. 2315 // it can rely on the process visibility signal for binding management.
2314 // TODO: Call ChildProcessLauncher#determinedVisibility() at a more intu itive time. 2316 // TODO: Call ChildProcessLauncher#determinedVisibility() at a more intu itive time.
2315 // See crbug.com/537671 2317 // See crbug.com/537671
2316 if (!mContentViewCore.getWebContents().getLastCommittedUrl().equals("")) { 2318 if (!mContentViewCore.getWebContents().getLastCommittedUrl().equals("")) {
2317 ChildProcessLauncher.determinedVisibility(mContentViewCore.getCurren tRenderProcessId()); 2319 ChildProcessLauncher.determinedVisibility(mContentViewCore.getCurren tRenderProcessId());
2318 } 2320 }
2319 2321
2320 destroyNativePageInternal(previousNativePage); 2322 destroyNativePageInternal(previousNativePage);
2321 for (TabObserver observer : mObservers) { 2323 for (TabObserver observer : mObservers) {
(...skipping 752 matching lines...) Expand 10 before | Expand all | Expand 10 after
3074 private native boolean nativeIsOfflinePage(long nativeTabAndroid); 3076 private native boolean nativeIsOfflinePage(long nativeTabAndroid);
3075 private native OfflinePageItem nativeGetOfflinePage(long nativeTabAndroid); 3077 private native OfflinePageItem nativeGetOfflinePage(long nativeTabAndroid);
3076 private native void nativeSetInterceptNavigationDelegate(long nativeTabAndro id, 3078 private native void nativeSetInterceptNavigationDelegate(long nativeTabAndro id,
3077 InterceptNavigationDelegate delegate); 3079 InterceptNavigationDelegate delegate);
3078 private native void nativeAttachToTabContentManager(long nativeTabAndroid, 3080 private native void nativeAttachToTabContentManager(long nativeTabAndroid,
3079 TabContentManager tabContentManager); 3081 TabContentManager tabContentManager);
3080 private native boolean nativeHasPrerenderedUrl(long nativeTabAndroid, String url); 3082 private native boolean nativeHasPrerenderedUrl(long nativeTabAndroid, String url);
3081 private native void nativeSetWebappManifestScope(long nativeTabAndroid, Stri ng scope); 3083 private native void nativeSetWebappManifestScope(long nativeTabAndroid, Stri ng scope);
3082 private native void nativeEnableEmbeddedMediaExperience(long nativeTabAndroi d, boolean enabled); 3084 private native void nativeEnableEmbeddedMediaExperience(long nativeTabAndroi d, boolean enabled);
3083 } 3085 }
OLDNEW
« no previous file with comments | « no previous file | chrome/android/javatests/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchTapEventTest.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698