Implement the status tray/icon API for ChromeOS.

chrome/browser/ui/views/status_icons/status_tray_chromeos_browsertest.cc

Index: chrome/browser/ui/views/status_icons/status_tray_chromeos_browsertest.cc
diff --git a/chrome/browser/ui/views/status_icons/status_tray_chromeos_browsertest.cc b/chrome/browser/ui/views/status_icons/status_tray_chromeos_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..75bc79aa453d10784cdf4321870ac1d80b42f5d4
--- /dev/null
+++ b/chrome/browser/ui/views/status_icons/status_tray_chromeos_browsertest.cc
@@ -0,0 +1,59 @@
+// Copyright (c) 2011 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.
+
+#include "base/string_util.h"
+#include "base/utf_string_conversions.h"
+#include "chrome/browser/ui/views/status_icons/status_icon_chromeos.h"
+#include "chrome/browser/ui/views/status_icons/status_tray_chromeos.h"
+#include "chrome/test/base/in_process_browser_test.h"
+#include "grit/theme_resources.h"
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "ui/base/models/simple_menu_model.h"
+#include "ui/base/resource/resource_bundle.h"
+
+class SkBitmap;
+
+class MockStatusIconObserver : public StatusIcon::Observer {
+ public:
+  MOCK_METHOD0(OnClicked, void());
+};
+
+// Using a browser test since in ChromeOS the status icons are added
+// to the browser windows. If there is no browser nothing is tested.
+class StatusTrayChromeOSBrowserTest : public InProcessBrowserTest {};
+
+IN_PROC_BROWSER_TEST_F(StatusTrayChromeOSBrowserTest, CreateTray) {
+  // Just tests creation/destruction.
+  StatusTrayChromeOS tray;
+}
+
+IN_PROC_BROWSER_TEST_F(StatusTrayChromeOSBrowserTest, CreateIconAndMenu) {
+  // Create an icon, set the images, tooltip, and context menu, then shut it
+  // down.
+  StatusTrayChromeOS tray;
+  StatusIcon* icon = tray.CreateStatusIcon();
+  SkBitmap* bitmap = ResourceBundle::GetSharedInstance().GetBitmapNamed(
+      IDR_STATUS_TRAY_ICON);
+  icon->SetImage(*bitmap);
+  icon->SetPressedImage(*bitmap);
+  icon->SetToolTip(ASCIIToUTF16("tool tip"));
+  ui::SimpleMenuModel* menu = new ui::SimpleMenuModel(NULL);
+  menu->AddItem(0, ASCIIToUTF16("foo"));
+  icon->SetContextMenu(menu);

[Andrew T Wilson (Slow), 2011/11/16 00:28:54]

This is just checking to make sure we don't crash - is there any way to actually
make sure an icon is displayed in the BrowserView?

[Leandro Graciá Gil, 2011/11/17 16:18:48]

I'm not sure about displayed, but we should be able to check if the button is
contained as a child view of the status area. I'm implementing it.

+}
+
+IN_PROC_BROWSER_TEST_F(StatusTrayChromeOSBrowserTest, ClickOnIcon) {
+  // Create an icon, send a fake click event, make sure observer is called.
+  StatusTrayChromeOS tray;
+  StatusIconChromeOS* icon =
+      static_cast<StatusIconChromeOS*>(tray.CreateStatusIcon());
+  MockStatusIconObserver observer;
+  icon->AddObserver(&observer);
+  EXPECT_CALL(observer, OnClicked());
+  // Mimic a click.
+  icon->TestClicked();

[Andrew T Wilson (Slow), 2011/11/16 00:28:54]

Instead of exposing TestClicked(), is it possible to find the button that was
added to the BrowserView, and force a click event on it? I think that'd be a
better test.

[Leandro Graciá Gil, 2011/11/17 16:18:48]

I can reverse the way it's done. The button's Activate method is called when
clicked. Instead of calling TestClicked from it, I can add a testing method to
invoke Activate. It's still not a real click but at least it will follow the
same callback flow.

+  icon->RemoveObserver(&observer);
+}
+