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

Unified Diff: components/view_manager/display_manager.cc

Issue 1308973005: html_viewer/web_view: An app for running layout-tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix-win-build Created 5 years, 3 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: components/view_manager/display_manager.cc
diff --git a/components/view_manager/display_manager.cc b/components/view_manager/display_manager.cc
index e3c4201b8ee94e5c3ceefecc0b1a55706ad305e1..f592d453559e8d05abab87d93ace61d2ea6fefde 100644
--- a/components/view_manager/display_manager.cc
+++ b/components/view_manager/display_manager.cc
@@ -158,6 +158,7 @@ DefaultDisplayManager::~DefaultDisplayManager() {
// Destroy the PlatformWindow early on as it may call us back during
// destruction and we want to be in a known state. But destroy the surface
// first because it can still be using the platform window.
+ dirty_rect_ = gfx::Rect();
sky 2015/09/03 19:31:59 Why do you need to reset the dirty_rect_ here? Ass
sadrul 2015/09/03 21:49:00 Looks like I don't need this anymore. Removed.
top_level_display_client_.reset();
platform_window_.reset();
}

Powered by Google App Engine
This is Rietveld 408576698