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

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

Issue 2752113005: Let ImeAdapterAndroid have the same lifecycle as its Java peer (Closed)
Patch Set: fix tests 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 2303 matching lines...) Expand 10 before | Expand all | Expand 10 after
2314 // (zero) to the renderer process, although the new size will be set soo n. 2314 // (zero) to the renderer process, although the new size will be set soo n.
2315 // However, this size fluttering may confuse Blink and rendered result c an be broken 2315 // However, this size fluttering may confuse Blink and rendered result c an be broken
2316 // (see http://crbug.com/340987). 2316 // (see http://crbug.com/340987).
2317 newContentViewCore.onSizeChanged(originalWidth, originalHeight, 0, 0); 2317 newContentViewCore.onSizeChanged(originalWidth, originalHeight, 0, 0);
2318 if (!bounds.isEmpty()) { 2318 if (!bounds.isEmpty()) {
2319 newContentViewCore.onPhysicalBackingSizeChanged(bounds.right, bounds .bottom); 2319 newContentViewCore.onPhysicalBackingSizeChanged(bounds.right, bounds .bottom);
2320 } 2320 }
2321 newContentViewCore.onShow(); 2321 newContentViewCore.onShow();
2322 setContentViewCore(newContentViewCore); 2322 setContentViewCore(newContentViewCore);
2323 2323
2324 mContentViewCore.attachImeAdapter();
2325
2326 // If the URL has already committed (e.g. prerendering), tell process ma nagement logic that 2324 // If the URL has already committed (e.g. prerendering), tell process ma nagement logic that
2327 // it can rely on the process visibility signal for binding management. 2325 // it can rely on the process visibility signal for binding management.
2328 // TODO: Call ChildProcessLauncher#determinedVisibility() at a more intu itive time. 2326 // TODO: Call ChildProcessLauncher#determinedVisibility() at a more intu itive time.
2329 // See crbug.com/537671 2327 // See crbug.com/537671
2330 if (!mContentViewCore.getWebContents().getLastCommittedUrl().equals("")) { 2328 if (!mContentViewCore.getWebContents().getLastCommittedUrl().equals("")) {
2331 ChildProcessLauncher.determinedVisibility(mContentViewCore.getCurren tRenderProcessId()); 2329 ChildProcessLauncher.determinedVisibility(mContentViewCore.getCurren tRenderProcessId());
2332 } 2330 }
2333 2331
2334 destroyNativePageInternal(previousNativePage); 2332 destroyNativePageInternal(previousNativePage);
2335 for (TabObserver observer : mObservers) { 2333 for (TabObserver observer : mObservers) {
(...skipping 752 matching lines...) Expand 10 before | Expand all | Expand 10 after
3088 private native boolean nativeIsOfflinePage(long nativeTabAndroid); 3086 private native boolean nativeIsOfflinePage(long nativeTabAndroid);
3089 private native OfflinePageItem nativeGetOfflinePage(long nativeTabAndroid); 3087 private native OfflinePageItem nativeGetOfflinePage(long nativeTabAndroid);
3090 private native void nativeSetInterceptNavigationDelegate(long nativeTabAndro id, 3088 private native void nativeSetInterceptNavigationDelegate(long nativeTabAndro id,
3091 InterceptNavigationDelegate delegate); 3089 InterceptNavigationDelegate delegate);
3092 private native void nativeAttachToTabContentManager(long nativeTabAndroid, 3090 private native void nativeAttachToTabContentManager(long nativeTabAndroid,
3093 TabContentManager tabContentManager); 3091 TabContentManager tabContentManager);
3094 private native boolean nativeHasPrerenderedUrl(long nativeTabAndroid, String url); 3092 private native boolean nativeHasPrerenderedUrl(long nativeTabAndroid, String url);
3095 private native void nativeSetWebappManifestScope(long nativeTabAndroid, Stri ng scope); 3093 private native void nativeSetWebappManifestScope(long nativeTabAndroid, Stri ng scope);
3096 private native void nativeEnableEmbeddedMediaExperience(long nativeTabAndroi d, boolean enabled); 3094 private native void nativeEnableEmbeddedMediaExperience(long nativeTabAndroi d, boolean enabled);
3097 } 3095 }
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