[BackgroundSync] Handle storage failure

content/browser/cache_storage/cache_storage_scheduler.cc

Index: content/browser/cache_storage/cache_storage_scheduler.cc
diff --git a/content/browser/cache_storage/cache_storage_scheduler.cc b/content/browser/cache_storage/cache_storage_scheduler.cc
index 2e1ac15e5f650ddbf13d1c87e2b310b3d9ef30ad..33447b93787db708d73e5a79cb7dd1fa1c47f61a 100644
--- a/content/browser/cache_storage/cache_storage_scheduler.cc
+++ b/content/browser/cache_storage/cache_storage_scheduler.cc
@@ -6,7 +6,10 @@
 
 #include <string>
 
+#include "base/bind.h"
+#include "base/location.h"
 #include "base/logging.h"
+#include "base/message_loop/message_loop_proxy.h"
 
 namespace content {
 
@@ -37,7 +40,7 @@ void CacheStorageScheduler::RunOperationIfIdle() {
     // TODO(jkarlin): Run multiple operations in parallel where allowed.
     base::Closure closure = pending_operations_.front();
     pending_operations_.pop_front();
-    closure.Run();
+    base::MessageLoopProxy::current()->PostTask(FROM_HERE, base::Bind(closure));

[jsbell, 2015/04/02 20:25:13]

Thanks for this change - I was slightly worried that if any synchronous
operations were introduced it could hog the thread and/or blow out the stack.

Looks like the CacheStorageSchedulerTest cases need RunUntilIdle() sprinkles
now, though.

[jkarlin, 2015/04/03 15:15:04]

Done.

   }
 }