Show Goodies page to new Chromebook users

chrome/browser/chromeos/first_run/goodies_displayer.h

+// Copyright 2015 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_CHROMEOS_FIRST_RUN_GOODIES_DISPLAYER_H_
+#define CHROME_BROWSER_CHROMEOS_FIRST_RUN_GOODIES_DISPLAYER_H_
+
+#include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/browser_list_observer.h"
+
+namespace chromeos {
+namespace first_run {
+
+#if defined(OS_CHROMEOS)

[achuithb, 2015/09/15 18:20:54]

Pretty sure you don't need this - gyp file should be taking care of this?

[Greg Levin, 2015/09/15 21:44:48]

Yep, good call!

+
+// Handles display of OOBE Goodies page on first display of browser window on
+// new Chromebooks.
+class GoodiesDisplayer : public chrome::BrowserListObserver {
+ public:
+  GoodiesDisplayer();
+
+  ~GoodiesDisplayer() override {}
+
+  void OnBrowserSetLastActive(Browser* browser) override;

[achuithb, 2015/09/15 18:20:54]

Add comment like:
// BrowserListObserver overrides.

Also, make this private.

[Greg Levin, 2015/09/15 21:44:48]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(GoodiesDisplayer);
+};
+
+#endif  // defined(OS_CHROMEOS)
+
+void InitializeGoodiesDisplayer();

[achuithb, 2015/09/15 18:20:54]

Make this a static member of GoodiesDisplayer, and call it Init

[Greg Levin, 2015/09/15 21:44:48]

Done.

+
+}  // namespace first_run
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_FIRST_RUN_GOODIES_DISPLAYER_H_