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

Unified Diff: chrome/browser/ui/views/frame/contents_layout_manager.cc

Issue 2897533003: Change default ink drop center points to center of contents bounds (Closed)
Patch Set: fix compile Created 3 years, 7 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: chrome/browser/ui/views/frame/contents_layout_manager.cc
diff --git a/chrome/browser/ui/views/frame/contents_layout_manager.cc b/chrome/browser/ui/views/frame/contents_layout_manager.cc
index a9471ffc61bcc2934a91de683650137e95f6cf31..98b2a2cb26f7794287edda9bd1b5add7806df952 100644
--- a/chrome/browser/ui/views/frame/contents_layout_manager.cc
+++ b/chrome/browser/ui/views/frame/contents_layout_manager.cc
@@ -55,11 +55,8 @@ void ContentsLayoutManager::Layout(views::View* contents_container) {
// DevTools cares about the specific position, so we have to compensate RTL
// layout here.
- new_devtools_bounds.set_x(host_->GetMirroredXForRect(new_devtools_bounds));
- new_contents_bounds.set_x(host_->GetMirroredXForRect(new_contents_bounds));
-
- devtools_view_->SetBoundsRect(new_devtools_bounds);
- contents_view_->SetBoundsRect(new_contents_bounds);
+ devtools_view_->SetBoundsRect(host_->GetMirroredRect(new_devtools_bounds));
+ contents_view_->SetBoundsRect(host_->GetMirroredRect(new_contents_bounds));
}
gfx::Size ContentsLayoutManager::GetPreferredSize(

Powered by Google App Engine
This is Rietveld 408576698