<webview>: Implement WebRequest API

chrome/browser/extensions/extension_process_manager.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 "base/bind.h"
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/metrics/histogram.h"
@@ skipping 433 matching lines @@
 
   return count;
 }
 
 int ExtensionProcessManager::DecrementLazyKeepaliveCount(
      const Extension* extension) {
   if (!extension->has_lazy_background_page())
     return 0;
 
   int& count = background_page_data_[extension->id()].lazy_keepalive_count;
-  DCHECK_GT(count, 0);
+  // TODO(fsamuel): What to do here? This fires when using WebRequests in
+  // <webview>.

[Matt Perry, 2013/02/06 22:40:24]

That's not good. These refcount errors are not fun to track down. When do you
hit the DCHECK? Is it on navigation/destruction of a webview?

[Fady Samuel, 2013/05/15 21:58:53]

Fixed. Calling IncrementInFlightEventsOnUI in EventRouter::DispatchEvent

+  //DCHECK_GT(count, 0);
   if (--count == 0) {
     MessageLoop::current()->PostDelayedTask(
         FROM_HERE,
         base::Bind(&ExtensionProcessManager::OnLazyBackgroundPageIdle,
                    weak_ptr_factory_.GetWeakPtr(), extension->id(),
                    ++background_page_data_[extension->id()].close_sequence_id),
         event_page_idle_time_);
   }
 
   return count;
@@ skipping 391 matching lines @@
           CreateBackgroundHostsForProfileStartup(GetProfile(),
                                                  this, service->extensions());
       }
       break;
     }
     default:
       ExtensionProcessManager::Observe(type, source, details);
       break;
   }
 }