alternate ntp: add "Recent Tabs" submenu to wrench menu

chrome/browser/sync/glue/session_model_associator.h

Index: chrome/browser/sync/glue/session_model_associator.h
diff --git a/chrome/browser/sync/glue/session_model_associator.h b/chrome/browser/sync/glue/session_model_associator.h
index 0a61fa12d32162b983e5b5885f1fef6f0805b5ed..2b16534021c2ca616abcb70cbbd0b8347339563a 100644
--- a/chrome/browser/sync/glue/session_model_associator.h
+++ b/chrome/browser/sync/glue/session_model_associator.h
@@ -233,6 +233,24 @@ class SessionModelAssociator
   bool GetSyncedFaviconForPageURL(const std::string& url,
                                   std::string* png_favicon) const;
 
+#if defined(UNIT_TEST)

[akalin, 2012/11/10 00:08:40]

i don't think we should use this ifdef.  having different definitions per
compilation unit is technically undefined behavior, and the inline functions
should get optimized out anyway for non-unit-test executables.

[akalin, 2012/11/10 00:08:40]

But see comments in unit test file -- I think you can do what you want while
just working with the public interface.

[kuan, 2012/11/10 19:34:46]

i assume u mean i can keep it?

+  // Returns synced session track.
+  SyncedSessionTracker& GetSyncedSessionTrackerForTest() {

[akalin, 2012/11/10 00:08:40]

non-const refs are against style guide.  please use a pointer.

[kuan, 2012/11/10 19:34:46]

function is removed.

+    return synced_session_tracker_;
+  }
+
+  // Wrapper for private PopulateSessionWindowFromSpecifics.
+  static void PopulateSessionWindowFromSpecificsForTest(
+      const std::string& foreign_session_tag,
+      const sync_pb::SessionWindow& window,
+      base::Time mtime,
+      SessionWindow* session_window,
+      SyncedSessionTracker* tracker) {
+    PopulateSessionWindowFromSpecifics(foreign_session_tag, window, mtime,
+                                       session_window, tracker);
+  }
+#endif  // defined(UNIT_TEST)
+
  private:
   friend class SyncSessionModelAssociatorTest;
   FRIEND_TEST_ALL_PREFIXES(ProfileSyncServiceSessionTest, WriteSessionToNode);