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

Side by Side Diff: content/browser/web_contents/web_contents_impl.cc

Issue 1037593003: Propagate media play/pause to WebContentsObserver for Chromecast metrics (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/web_contents/web_contents_impl.h" 5 #include "content/browser/web_contents/web_contents_impl.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/lazy_instance.h" 10 #include "base/lazy_instance.h"
(...skipping 3308 matching lines...) Expand 10 before | Expand all | Expand 10 after
3319 } 3319 }
3320 } 3320 }
3321 3321
3322 if (has_video) { 3322 if (has_video) {
3323 AddMediaPlayerEntry(player_cookie, &active_video_players_); 3323 AddMediaPlayerEntry(player_cookie, &active_video_players_);
3324 3324
3325 // If we're not hidden and have just created a player, create a blocker. 3325 // If we're not hidden and have just created a player, create a blocker.
3326 if (!video_power_save_blocker_ && !IsHidden()) 3326 if (!video_power_save_blocker_ && !IsHidden())
3327 CreateVideoPowerSaveBlocker(); 3327 CreateVideoPowerSaveBlocker();
3328 } 3328 }
3329
3330 FOR_EACH_OBSERVER(WebContentsObserver, observers_,
3331 MediaPlayingNotification());
3329 } 3332 }
3330 3333
3331 void WebContentsImpl::OnMediaPausedNotification(int64 player_cookie) { 3334 void WebContentsImpl::OnMediaPausedNotification(int64 player_cookie) {
3332 RemoveMediaPlayerEntry(player_cookie, &active_audio_players_); 3335 RemoveMediaPlayerEntry(player_cookie, &active_audio_players_);
3333 RemoveMediaPlayerEntry(player_cookie, &active_video_players_); 3336 RemoveMediaPlayerEntry(player_cookie, &active_video_players_);
3334 MaybeReleasePowerSaveBlockers(); 3337 MaybeReleasePowerSaveBlockers();
3338
3339 FOR_EACH_OBSERVER(WebContentsObserver, observers_,
3340 MediaPausedNotification());
3335 } 3341 }
3336 3342
3337 void WebContentsImpl::OnFirstVisuallyNonEmptyPaint() { 3343 void WebContentsImpl::OnFirstVisuallyNonEmptyPaint() {
3338 FOR_EACH_OBSERVER(WebContentsObserver, observers_, 3344 FOR_EACH_OBSERVER(WebContentsObserver, observers_,
3339 DidFirstVisuallyNonEmptyPaint()); 3345 DidFirstVisuallyNonEmptyPaint());
3340 3346
3341 if (theme_color_ != last_sent_theme_color_) { 3347 if (theme_color_ != last_sent_theme_color_) {
3342 // Theme color should have updated by now if there was one. 3348 // Theme color should have updated by now if there was one.
3343 FOR_EACH_OBSERVER(WebContentsObserver, observers_, 3349 FOR_EACH_OBSERVER(WebContentsObserver, observers_,
3344 DidChangeThemeColor(theme_color_)); 3350 DidChangeThemeColor(theme_color_));
(...skipping 1277 matching lines...) Expand 10 before | Expand all | Expand 10 after
4622 node->render_manager()->ResumeResponseDeferredAtStart(); 4628 node->render_manager()->ResumeResponseDeferredAtStart();
4623 } 4629 }
4624 4630
4625 void WebContentsImpl::SetForceDisableOverscrollContent(bool force_disable) { 4631 void WebContentsImpl::SetForceDisableOverscrollContent(bool force_disable) {
4626 force_disable_overscroll_content_ = force_disable; 4632 force_disable_overscroll_content_ = force_disable;
4627 if (view_) 4633 if (view_)
4628 view_->SetOverscrollControllerEnabled(CanOverscrollContent()); 4634 view_->SetOverscrollControllerEnabled(CanOverscrollContent());
4629 } 4635 }
4630 4636
4631 } // namespace content 4637 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698