Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1192)

Unified Diff: content/browser/web_contents/web_contents_impl.cc

Issue 2692903002: Don't show media controls when there is a persistent video. (Closed)
Patch Set: cleanup and tests Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 30cc8989e92062def7bacd32f14c6e30c5107a27..482650c8d4ed42fc954c051fa8c17a6e29eaa0b8 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -4254,6 +4254,10 @@ bool WebContentsImpl::HideDownloadUI() const {
return is_overlay_content_;
}
+bool WebContentsImpl::HasPersistentVideo() const {
+ return has_persistent_video_;
+}
+
bool WebContentsImpl::IsFocusedElementEditable() {
RenderFrameHostImpl* frame = GetFocusedFrame();
return frame && frame->has_focused_editable_element();
@@ -5280,6 +5284,14 @@ void WebContentsImpl::DecrementBluetoothConnectedDeviceCount() {
}
}
+void WebContentsImpl::SetHasPersistentVideo(bool value) {
+ if (has_persistent_video_ == value)
+ return;
+
+ has_persistent_video_ = value;
+ NotifyPreferencesChanged();
+}
+
#if defined(OS_ANDROID)
void WebContentsImpl::NotifyFindMatchRectsReply(
int version,
@@ -5344,19 +5356,7 @@ void WebContentsImpl::UpdateWebContentsVisibility(bool visible) {
}
void WebContentsImpl::UpdateOverridingUserAgent() {
- std::set<RenderViewHost*> render_view_host_set;
- for (FrameTreeNode* node : frame_tree_.Nodes()) {
- RenderWidgetHost* render_widget_host =
- node->current_frame_host()->GetRenderWidgetHost();
- if (!render_widget_host)
- continue;
- RenderViewHost* render_view_host = RenderViewHost::From(render_widget_host);
- if (!render_view_host)
- continue;
- render_view_host_set.insert(render_view_host);
- }
- for (RenderViewHost* render_view_host : render_view_host_set)
- render_view_host->OnWebkitPreferencesChanged();
+ NotifyPreferencesChanged();
}
void WebContentsImpl::SetJavaScriptDialogManagerForTesting(
@@ -5417,4 +5417,20 @@ void WebContentsImpl::ShowInsecureLocalhostWarningIfNeeded() {
" releasing your website to the public."));
}
+void WebContentsImpl::NotifyPreferencesChanged() {
+ std::set<RenderViewHost*> render_view_host_set;
+ for (FrameTreeNode* node : frame_tree_.Nodes()) {
+ RenderWidgetHost* render_widget_host =
+ node->current_frame_host()->GetRenderWidgetHost();
+ if (!render_widget_host)
+ continue;
+ RenderViewHost* render_view_host = RenderViewHost::From(render_widget_host);
+ if (!render_view_host)
+ continue;
+ render_view_host_set.insert(render_view_host);
+ }
+ for (RenderViewHost* render_view_host : render_view_host_set)
+ render_view_host->OnWebkitPreferencesChanged();
+}
+
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698