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

Unified Diff: blimp/client/public/blimp_client_context.h

Issue 2320923002: Add a full Blimp integration test. (Closed)
Patch Set: Fixed this a bit more. Still some thread violations :(. Created 4 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: blimp/client/public/blimp_client_context.h
diff --git a/blimp/client/public/blimp_client_context.h b/blimp/client/public/blimp_client_context.h
index 786330ceff5d8d24ad30ec6df07ae3b973e10eef..bfcc3f96824700babdabbba3d359a39c20751444 100644
--- a/blimp/client/public/blimp_client_context.h
+++ b/blimp/client/public/blimp_client_context.h
@@ -64,6 +64,7 @@ class BlimpClientContext : public KeyedService {
// Start authentication flow and connection to engine.
virtual void Connect() = 0;
+ virtual void ConnectWithAssignment(const Assignment& assignment) = 0;
Khushal 2016/09/13 04:47:24 If its never meant to be used in production code,
David Trainor- moved to gerrit 2016/09/13 06:18:04 It's actually used by other methods internally whe
Khushal 2016/09/13 23:47:10 Its fine, doesn't look like something too bad to b
protected:
BlimpClientContext() = default;

Powered by Google App Engine
This is Rietveld 408576698