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

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

Issue 507373002: MacViews: All targets compile. Fixes unit_tests and message_center_unittests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@20140827-MacViews-WebContentsViewDelegateMac
Patch Set: +comment, +gn changes Created 6 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
« no previous file with comments | « chrome/browser/ui/views/confirm_bubble_views_unittest.cc ('k') | chrome/chrome_tests_unit.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views_unittest.cc
diff --git a/chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views_unittest.cc b/chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views_unittest.cc
index d69f48af75b3690774c97ab4e1133b764a89fb9d..bff7489f55a850e18731fbb68df39f5a6d5620f7 100644
--- a/chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views_unittest.cc
+++ b/chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views_unittest.cc
@@ -4,24 +4,18 @@
#include "chrome/browser/ui/views/tab_contents/chrome_web_contents_view_delegate_views.h"
+#include "chrome/browser/ui/tab_contents/chrome_web_contents_view_delegate.h"
#include "chrome/test/base/chrome_render_view_host_test_harness.h"
#include "content/public/browser/web_contents.h"
#include "content/public/common/context_menu_params.h"
-class ChromeWebContentsViewDelegateViewsTest
- : public ChromeRenderViewHostTestHarness {
- public:
- ChromeWebContentsViewDelegateViewsTest() {}
-
- private:
- DISALLOW_COPY_AND_ASSIGN(ChromeWebContentsViewDelegateViewsTest);
-};
+typedef ChromeRenderViewHostTestHarness ChromeWebContentsViewDelegateViewsTest;
// Test that makes sure ShowContextMenu does not crash if web_contents() does
// not have a focused frame.
TEST_F(ChromeWebContentsViewDelegateViewsTest, ContextMenuNoFocusedFrame) {
- scoped_ptr<ChromeWebContentsViewDelegateViews> delegate_view(
- new ChromeWebContentsViewDelegateViews(web_contents()));
+ scoped_ptr<content::WebContentsViewDelegate> delegate_view(
+ chrome::CreateWebContentsViewDelegate(web_contents()));
EXPECT_FALSE(web_contents()->GetFocusedFrame());
const content::ContextMenuParams params;
delegate_view->ShowContextMenu(web_contents()->GetMainFrame(), params);
« no previous file with comments | « chrome/browser/ui/views/confirm_bubble_views_unittest.cc ('k') | chrome/chrome_tests_unit.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698