Commit de058112 authored by Stepan Khapugin's avatar Stepan Khapugin Committed by Commit Bot

[iOS] Disable page info on NTP.

Disable Site Info on the NTP to avoid user's confusion.

Bug: 977056
Change-Id: Ia47495580d8ec7a3eee3347e22f91d7c63647026
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1719009
Auto-Submit: Stepan Khapugin <stkhapugin@chromium.org>
Commit-Queue: Joe DeBlasio <jdeblasio@chromium.org>
Reviewed-by: default avatarGauthier Ambard <gambard@chromium.org>
Reviewed-by: default avatarJoe DeBlasio <jdeblasio@chromium.org>
Cr-Commit-Position: refs/heads/master@{#681875}
parent 2ff3c773
...@@ -571,6 +571,12 @@ PopupMenuToolsItem* CreateTableViewItem(int titleID, ...@@ -571,6 +571,12 @@ PopupMenuToolsItem* CreateTableViewItem(int titleID,
if (URL.SchemeIs(kChromeUIScheme) && URL.host() == kChromeUIOfflineHost) { if (URL.SchemeIs(kChromeUIScheme) && URL.host() == kChromeUIOfflineHost) {
return YES; return YES;
} }
// Do not show site info for NTP.
if (URL.spec() == kChromeUIAboutNewTabURL ||
URL.spec() == kChromeUINewTabURL) {
return NO;
}
return navItem->GetVirtualURL().is_valid(); return navItem->GetVirtualURL().is_valid();
} }
......
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