Added pref for snapshot document collapsed state

chrome/browser/android/new_tab_page_prefs.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 CHROME_BROWSER_ANDROID_NEW_TAB_PAGE_PREFS_H_
+#define CHROME_BROWSER_ANDROID_NEW_TAB_PAGE_PREFS_H_
+
+#include <jni.h>
+
+#include "chrome/browser/profiles/profile.h"
+
+class NewTabPagePrefs {
+ public:
+  explicit NewTabPagePrefs(Profile* profile);
+  void Destroy(JNIEnv* env, jobject obj);
+
+  jboolean GetSnapshotDocumentCollapsed(JNIEnv* env, jobject obj);
+  void SetSnapshotDocumentCollapsed(JNIEnv* env,
+                                    jobject obj,
+                                    jboolean is_collapsed);
+
+  jboolean GetRecentlyClosedTabsCollapsed(JNIEnv* env, jobject obj);
+  void SetRecentlyClosedTabsCollapsed(JNIEnv* env,
+                                      jobject obj,
+                                      jboolean is_collapsed);
+
+  jboolean GetSyncPromoCollapsed(JNIEnv* env, jobject obj);
+  void SetSyncPromoCollapsed(JNIEnv* env,
+                             jobject obj,
+                             jboolean is_collapsed);
+
+  jboolean GetForeignSessionCollapsed(JNIEnv* env,
+                                      jobject obj,
+                                      jstring session_tag);
+  void SetForeignSessionCollapsed(JNIEnv* env, jobject obj,
+                                  jstring session_tag,
+                                  jboolean is_collapsed);
+
+  static bool RegisterNewTabPagePrefs(JNIEnv* env);
+  static void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry);
+ private:
+  virtual ~NewTabPagePrefs();
+
+  Profile* profile_;  // weak

[newt (away), 2013/10/30 23:47:51]

could we store a pointer to PrefService instead of Profile? not sure which is
better.

[apiccion, 2013/11/01 23:30:32]

I'm apprehensive because the preferences pointer may change (e.g. the underlying
object may be reconstructed) and we could end up holding onto a dead pointer.
I'm not sure if this actually happens, but it's not clear from the profile.h
interface that GetPrefs() will always return the same object.

[newt (away), 2013/11/01 23:49:18]

sounds good to me.

+  DISALLOW_COPY_AND_ASSIGN(NewTabPagePrefs);
+};
+
+#endif  // CHROME_BROWSER_ANDROID_NEW_TAB_PAGE_PREFS_H_