Commit 4d7bb41c authored by Yafei Duan's avatar Yafei Duan Committed by Commit Bot

Revert "[NTP tiles] Don't enable homepage tile if home page url is google.com."

This reverts commit ef042f5e.

Reason for revert: We don't want to skip pinning google.com as tile, per discussion with nickrad@.

Original change's description:
> [NTP tiles] Don't enable homepage tile if home page url is google.com.
> 
> Homepage tile should not be added to Most Visited tiles if the homepage
> url is google.com.
> 
> Bug: 862786
> Change-Id: Icb42ce4fb5f0c26bece80b7a7dd82bfd8526bcb2
> Reviewed-on: https://chromium-review.googlesource.com/1152298
> Reviewed-by: Theresa <twellington@chromium.org>
> Commit-Queue: Yafei Duan <romax@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578788}

TBR=twellington@chromium.org,romax@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 862786
Change-Id: Ifca7d377d5b3ff527cddbecafc63b3909c41cd53
Reviewed-on: https://chromium-review.googlesource.com/1157385Reviewed-by: default avatarYafei Duan <romax@chromium.org>
Commit-Queue: Yafei Duan <romax@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579620}
parent 0bf13621
......@@ -12,7 +12,6 @@ import org.chromium.chrome.browser.ntp.NewTabPage;
import org.chromium.chrome.browser.partnercustomizations.HomepageManager;
import org.chromium.chrome.browser.profiles.Profile;
import org.chromium.chrome.browser.util.FeatureUtilities;
import org.chromium.chrome.browser.util.UrlUtilities;
import java.util.ArrayList;
import java.util.Date;
......@@ -50,7 +49,6 @@ public class MostVisitedSitesBridge
public boolean isHomepageTileEnabled() {
return HomepageManager.isHomepageEnabled()
&& !NewTabPage.isNTPUrl(getHomepageUrl())
&& !UrlUtilities.nativeIsGoogleHomePageUrl(getHomepageUrl())
&& !TextUtils.isEmpty(HomepageManager.getHomepageUri());
}
......
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