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

Unified Diff: chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views.cc

Issue 137993009: Remove more non-aura windows code. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: renable the disabled tests Created 6 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: chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views.cc
===================================================================
--- chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views.cc (revision 244775)
+++ chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views.cc (working copy)
@@ -5,6 +5,7 @@
#include "chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views.h"
#include "chrome/browser/browser_shutdown.h"
+#include "chrome/browser/ui/aura/tab_contents/web_drag_bookmark_handler_aura.h"
#include "chrome/browser/ui/sad_tab_helper.h"
#include "chrome/browser/ui/tab_contents/chrome_web_contents_view_delegate.h"
#include "chrome/browser/ui/views/sad_tab_view.h"
@@ -17,18 +18,12 @@
#include "content/public/browser/web_contents.h"
#include "content/public/browser/web_contents_delegate.h"
#include "content/public/browser/web_contents_view.h"
+#include "ui/aura/client/screen_position_client.h"
+#include "ui/aura/window.h"
#include "ui/views/focus/focus_manager.h"
#include "ui/views/focus/view_storage.h"
#include "ui/views/widget/widget.h"
-#if defined(USE_AURA)
-#include "chrome/browser/ui/aura/tab_contents/web_drag_bookmark_handler_aura.h"
-#include "ui/aura/client/screen_position_client.h"
-#include "ui/aura/window.h"
-#else
-#include "chrome/browser/ui/views/tab_contents/web_drag_bookmark_handler_win.h"
-#endif
-
using web_modal::WebContentsModalDialogManager;
ChromeWebContentsViewDelegateViews::ChromeWebContentsViewDelegateViews(
@@ -52,11 +47,7 @@
ChromeWebContentsViewDelegateViews::GetDragDestDelegate() {
// We install a chrome specific handler to intercept bookmark drags for the
// bookmark manager/extension API.
-#if defined(USE_AURA)
bookmark_handler_.reset(new WebDragBookmarkHandlerAura);
-#else
- bookmark_handler_.reset(new WebDragBookmarkHandlerWin);
-#endif
return bookmark_handler_.get();
}
@@ -145,7 +136,6 @@
gfx::Point screen_point(params.x, params.y);
-#if defined(USE_AURA)
// Convert from content coordinates to window coordinates.
aura::Window* web_contents_window =
web_contents_->GetView()->GetNativeView();
@@ -156,12 +146,6 @@
screen_position_client->ConvertPointToScreen(web_contents_window,
&screen_point);
}
-#else
- POINT temp = screen_point.ToPOINT();
- ClientToScreen(web_contents_->GetView()->GetNativeView(), &temp);
- screen_point = temp;
-#endif
-
// Enable recursive tasks on the message loop so we can get updates while
// the context menu is being displayed.
base::MessageLoop::ScopedNestableTaskAllower allow(
« no previous file with comments | « chrome/browser/ui/views/sad_tab_view.cc ('k') | chrome/browser/ui/views/tab_contents/web_drag_bookmark_handler_win.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698