cc/resources/video_resource_updater.cc - Issue 1142343008: cc: Rework overlays to not use the ResourceProvider and pass texture size

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: cc/resources/video_resource_updater.cc

Issue 1142343008: cc: Rework overlays to not use the ResourceProvider and pass texture size (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: move overlay vars out of DrawQuad::Resources 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: cc/resources/video_resource_updater.cc

diff --git a/cc/resources/video_resource_updater.cc b/cc/resources/video_resource_updater.cc

index d681b39365860bcd7d6b760d7cc696d781d991c8..7c4746f07d4a90c7c5191b60e18a80350c1b8666 100644

--- a/cc/resources/video_resource_updater.cc

+++ b/cc/resources/video_resource_updater.cc

@@ -417,9 +417,8 @@ VideoFrameExternalResources VideoResourceUpdater::CreateForHardwarePlanes(

const gpu::MailboxHolder& mailbox_holder = video_frame->mailbox_holder(i);

external_resources.mailboxes.push_back(

TextureMailbox(mailbox_holder.mailbox, mailbox_holder.texture_target,

- mailbox_holder.sync_point));

- external_resources.mailboxes.back().set_allow_overlay(

- video_frame->allow_overlay());

+ mailbox_holder.sync_point, video_frame->coded_size(),

+ video_frame->allow_overlay()));

external_resources.release_callbacks.push_back(

base::Bind(&ReturnTexture, AsWeakPtr(), video_frame));

}

« cc/quads/stream_video_draw_quad.cc ('K') | « cc/resources/texture_mailbox.cc ('k') | cc/surfaces/surface_aggregator_unittest.cc » ('j') | content/common/cc_messages.cc » ('J')