| Index: chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java
|
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java b/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java
|
| index 585e9db9bd4dabcb80da51f5545477d9e4a64e40..3623b757b8e32a834c28c8e4b4794ab6846636ff 100644
|
| --- a/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java
|
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java
|
| @@ -67,8 +67,6 @@ import org.chromium.chrome.browser.ChromeSwitches;
|
| import org.chromium.chrome.browser.WebsiteSettingsPopup;
|
| import org.chromium.chrome.browser.WindowDelegate;
|
| import org.chromium.chrome.browser.appmenu.AppMenuButtonHelper;
|
| -import org.chromium.chrome.browser.dom_distiller.DomDistillerServiceFactory;
|
| -import org.chromium.chrome.browser.dom_distiller.DomDistillerTabUtils;
|
| import org.chromium.chrome.browser.ntp.NativePageFactory;
|
| import org.chromium.chrome.browser.ntp.NewTabPage;
|
| import org.chromium.chrome.browser.ntp.NewTabPage.FakeboxDelegate;
|
| @@ -96,8 +94,6 @@ import org.chromium.chrome.browser.util.FeatureUtilities;
|
| import org.chromium.chrome.browser.util.KeyNavigationUtil;
|
| import org.chromium.chrome.browser.util.ViewUtils;
|
| import org.chromium.chrome.browser.widget.TintedImageButton;
|
| -import org.chromium.components.dom_distiller.core.DomDistillerService;
|
| -import org.chromium.components.dom_distiller.core.DomDistillerUrlUtils;
|
| import org.chromium.content.browser.ContentViewCore;
|
| import org.chromium.content.browser.accessibility.BrowserAccessibilityManager;
|
| import org.chromium.content_public.browser.LoadUrlParams;
|
| @@ -918,6 +914,21 @@ public class LocationBarLayout extends FrameLayout implements OnClickListener,
|
| }
|
|
|
| @Override
|
| + public void revertChanges() {
|
| + if (!mUrlHasFocus) {
|
| + setUrlToPageUrl();
|
| + } else {
|
| + Tab tab = mToolbarDataProvider.getTab();
|
| + if (NativePageFactory.isNativePageUrl(tab.getUrl(), tab.isIncognito())) {
|
| + mUrlBar.setUrl("", null);
|
| + } else {
|
| + mUrlBar.setUrl(
|
| + mToolbarDataProvider.getText(), mToolbarDataProvider.getTab().getUrl());
|
| + }
|
| + }
|
| + }
|
| +
|
| + @Override
|
| public long getFirstUrlBarFocusTime() {
|
| return mUrlBar.getFirstFocusTime();
|
| }
|
| @@ -1297,14 +1308,6 @@ public class LocationBarLayout extends FrameLayout implements OnClickListener,
|
| findViewById(R.id.location_bar_icon).setVisibility(showContainer ? VISIBLE : GONE);
|
| }
|
|
|
| - private boolean isStoredArticle(String url) {
|
| - DomDistillerService domDistillerService =
|
| - DomDistillerServiceFactory.getForProfile(Profile.getLastUsedProfile());
|
| - String entryIdFromUrl = DomDistillerUrlUtils.getValueForKeyInUrl(url, "entry_id");
|
| - if (TextUtils.isEmpty(entryIdFromUrl)) return false;
|
| - return domDistillerService.hasEntry(entryIdFromUrl);
|
| - }
|
| -
|
| /**
|
| * Updates the layout params for the location bar start aligned views.
|
| */
|
| @@ -1960,12 +1963,15 @@ public class LocationBarLayout extends FrameLayout implements OnClickListener,
|
|
|
| boolean showingQuery = false;
|
| String displayText = mToolbarDataProvider.getText();
|
| - int securityLevel = getSecurityLevel();
|
| - if (securityLevel != ConnectionSecurityLevel.SECURITY_ERROR
|
| - && !TextUtils.isEmpty(displayText) && mToolbarDataProvider.wouldReplaceURL()) {
|
| - url = displayText.trim();
|
| - showingQuery = true;
|
| - mQueryInTheOmnibox = true;
|
| + if (!TextUtils.isEmpty(displayText) && mToolbarDataProvider.wouldReplaceURL()) {
|
| + if (getSecurityLevel() == ConnectionSecurityLevel.SECURITY_ERROR) {
|
| + assert false : "Search terms should not be shown for https error pages.";
|
| + displayText = url;
|
| + } else {
|
| + url = displayText.trim();
|
| + showingQuery = true;
|
| + mQueryInTheOmnibox = true;
|
| + }
|
| }
|
| String path = null;
|
| if (!showingQuery && FeatureUtilities.isDocumentMode(getContext())) {
|
| @@ -1974,21 +1980,6 @@ public class LocationBarLayout extends FrameLayout implements OnClickListener,
|
| path = urlText.second;
|
| }
|
|
|
| - if (DomDistillerUrlUtils.isDistilledPage(url)) {
|
| - if (isStoredArticle(url)) {
|
| - DomDistillerService domDistillerService =
|
| - DomDistillerServiceFactory.getForProfile(profile);
|
| - String originalUrl = domDistillerService.getUrlForEntry(
|
| - DomDistillerUrlUtils.getValueForKeyInUrl(url, "entry_id"));
|
| - displayText =
|
| - DomDistillerTabUtils.getFormattedUrlFromOriginalDistillerUrl(originalUrl);
|
| - } else if (DomDistillerUrlUtils.getOriginalUrlFromDistillerUrl(url) != null) {
|
| - String originalUrl = DomDistillerUrlUtils.getOriginalUrlFromDistillerUrl(url);
|
| - displayText =
|
| - DomDistillerTabUtils.getFormattedUrlFromOriginalDistillerUrl(originalUrl);
|
| - }
|
| - }
|
| -
|
| if (setUrlBarText(displayText, path, url)) {
|
| mUrlBar.deEmphasizeUrl();
|
| emphasizeUrl();
|
|
|