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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/surfaces/display_unittest.cc

Issue 2166423002: ui::ContextFactory should not create SurfaceIdAllocators (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed TestDelegatingOutputSurface Created 4 years, 5 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 | « ash/sysui/stub_context_factory.cc ('k') | cc/surfaces/surface_id_allocator.h » ('j') | cc/test/test_delegating_output_surface.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/surfaces/display_unittest.cc
diff --git a/cc/surfaces/display_unittest.cc b/cc/surfaces/display_unittest.cc
index b17a63a03effb11b1ab63772c5372042c5594f19..74c650802d2111bf5455bbefa9bff3b5f2c95cbc 100644
--- a/cc/surfaces/display_unittest.cc
+++ b/cc/surfaces/display_unittest.cc
@@ -99,7 +99,11 @@ class DisplayTest : public testing::Test {
: factory_(&manager_, &surface_factory_client_),
id_allocator_(kArbitraryClientId),
task_runner_(new base::NullTaskRunner) {
- id_allocator_.RegisterSurfaceClientId(&manager_);
+ manager_.RegisterSurfaceClientId(id_allocator_.client_id());
+ }
+
+ ~DisplayTest() override {
+ manager_.InvalidateSurfaceClientId(id_allocator_.client_id());
}
void SetUpDisplay(const RendererSettings& settings,
« no previous file with comments | « ash/sysui/stub_context_factory.cc ('k') | cc/surfaces/surface_id_allocator.h » ('j') | cc/test/test_delegating_output_surface.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698