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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java

Issue 1514603007: Use correct width for narrow panel in Reader Mode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comment Created 5 years 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanel.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java
index bc8550d2d0d32afe163629955ebbe5dbe3eb1b2c..9f30c738f0af9cbdd8b5e57505fb89a7e35c19e5 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java
@@ -6,7 +6,6 @@ package org.chromium.chrome.browser.compositor.bottombar.contextualsearch;
import android.content.Context;
import android.os.Handler;
-import android.view.View.MeasureSpec;
import org.chromium.base.VisibleForTesting;
import org.chromium.chrome.browser.compositor.bottombar.OverlayContentProgressObserver;
@@ -19,7 +18,6 @@ import org.chromium.chrome.browser.compositor.scene_layer.ContextualSearchSceneL
import org.chromium.chrome.browser.compositor.scene_layer.SceneLayer;
import org.chromium.chrome.browser.contextualsearch.ContextualSearchManagementDelegate;
import org.chromium.chrome.browser.preferences.PrefServiceBridge;
-import org.chromium.content.browser.ContentViewClient;
import org.chromium.ui.resources.ResourceManager;
/**
@@ -86,31 +84,8 @@ public class ContextualSearchPanel extends OverlayPanel {
@Override
public OverlayPanelContent createNewOverlayPanelContent() {
- OverlayPanelContent overlayPanelContent = new OverlayPanelContent(
- mManagementDelegate.getOverlayContentDelegate(),
- new PanelProgressObserver(),
- mActivity);
-
- // Adds a ContentViewClient to override the default fullscreen size.
- if (!isFullscreenSizePanel()) {
- overlayPanelContent.setContentViewClient(new ContentViewClient() {
- @Override
- public int getDesiredWidthMeasureSpec() {
- return MeasureSpec.makeMeasureSpec(
- getSearchContentViewWidthPx(),
- MeasureSpec.EXACTLY);
- }
-
- @Override
- public int getDesiredHeightMeasureSpec() {
- return MeasureSpec.makeMeasureSpec(
- getSearchContentViewHeightPx(),
- MeasureSpec.EXACTLY);
- }
- });
- }
-
- return overlayPanelContent;
+ return new OverlayPanelContent(mManagementDelegate.getOverlayContentDelegate(),
+ new PanelProgressObserver(), mActivity);
}
/**
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanel.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698