Commit 27afc195 authored by James Hollyer's avatar James Hollyer Committed by Commit Bot

Reset |set_should_auto_open_bubble| when updating Media Image Model

Previously once the auto_open bubble was set it was never reset so it
continued to pop up until the browser was closed. This change resets
that on every update so it only prompts when the user selects allow on
the per site permission prompt.

Bug: 1111087
Change-Id: Ic377cfa8b764a577eaa5acbf0d416865360465fe
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2327467Reviewed-by: default avatarBalazs Engedy <engedy@chromium.org>
Commit-Queue: Balazs Engedy <engedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#793177}
parent 38bf8961
...@@ -626,6 +626,7 @@ ContentSettingMediaImageModel::ContentSettingMediaImageModel() ...@@ -626,6 +626,7 @@ ContentSettingMediaImageModel::ContentSettingMediaImageModel()
bool ContentSettingMediaImageModel::UpdateAndGetVisibility( bool ContentSettingMediaImageModel::UpdateAndGetVisibility(
WebContents* web_contents) { WebContents* web_contents) {
set_should_auto_open_bubble(false);
TabSpecificContentSettings* content_settings = TabSpecificContentSettings* content_settings =
TabSpecificContentSettings::FromWebContents(web_contents); TabSpecificContentSettings::FromWebContents(web_contents);
if (!content_settings) if (!content_settings)
......
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