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

Unified Diff: content/browser/media/capture/aura_window_capture_machine.cc

Issue 1204063005: Reland: Video Capture: extract storage info from pixel format in VideoCaptureFormat. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: dcheng@ comments Created 5 years, 6 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/media/capture/aura_window_capture_machine.cc
diff --git a/content/browser/media/capture/aura_window_capture_machine.cc b/content/browser/media/capture/aura_window_capture_machine.cc
index 87b8623a9a9cc587c567fba34ce00df9eef6892e..4b981b2c5c51114403584cd3449a741907476b0a 100644
--- a/content/browser/media/capture/aura_window_capture_machine.cc
+++ b/content/browser/media/capture/aura_window_capture_machine.cc
@@ -306,8 +306,10 @@ bool AuraWindowCaptureMachine::ProcessCopyOutputResponse(
if (result->IsEmpty() || result->size().IsEmpty() || !desktop_window_)
return false;
- if (capture_params_.requested_format.pixel_format ==
- media::PIXEL_FORMAT_TEXTURE) {
+ if (capture_params_.requested_format.pixel_storage ==
+ media::PIXEL_STORAGE_TEXTURE) {
+ DCHECK_EQ(capture_params_.requested_format.pixel_format,
dcheng 2015/06/24 21:41:47 The arguments to this macro should be reversed.
mcasas 2015/06/25 01:19:51 Done.
+ media::PIXEL_FORMAT_ARGB);
DCHECK(!video_frame.get());
cc::TextureMailbox texture_mailbox;
scoped_ptr<cc::SingleReleaseCallback> release_callback;

Powered by Google App Engine
This is Rietveld 408576698