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

Unified Diff: content/browser/media/session/pepper_player_delegate.cc

Issue 2551093002: Route Pepper MediaSession messages to frames (Closed)
Patch Set: fixed Android build Created 4 years 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/media/session/pepper_player_delegate.cc
diff --git a/content/browser/media/session/pepper_player_delegate.cc b/content/browser/media/session/pepper_player_delegate.cc
index b1d0d671165ee7942eb7dce1947edaddce223415..5e6f168990494f14148d58044f884afb57de00d3 100644
--- a/content/browser/media/session/pepper_player_delegate.cc
+++ b/content/browser/media/session/pepper_player_delegate.cc
@@ -6,7 +6,6 @@
#include "content/browser/frame_host/render_frame_host_impl.h"
#include "content/browser/media/session/pepper_playback_observer.h"
-#include "content/browser/web_contents/web_contents_impl.h"
#include "content/common/frame_messages.h"
#include "media/base/media_switches.h"
@@ -27,8 +26,8 @@ bool ShouldDuckFlash() {
const int PepperPlayerDelegate::kPlayerId = 0;
PepperPlayerDelegate::PepperPlayerDelegate(
- WebContentsImpl* contents, int32_t pp_instance)
- : contents_(contents),
+ RenderFrameHost* render_frame_host, int32_t pp_instance)
+ : render_frame_host_(render_frame_host),
pp_instance_(pp_instance) {}
PepperPlayerDelegate::~PepperPlayerDelegate() = default;
@@ -60,13 +59,12 @@ void PepperPlayerDelegate::OnSetVolumeMultiplier(int player_id,
}
RenderFrameHost* PepperPlayerDelegate::GetRenderFrameHost() const {
- // TODO(zqzhang): Pepper player should be associated to a RenderFrameHost.
- return nullptr;
+ return render_frame_host_;
}
void PepperPlayerDelegate::SetVolume(int player_id, double volume) {
- contents_->Send(new FrameMsg_SetPepperVolume(
- contents_->GetMainFrame()->routing_id(), pp_instance_, volume));
+ render_frame_host_->Send(new FrameMsg_SetPepperVolume(
+ render_frame_host_->GetRoutingID(), pp_instance_, volume));
}
} // namespace content
« no previous file with comments | « content/browser/media/session/pepper_player_delegate.h ('k') | content/browser/web_contents/web_contents_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698