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

Unified Diff: chrome/browser/ui/gtk/panels/panel_gtk.cc

Issue 10987037: Add tests for panel app icon functionaility. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Patch to land Created 8 years, 2 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/cocoa/panels/panel_cocoa.mm ('k') | chrome/browser/ui/panels/native_panel.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/gtk/panels/panel_gtk.cc
diff --git a/chrome/browser/ui/gtk/panels/panel_gtk.cc b/chrome/browser/ui/gtk/panels/panel_gtk.cc
index 268b6fac2e766ae36866ab7b8fe0f91655769c53..44b579efc7bb091c9354549d1940e7e9f3316fe9 100644
--- a/chrome/browser/ui/gtk/panels/panel_gtk.cc
+++ b/chrome/browser/ui/gtk/panels/panel_gtk.cc
@@ -1028,6 +1028,7 @@ class GtkNativePanelTesting : public NativePanelTesting {
virtual void FinishDragTitlebar() OVERRIDE;
virtual bool VerifyDrawingAttention() const OVERRIDE;
virtual bool VerifyActiveState(bool is_active) OVERRIDE;
+ virtual bool VerifyAppIcon() const OVERRIDE;
virtual void WaitForWindowCreationToComplete() const OVERRIDE;
virtual bool IsWindowSizeKnown() const OVERRIDE;
virtual bool IsAnimatingBounds() const OVERRIDE;
@@ -1111,6 +1112,13 @@ bool GtkNativePanelTesting::VerifyActiveState(bool is_active) {
return false;
}
+bool GtkNativePanelTesting::VerifyAppIcon() const {
+ GdkPixbuf* icon = gtk_window_get_icon(panel_gtk_->GetNativePanelHandle());
+ return icon &&
+ gdk_pixbuf_get_width(icon) == panel::kPanelAppIconSize &&
+ gdk_pixbuf_get_height(icon) == panel::kPanelAppIconSize;
+}
+
void GtkNativePanelTesting::WaitForWindowCreationToComplete() const {
while (GetFrameSize().IsEmpty())
MessageLoopForUI::current()->RunAllPending();
« no previous file with comments | « chrome/browser/ui/cocoa/panels/panel_cocoa.mm ('k') | chrome/browser/ui/panels/native_panel.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698