Add sequential dispatching for InitializationRunner

ios/chrome/app/deferred_initialization_runner.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef IOS_CHROME_APP_DEFERRED_INITIALIZATION_RUNNER_H_
 #define IOS_CHROME_APP_DEFERRED_INITIALIZATION_RUNNER_H_
 
 #import <Foundation/Foundation.h>
 
 #include "base/ios/block_types.h"
 
 // A singleton object to run initialization code asynchronously. Blocks are
 // scheduled to be run after a delay. The block is named when added to the
 // singleton so that other code can force a deferred block to be run
 // synchronously if necessary.
 @interface DeferredInitializationRunner : NSObject
 
 // 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
 // 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)runBlockNamed:(NSString*)name
                 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;
 
 // Cancels a previously scheduled block of |name|. This is a no-op if the
 // block has already been executed.
 - (void)cancelBlockNamed:(NSString*)name;
 
 // Number of blocks that have been registered but not executed yet.
 // 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 just for testing? If it is
just for facilitating testing, I would rather make this private and unit tests
can access it via categories.

[gambard, 2016/08/09 08:17:05]

For now it is used for testing but I think it is relevant as part of the public
API.

+@property(nonatomic) NSTimeInterval delayBetweenBlocks;
+
 @end
 
 #endif  // IOS_CHROME_APP_DEFERRED_INITIALIZATION_RUNNER_H_