Commit 489c1417 authored by Bo Liu's avatar Bo Liu Committed by Commit Bot

weblayer: Remove extra onShow/onHide

The code is also incorrect as it skips checking other states.

Bug: 1084500
Change-Id: I44ef55b2c47fca901e9c6f52e5cc07834aa858d6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2215405Reviewed-by: default avatarScott Violet <sky@chromium.org>
Commit-Queue: Bo <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#771847}
parent f74063bc
...@@ -328,7 +328,6 @@ public final class TabImpl extends ITab.Stub { ...@@ -328,7 +328,6 @@ public final class TabImpl extends ITab.Stub {
TabImplJni.get().setBrowserControlsContainerViews( TabImplJni.get().setBrowserControlsContainerViews(
mNativeTab, topControlsContainerViewHandle, bottomControlsContainerViewHandle); mNativeTab, topControlsContainerViewHandle, bottomControlsContainerViewHandle);
updateWebContentsVisibility(); updateWebContentsVisibility();
mWebContents.onShow();
} }
/** /**
...@@ -336,7 +335,6 @@ public final class TabImpl extends ITab.Stub { ...@@ -336,7 +335,6 @@ public final class TabImpl extends ITab.Stub {
*/ */
public void onDidLoseActive() { public void onDidLoseActive() {
hideFindInPageUiAndNotifyClient(); hideFindInPageUiAndNotifyClient();
mWebContents.onHide();
updateWebContentsVisibility(); updateWebContentsVisibility();
TabImplJni.get().setBrowserControlsContainerViews(mNativeTab, 0, 0); TabImplJni.get().setBrowserControlsContainerViews(mNativeTab, 0, 0);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment