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

Unified Diff: chrome/browser/chromeos/first_run/goodies_displayer.h

Issue 1308833004: Show Goodies page to new Chromebook users (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use LazyInstance<GoodiesDisplayer> Created 5 years, 3 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
Index: chrome/browser/chromeos/first_run/goodies_displayer.h
diff --git a/chrome/browser/chromeos/first_run/goodies_displayer.h b/chrome/browser/chromeos/first_run/goodies_displayer.h
new file mode 100644
index 0000000000000000000000000000000000000000..41244d4a96de73f7576eebf24f9da65a308706c8
--- /dev/null
+++ b/chrome/browser/chromeos/first_run/goodies_displayer.h
@@ -0,0 +1,36 @@
+// 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
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
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
Greg Levin 2015/09/15 21:44:48 Done.
+
+} // namespace first_run
+} // namespace chromeos
+
+#endif // CHROME_BROWSER_CHROMEOS_FIRST_RUN_GOODIES_DISPLAYER_H_

Powered by Google App Engine
This is Rietveld 408576698