Layout panels on top of their launcher icons

ash/test/test_launcher_delegate.h

+// Copyright (c) 2012 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 ASH_TEST_TEST_LAUNCHER_DELEGATE_H_
+#define ASH_TEST_TEST_LAUNCHER_DELEGATE_H_
+#pragma once
+
+#include <map>
+
+#include "ash/launcher/launcher_delegate.h"
+#include "base/compiler_specific.h"
+
+namespace ash {
+
+class LauncherModel;
+
+namespace test {
+
+class TestLauncherDelegate : public LauncherDelegate {

[sky, 2012/04/11 15:47:49]

Add a description.

+ public:
+  TestLauncherDelegate(LauncherModel* model);
+
+  virtual void CreateNewTab() OVERRIDE;

[sky, 2012/04/11 15:47:49]

Style for overriden methods is to prefix section with something like:
// LauncherDelegate:
then overriden methods with no newlines between each method.

Typically we put these at the end of the section. In other words AddLauncherItem
would be before these.

+  virtual void CreateNewWindow() OVERRIDE;
+  virtual void ItemClicked(const LauncherItem& item) OVERRIDE;
+
+  virtual int GetBrowserShortcutResourceId() OVERRIDE;
+
+  virtual string16 GetTitle(const LauncherItem& item) OVERRIDE;
+
+  virtual ui::MenuModel* CreateContextMenu(const LauncherItem& item) OVERRIDE;
+
+  virtual ui::MenuModel* CreateContextMenuForLauncher() OVERRIDE;
+
+  virtual ash::LauncherID GetIDByWindow(aura::Window* window) OVERRIDE;
+
+  void AddLauncherItem(aura::Window* window);
+ private:

[sky, 2012/04/11 15:47:49]

nit: newline between 38/39.

+  LauncherModel* model_;
+
+  typedef std::map<aura::Window*, ash::LauncherID> WindowToID;
+
+  // Maps from window to the id we gave it.
+  WindowToID window_to_id_;
+};

[sky, 2012/04/11 15:47:49]

DISALLOW_COPY_AND_ASSIGN

+
+} // namespace test
+} // namespace ash
+
+#endif // ASH_TEST_TEST_LAUNCHER_DELEGATE