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

Unified Diff: ui/ozone/platform/dri/gpu_platform_support_gbm.cc

Issue 469343003: [Ozone-GBM] Pumb DriWindowDelegate throughout the platform (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix unittest && rebase Created 6 years, 4 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: ui/ozone/platform/dri/gpu_platform_support_gbm.cc
diff --git a/ui/ozone/platform/dri/gpu_platform_support_gbm.cc b/ui/ozone/platform/dri/gpu_platform_support_gbm.cc
index c18084e81b182449f746b384568cba67597582f8..298ad765ec5000dcb0563ab4955644ebcac647c1 100644
--- a/ui/ozone/platform/dri/gpu_platform_support_gbm.cc
+++ b/ui/ozone/platform/dri/gpu_platform_support_gbm.cc
@@ -66,21 +66,14 @@ void GpuPlatformSupportGbm::OnCreateWindowDelegate(
scoped_ptr<DriWindowDelegate> delegate(
new DriWindowDelegateImpl(widget, screen_manager_));
delegate->Initialize();
- window_manager_->AddWindowDelegate(widget, delegate.get());
-
- std::pair<WidgetToWindowDelegateMap::iterator, bool> result =
- window_delegate_owner_.add(widget, delegate.Pass());
- DCHECK(result.second) << "Delegate already added.";
+ window_manager_->AddWindowDelegate(widget, delegate.Pass());
}
}
void GpuPlatformSupportGbm::OnDestroyWindowDelegate(
gfx::AcceleratedWidget widget) {
scoped_ptr<DriWindowDelegate> delegate =
- window_delegate_owner_.take_and_erase(widget);
- DCHECK(delegate) << "Attempting to remove non-existing delegate.";
-
- window_manager_->RemoveWindowDelegate(widget);
+ window_manager_->RemoveWindowDelegate(widget);
delegate->Shutdown();
}
« no previous file with comments | « ui/ozone/platform/dri/gpu_platform_support_gbm.h ('k') | ui/ozone/platform/dri/hardware_display_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698