Refactor RenderWidgetHost::set_paint_observer()

chrome/browser/renderer_host/backing_store_manager.cc

 // Copyright (c) 2010 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/renderer_host/backing_store_manager.h"
 
 #include "base/sys_info.h"
 #include "base/command_line.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/browser/renderer_host/backing_store.h"
 #include "chrome/browser/renderer_host/render_widget_host.h"
-#include "chrome/browser/renderer_host/render_widget_host_painting_observer.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/mru_cache.h"
+#include "chrome/common/notification_service.h"
 
 namespace {
 
 // There are two separate caches, |large_cache| and |small_cache|.  large_cache
 // is meant for large items (tabs, popup windows), while small_cache is meant
 // for small items (extension toolstrips and buttons, etc.).  The idea is that
 // we'll almost always try to evict from large_cache first since small_cache
 // items will tend to be visible more of the time.
 typedef OwningMRUCache<RenderWidgetHost*, BackingStore*> BackingStoreCache;
 static BackingStoreCache* large_cache = NULL;
@@ skipping 37 matching lines @@
 
 // The maximum about of memory to use for all BackingStoreCache object combined.
 static size_t MaxBackingStoreMemory() {
   // Compute in terms of the number of large monitor's worth of backing-store.
   return MaxNumberOfBackingStores() * kMemoryMultiplier;
 }
 
 // Expires the given |backing_store| from |cache|.
 void ExpireBackingStoreAt(BackingStoreCache* cache,
                           BackingStoreCache::iterator backing_store) {
-  RenderWidgetHost* rwh = backing_store->second->render_widget_host();
-  if (rwh->painting_observer()) {
-    rwh->painting_observer()->WidgetWillDestroyBackingStore(
-        backing_store->first,
-        backing_store->second);
-  }
+  NotificationService::current()->Notify(
+      NotificationType::RENDER_WIDGET_HOST_WILL_DESTROY_BACKING_STORE,
+      Source<RenderWidgetHost>(backing_store->first),
+      Details<BackingStore>(backing_store->second));
   cache->Erase(backing_store);
 }
 
 size_t ExpireLastBackingStore(BackingStoreCache* cache) {
   if (cache->size() < 1)
     return 0;
 
   // Crazy C++ alert: rbegin.base() is a forward iterator pointing to end(),
   // so we need to do -- to move one back to the actual last item.
   BackingStoreCache::iterator entry = --cache->rbegin().base();
@@ skipping 191 matching lines @@
   size_t mem = 0;
   BackingStoreCache::iterator it;
   for (it = large_cache->begin(); it != large_cache->end(); ++it)
     mem += it->second->MemorySize();
 
   for (it = small_cache->begin(); it != small_cache->end(); ++it)
     mem += it->second->MemorySize();
 
   return mem;
 }