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

Side by Side Diff: content/browser/renderer_host/media/media_stream_ui_proxy.cc

Issue 1005683003: favor DCHECK_CURRENTLY_ON for better logs in content/browser/[q-z]* (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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/renderer_host/media/media_stream_ui_proxy.h" 5 #include "content/browser/renderer_host/media/media_stream_ui_proxy.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "content/browser/frame_host/frame_tree_node.h" 8 #include "content/browser/frame_host/frame_tree_node.h"
9 #include "content/browser/frame_host/render_frame_host_delegate.h" 9 #include "content/browser/frame_host/render_frame_host_delegate.h"
10 #include "content/browser/frame_host/render_frame_host_impl.h" 10 #include "content/browser/frame_host/render_frame_host_impl.h"
11 #include "content/public/browser/browser_thread.h" 11 #include "content/public/browser/browser_thread.h"
12 #include "content/public/common/content_switches.h" 12 #include "content/public/common/content_switches.h"
13 #include "media/video/capture/fake_video_capture_device.h" 13 #include "media/video/capture/fake_video_capture_device.h"
14 14
15 namespace content { 15 namespace content {
16 16
17 void SetAndCheckAncestorFlag(MediaStreamRequest* request) { 17 void SetAndCheckAncestorFlag(MediaStreamRequest* request) {
18 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 18 DCHECK_CURRENTLY_ON(BrowserThread::UI);
19 RenderFrameHostImpl* rfh = 19 RenderFrameHostImpl* rfh =
20 RenderFrameHostImpl::FromID(request->render_process_id, 20 RenderFrameHostImpl::FromID(request->render_process_id,
21 request->render_frame_id); 21 request->render_frame_id);
22 22
23 if (rfh == NULL) { 23 if (rfh == NULL) {
24 // RenderFrame destroyed before the request is handled? 24 // RenderFrame destroyed before the request is handled?
25 return; 25 return;
26 } 26 }
27 FrameTreeNode* node = rfh->frame_tree_node(); 27 FrameTreeNode* node = rfh->frame_tree_node();
28 28
(...skipping 342 matching lines...) Expand 10 before | Expand all | Expand 10 after
371 callback, 371 callback,
372 have_access)); 372 have_access));
373 return; 373 return;
374 } 374 }
375 375
376 void FakeMediaStreamUIProxy::OnStarted( 376 void FakeMediaStreamUIProxy::OnStarted(
377 const base::Closure& stop_callback, 377 const base::Closure& stop_callback,
378 const WindowIdCallback& window_id_callback) {} 378 const WindowIdCallback& window_id_callback) {}
379 379
380 } // namespace content 380 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698