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

Unified Diff: content/renderer/renderer_blink_platform_impl.cc

Issue 2626023006: Check that render-frame is valid before returning audio params in Blink. (Closed)
Patch Set: Add test Created 3 years, 11 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
« no previous file with comments | « content/renderer/blink_platform_audio_hardware_browsertest.cc ('k') | content/test/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/renderer_blink_platform_impl.cc
diff --git a/content/renderer/renderer_blink_platform_impl.cc b/content/renderer/renderer_blink_platform_impl.cc
index c616aa7b5a85b4b48d7a4291c6ff1afeeb951018..9a06e8893788189ef6c428b9a55d3bb61c732163 100644
--- a/content/renderer/renderer_blink_platform_impl.cc
+++ b/content/renderer/renderer_blink_platform_impl.cc
@@ -180,6 +180,9 @@ media::AudioParameters GetAudioHardwareParams() {
blink::WebLocalFrame* const web_frame =
blink::WebLocalFrame::frameForCurrentContext();
RenderFrame* const render_frame = RenderFrame::FromWebFrame(web_frame);
+ if (!render_frame)
+ return media::AudioParameters::UnavailableDeviceParams();
+
return AudioDeviceFactory::GetOutputDeviceInfo(render_frame->GetRoutingID(),
0, std::string(),
web_frame->getSecurityOrigin())
« no previous file with comments | « content/renderer/blink_platform_audio_hardware_browsertest.cc ('k') | content/test/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698