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

Unified Diff: content/browser/browser_plugin/browser_plugin_guest.cc

Issue 2614423003: Adding SatisfySequence and RequireSequence to SurfaceManager (Closed)
Patch Set: up 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
Index: content/browser/browser_plugin/browser_plugin_guest.cc
diff --git a/content/browser/browser_plugin/browser_plugin_guest.cc b/content/browser/browser_plugin/browser_plugin_guest.cc
index d4d4f14d881c1e6f02b7efe8578dc5bbd3785819..03de0e27ea0b8c5d25d8f9cf2fc0fc90ad919e14 100644
--- a/content/browser/browser_plugin/browser_plugin_guest.cc
+++ b/content/browser/browser_plugin/browser_plugin_guest.cc
@@ -427,23 +427,14 @@ void BrowserPluginGuest::SetChildFrameSurface(
void BrowserPluginGuest::OnSatisfySequence(
int instance_id,
const cc::SurfaceSequence& sequence) {
- std::vector<uint32_t> sequences;
- sequences.push_back(sequence.sequence);
- cc::SurfaceManager* manager = GetSurfaceManager();
- manager->DidSatisfySequences(sequence.frame_sink_id, &sequences);
+ GetSurfaceManager()->SatisfySequence(sequence);
}
void BrowserPluginGuest::OnRequireSequence(
int instance_id,
const cc::SurfaceId& id,
const cc::SurfaceSequence& sequence) {
- cc::SurfaceManager* manager = GetSurfaceManager();
- cc::Surface* surface = manager->GetSurfaceForId(id);
- if (!surface) {
- LOG(ERROR) << "Attempting to require callback on nonexistent surface";
- return;
- }
- surface->AddDestructionDependency(sequence);
+ GetSurfaceManager()->RequireSequence(id, sequence);
}
bool BrowserPluginGuest::HandleFindForEmbedder(

Powered by Google App Engine
This is Rietveld 408576698