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

Unified Diff: chrome/test/sync/engine/test_directory_setter_upper.h

Issue 246098: Sync: Remove pthreads from auth_watcher. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 2 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/test/sync/engine/test_directory_setter_upper.h
===================================================================
--- chrome/test/sync/engine/test_directory_setter_upper.h (revision 28272)
+++ chrome/test/sync/engine/test_directory_setter_upper.h (working copy)
@@ -59,6 +59,8 @@
const PathString& name() const { return name_; }
protected:
+ // Subclasses may want to use a different directory name.
+ explicit TestDirectorySetterUpper(const PathString& name);
virtual void Init();
private:
@@ -82,6 +84,18 @@
bool was_opened_;
};
+// Use this flavor if you have a test that will trigger the opening event to
+// happen automagically. It is careful on teardown to close only if needed.
+class TriggeredOpenTestDirectorySetterUpper : public TestDirectorySetterUpper {
+ public:
+ // A triggered open is typically in response to a successful auth event just
+ // as in "real life". In this case, the name that will be used should be
+ // deterministically known at construction, and is passed in |name|.
+ TriggeredOpenTestDirectorySetterUpper(const std::string& name);
+ virtual void SetUp();
+ virtual void TearDown();
+};
+
} // namespace browser_sync
#endif // CHROME_TEST_SYNC_ENGINE_TEST_DIRECTORY_SETTER_UPPER_H_
« no previous file with comments | « chrome/test/sync/engine/mock_server_connection.cc ('k') | chrome/test/sync/engine/test_directory_setter_upper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698