Created ash::FirstRunHelper.

ash/first_run/first_run_delegate.h

+// Copyright 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 ASH_FIRST_RUN_FIRST_RUN_DELEGATE_H_
+#define ASH_FIRST_RUN_FIRST_RUN_DELEGATE_H_
+
+#include "ash/ash_export.h"
+#include "base/basictypes.h"
+
+namespace gfx {
+class Rect;
+}
+
+namespace ash {
+
+// Interface used by first-run tutorial to manipulate and retreive information
+// about shell elements.
+// All returned coordinates are in screen coordinate system.
+class ASH_EXPORT FirstRunDelegate {

[oshima, 2013/10/11 20:58:44]

XxxDelegate is for interface to delegate the part of implementation.
This is more like information retrieval, so it shouldn't be a delegate.

Also what is the benefit of defining a interface for this?
Will there be test impl?

[dzhioev (left Google), 2013/10/12 00:02:50]

It's not only for information retrieval, but also for interacting with shell.
Have you some proper name in mind? 

I'm going to implement stub for testing UI independently of Ash.

[oshima, 2013/10/16 16:32:05]

It's still not delegate. Controller/Manager would be more appropriate.

By the way, have you considered using a snapshot of the screen to
implement this feature, instead of using real, active app_list, launcher
widgets?
That way, you can avoid dealing with dynamic aspect of ash
(not only unhiding, or opening app_list, but launcher bounds may change while
syncing, a notification may popup during tutorial etc)
What kind of scenario you can't test by just using ash impl?

+ public:
+  FirstRunDelegate();
+  virtual ~FirstRunDelegate();
+
+  // Opens and closes app list.
+  virtual void OpenAppList() = 0;
+  virtual void CloseAppList() = 0;
+
+  // Returns bounding rectangle of launcher elements.
+  virtual gfx::Rect GetLauncherBounds() = 0;

[oshima, 2013/10/11 20:58:44]

what if the launcher is hidden?

[dzhioev (left Google), 2013/10/12 00:02:50]

It's not decided yet how to handle this case. Maybe we will unhide shelf
temporarily during tutorial.

[oshima, 2013/10/16 16:32:05]

This issue sounds same as app_list. Shouldn't this use the
same patter as app_list? If we can automatically unhide,
then can't we do the same for app list?

+
+  // Returns bounds of application list button.
+  virtual gfx::Rect GetAppListButtonBounds() = 0;
+
+  // If application list is opened saves its bounds to |result| and returns
+  // |true|. Returns |false| otherwise.
+  virtual bool GetAppListBounds(gfx::Rect* result) = 0;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(FirstRunDelegate);
+};
+
+}  // namespace ash
+
+#endif  // ASH_FIRST_RUN_FIRST_RUN_DELEGATE_H_
+