Remove extension unload notification from LazyBackgroundLazyBackgroundTaskQueue

extensions/browser/lazy_background_task_queue.h

 // Copyright 2013 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 EXTENSIONS_BROWSER_LAZY_BACKGROUND_TASK_QUEUE_H_
 #define EXTENSIONS_BROWSER_LAZY_BACKGROUND_TASK_QUEUE_H_
 
 #include <map>
 #include <string>
 
 #include "base/callback_forward.h"
 #include "base/compiler_specific.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/linked_ptr.h"
+#include "base/scoped_observer.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
+#include "extensions/browser/extension_registry_observer.h"
 
 namespace content {
 class BrowserContext;
 }
 
 namespace extensions {
 class Extension;
+class ExtensionObserver;

[Devlin, 2014/07/15 17:27:34]

What is an ExtensionObserver? ;) (I think this is supposed to be
ExtensionRegistry)

[limasdf, 2014/07/16 05:44:58]

Done.
Damn! My bad T_T.(Thank you!)

 class ExtensionHost;
 
 // This class maintains a queue of tasks that should execute when an
 // extension's lazy background page is loaded. It is also in charge of loading
 // the page when the first task is queued.
 //
 // It is the consumer's responsibility to use this class when appropriate, i.e.
 // only with extensions that have not-yet-loaded lazy background pages.
-class LazyBackgroundTaskQueue : public content::NotificationObserver {
+class LazyBackgroundTaskQueue : public content::NotificationObserver,
+                                public ExtensionRegistryObserver {
  public:
   typedef base::Callback<void(ExtensionHost*)> PendingTask;
 
   explicit LazyBackgroundTaskQueue(content::BrowserContext* browser_context);
   virtual ~LazyBackgroundTaskQueue();
 
   // Returns the number of extensions having pending tasks.
   size_t extensions_with_pending_tasks() { return pending_tasks_.size(); }
 
   // Returns true if the task should be added to the queue (that is, if the
@@ skipping 22 matching lines @@
   typedef std::pair<content::BrowserContext*, ExtensionID> PendingTasksKey;
   typedef std::vector<PendingTask> PendingTasksList;
   typedef std::map<PendingTasksKey,
                    linked_ptr<PendingTasksList> > PendingTasksMap;
 
   // content::NotificationObserver interface.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
+  // ExtensionRegistryObserver interface.
+  virtual void OnExtensionUnloaded(
+      content::BrowserContext* browser_context,
+      const Extension* extension,
+      UnloadedExtensionInfo::Reason reason) OVERRIDE;
+
   // Called when a lazy background page has finished loading, or has failed to
   // load (host is NULL in that case). All enqueued tasks are run in order.
   void ProcessPendingTasks(
       ExtensionHost* host,
       content::BrowserContext* context,
       const Extension* extension);
 
   content::BrowserContext* browser_context_;
   content::NotificationRegistrar registrar_;
   PendingTasksMap pending_tasks_;
+
+  ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
+      extension_registry_observer_;
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_LAZY_BACKGROUND_TASK_QUEUE_H_