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

Unified Diff: chrome/browser/sync/glue/session_model_associator.h

Issue 11298004: alternate ntp: add "Recent Tabs" submenu to wrench menu (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixed disposition, minor style cleanup Created 8 years, 1 month 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/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..1dcaee1c9f6f980d0cb57770bcd2315e067f0044 100644
--- a/chrome/browser/sync/glue/session_model_associator.h
+++ b/chrome/browser/sync/glue/session_model_associator.h
@@ -235,6 +235,7 @@ class SessionModelAssociator
private:
friend class SyncSessionModelAssociatorTest;
+ friend class SessionModelAssociatorForRecentTabsSubMenuModelTest;
akalin 2012/11/09 18:11:58 I'd prefer to instead expose a method like "Tracke
kuan 2012/11/09 18:17:17 i also need PopulateSessionWindowFromSpecifics. s
kuan 2012/11/09 20:27:52 Done. i've enclosed the functions within #ifdef U
FRIEND_TEST_ALL_PREFIXES(ProfileSyncServiceSessionTest, WriteSessionToNode);
FRIEND_TEST_ALL_PREFIXES(ProfileSyncServiceSessionTest,
WriteFilledSessionToNode);

Powered by Google App Engine
This is Rietveld 408576698