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

Unified Diff: chrome/browser/ui/views/toolbar/site_chip_view.h

Issue 75873002: [SiteChip] Add the basic painting of the site chip button (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Use MediumFont Created 7 years 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/toolbar/site_chip_view.h
diff --git a/chrome/browser/ui/views/toolbar/site_chip_view.h b/chrome/browser/ui/views/toolbar/site_chip_view.h
new file mode 100644
index 0000000000000000000000000000000000000000..4d52104bd362852523d91096b25333af7f6f9458
--- /dev/null
+++ b/chrome/browser/ui/views/toolbar/site_chip_view.h
@@ -0,0 +1,62 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_VIEWS_TOOLBAR_SITE_CHIP_VIEW_H_
+#define CHROME_BROWSER_UI_VIEWS_TOOLBAR_SITE_CHIP_VIEW_H_
+
+#include "chrome/browser/ui/views/location_bar/location_icon_view.h"
+#include "chrome/browser/ui/views/toolbar/toolbar_button.h"
+#include "ui/views/controls/button/button.h"
+
+class ToolbarView;
+
+namespace content {
+class WebContents;
+}
+
+namespace gfx {
+class Canvas;
+}
+
+namespace views {
+class Button;
+class Label;
+}
+
+class SiteChipView : public ToolbarButton,
+ public views::ButtonListener {
+ public:
+ explicit SiteChipView(ToolbarView* toolbar_view);
+ virtual ~SiteChipView();
+
+ void Init();
+
+ // Returns whether the Site Chip should be shown in the toolbar. Returns
+ // true if the experiment is enabled and the site chip should be shown.
Peter Kasting 2013/12/04 00:57:31 Nit: Hmm, this added second sentence just sounds r
Greg Billock 2013/12/04 04:05:56 Yeah, the choice to just always show it simplifies
Greg Billock 2013/12/04 17:01:18 Done.
+ bool ShouldShow();
+
+ // Recalculates the contents of the Site Chip based on the displayed tab.
+ void Update(content::WebContents* tab);
+
+ views::ImageView* location_icon_view() { return location_icon_view_; }
+
+ // ToolbarButton:
+ virtual gfx::Size GetPreferredSize() OVERRIDE;
+ virtual void Layout() OVERRIDE;
+ virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE;
+
+ // views::ButtonListener:
+ virtual void ButtonPressed(views::Button* sender,
+ const ui::Event& event) OVERRIDE;
+
+ private:
+ ToolbarView* toolbar_view_;
+ views::Label* host_label_;
+ LocationIconView* location_icon_view_;
+ scoped_ptr<views::Painter> background_painter_;
+
+ DISALLOW_COPY_AND_ASSIGN(SiteChipView);
+};
+
+#endif // CHROME_BROWSER_UI_VIEWS_TOOLBAR_SITE_CHIP_VIEW_H_
« no previous file with comments | « no previous file | chrome/browser/ui/views/toolbar/site_chip_view.cc » ('j') | chrome/browser/ui/views/toolbar/site_chip_view.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698