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

Unified Diff: chrome/browser/views/pinned_contents_info_bubble.cc

Issue 2060004: Removing the app launcher button on ChromeOS (Closed) Base URL: git://codf21.jail/chromium.git
Patch Set: One more clean up Created 10 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/views/pinned_contents_info_bubble.cc
diff --git a/chrome/browser/views/pinned_contents_info_bubble.cc b/chrome/browser/views/pinned_contents_info_bubble.cc
index 532dcdffe4963dc8e5819b187d4856bf77c0245f..c064c163b7a362eb40bfd235885a92e38d7328ee 100644
--- a/chrome/browser/views/pinned_contents_info_bubble.cc
+++ b/chrome/browser/views/pinned_contents_info_bubble.cc
@@ -6,9 +6,6 @@
#include "chrome/browser/views/bubble_border.h"
-#if defined(OS_WIN)
-// BorderWidget ---------------------------------------------------------------
-
void PinnedContentsBorderContents::SizeAndGetBounds(
const gfx::Rect& position_relative_to,
BubbleBorder::ArrowLocation arrow_location,
@@ -34,11 +31,6 @@ void PinnedContentsBorderContents::SizeAndGetBounds(
window_bounds->Offset(0, -(kTopMargin + 1));
}
-BorderContents* PinnedContentsBorderWidget::CreateBorderContents() {
- return new PinnedContentsBorderContents(bubble_anchor_);
-}
-#endif
-
// InfoBubble -----------------------------------------------------------------
// static
@@ -56,10 +48,6 @@ PinnedContentsInfoBubble* PinnedContentsInfoBubble::Show(
return window;
}
-// TODO(finnur): This needs to be implemented for other platforms once we decide
-// this is the way to go.
-#if defined(OS_WIN)
-BorderWidget* PinnedContentsInfoBubble::CreateBorderWidget() {
- return new PinnedContentsBorderWidget(bubble_anchor_);
+BorderContents* PinnedContentsInfoBubble::CreateBorderContents() {
+ return new PinnedContentsBorderContents(bubble_anchor_);
}
-#endif
« no previous file with comments | « chrome/browser/views/pinned_contents_info_bubble.h ('k') | chrome/browser/views/tabs/browser_tab_strip_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698