Commit f55d1c53 authored by Kamila's avatar Kamila Committed by Commit Bot

Not showing Site settings button for Guest mode

In Guest mode, Site Settings link found in the Page info bubble, leads to
chrome://settings/, not to the site content settings, as the content settings
are not visible.

Changes:
-Removed the Site Settings button from guest mode.

Note: Enabling just the Site Settings would require a complicated refactoring
of privacy section.

Bug: 823703
Change-Id: Ie3f6ecb3c7b11eeb7652808eefb8608222525830
Reviewed-on: https://chromium-review.googlesource.com/1185097
Commit-Queue: Kamila Hasanbega <hkamila@google.com>
Reviewed-by: default avatarChristian Dullweber <dullweber@chromium.org>
Reviewed-by: default avatarJochen Eisinger <jochen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#585773}
parent ba288e55
...@@ -530,7 +530,8 @@ PageInfoBubbleView::PageInfoBubbleView( ...@@ -530,7 +530,8 @@ PageInfoBubbleView::PageInfoBubbleView(
layout->StartRowWithPadding(views::GridLayout::kFixedSize, kColumnId, layout->StartRowWithPadding(views::GridLayout::kFixedSize, kColumnId,
views::GridLayout::kFixedSize, 0); views::GridLayout::kFixedSize, 0);
layout->AddView(CreateSiteSettingsLink(side_margin, this).release()); if (!profile->IsGuestSession())
layout->AddView(CreateSiteSettingsLink(side_margin, this).release());
views::BubbleDialogDelegateView::CreateBubble(this); views::BubbleDialogDelegateView::CreateBubble(this);
presenter_.reset(new PageInfo( presenter_.reset(new PageInfo(
......
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