Introduce runtime.onSuspendCanceled() event.

chrome/browser/chromeos/extensions/file_handler_util.cc

 // Copyright (c) 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.
 
 #include "chrome/browser/chromeos/extensions/file_handler_util.h"
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/i18n/case_conversion.h"
 #include "base/json/json_writer.h"
@@ skipping 752 matching lines @@
     const FileDefinitionList& file_list,
     int handler_pid) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   const Extension* extension = GetExtension();
   if (!extension) {
     ExecuteDoneOnUIThread(false);
     return;
   }
 
-  if (handler_pid > 0) {

[Matt Perry, 2012/07/24 19:25:15]

This potentially changes the behavior.. If handler_pid is > 0, we now always try
to load the bg page. That may not be what they want.

[koz (OOO until 15th September), 2012/07/25 07:48:31]

Ah, true. We probably only want to change the else clause here.

-    SetupPermissionsAndDispatchEvent(file_system_name, file_system_root,
-        file_list, handler_pid, NULL);
-  } else {
-    // We have to wake the handler background page before we proceed.
-    extensions::LazyBackgroundTaskQueue* queue =
-        extensions::ExtensionSystem::Get(profile())->
-        lazy_background_task_queue();
-    if (!queue->ShouldEnqueueTask(profile(), extension)) {
-      ExecuteDoneOnUIThread(false);
-      return;
-    }
-    queue->AddPendingTask(
-        profile(), extension_id_,
-        base::Bind(&ExtensionTaskExecutor::SetupPermissionsAndDispatchEvent,
-                   this, file_system_name, file_system_root, file_list,
-                   handler_pid));
+  queue->AddPendingTask(
+      profile(), extension_id_,
+      base::Bind(&ExtensionTaskExecutor::SetupPermissionsAndDispatchEvent,
+                this, file_system_name, file_system_root, file_list,
+                handler_pid));
   }
 }
 
 void ExtensionTaskExecutor::SetupPermissionsAndDispatchEvent(
     const std::string& file_system_name,
     const GURL& file_system_root,
     const FileDefinitionList& file_list,
     int handler_pid_in,
     ExtensionHost* host) {
   int handler_pid = host ? host->render_process_host()->GetID() :
@@ skipping 91 matching lines @@
         handler_host_permissions_[i].first,
         handler_host_permissions_[i].second);
   }
 
   // We don't need this anymore.
   handler_host_permissions_.clear();
 }
 
 } // namespace file_handler_util