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

Unified Diff: ios/chrome/app/deferred_initialization_runner.h

Issue 2217083002: Add sequential dispatching for InitializationRunner (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: gyp Created 4 years, 4 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: ios/chrome/app/deferred_initialization_runner.h
diff --git a/ios/chrome/app/deferred_initialization_runner.h b/ios/chrome/app/deferred_initialization_runner.h
index e1bd59ce5e9006bb2fed80ffb1fd0202fc852000..dcab074c072869300fda0eb74703d23c14f3db1e 100644
--- a/ios/chrome/app/deferred_initialization_runner.h
+++ b/ios/chrome/app/deferred_initialization_runner.h
@@ -18,6 +18,7 @@
// Returns singleton instance.
+ (DeferredInitializationRunner*)sharedInstance;
+// Deprecated.
// Schedules |block| to be run after |delaySeconds| on the current queue.
// This |block| is stored as |name| so code can force this initialization to
// be run synchronously if necessary. This method may be called more than
@@ -28,6 +29,16 @@
after:(NSTimeInterval)delaySeconds
block:(ProceduralBlock)block;
+// Adds |block| to a block queue containing all block not run. The blocks in the
pkl (ping after 24h if needed) 2016/08/08 23:41:53 s/all block/all blocks/
gambard 2016/08/09 08:17:05 Done.
+// queue are runned sequentially with a small delay between each block.
pkl (ping after 24h if needed) 2016/08/08 23:41:53 s/runned/ran/
gambard 2016/08/09 08:17:05 Done.
+// If the queue is empty |block| is run after a small delay.
+// This |block| is stored as |name| so code can force this initialization to
+// be run synchronously if necessary. This method may be called more than
+// once with the same |name| parameter. Any block with the same |name|
+// cancels a previously scheduled block of the same |name| if the block has
+// not been run yet.
+- (void)enqueueBlockNamed:(NSString*)name block:(ProceduralBlock)block;
+
// Looks up a previously scheduled block of |name|. If block has not been
// run yet, run it synchronously now.
- (void)runBlockIfNecessary:(NSString*)name;
@@ -40,6 +51,9 @@
// Exposed for testing.
@property(nonatomic, readonly) NSUInteger numberOfBlocksRemaining;
+// Time interval between two blocks. Default value is 200ms.
pkl (ping after 24h if needed) 2016/08/08 23:41:53 Is this @property really part of the API or exists
gambard 2016/08/09 08:17:05 For now it is used for testing but I think it is r
+@property(nonatomic) NSTimeInterval delayBetweenBlocks;
+
@end
#endif // IOS_CHROME_APP_DEFERRED_INITIALIZATION_RUNNER_H_

Powered by Google App Engine
This is Rietveld 408576698