Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2796)

Unified Diff: content/renderer/render_thread_impl.cc

Issue 2416153005: Remove usage of FOR_EACH_OBSERVER macro in content/renderer (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/renderer/render_frame_impl.cc ('k') | content/renderer/render_view_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 9e7bf6551394a6ba1232031acaa508f33036cea7..7a4a7d1df67dbeaa316c148b2096e40156e563ca 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -904,8 +904,8 @@ RenderThreadImpl::~RenderThreadImpl() {
}
void RenderThreadImpl::Shutdown() {
- FOR_EACH_OBSERVER(
- RenderThreadObserver, observers_, OnRenderProcessShutdown());
+ for (auto& observer : observers_)
+ observer.OnRenderProcessShutdown();
if (memory_observer_) {
message_loop()->RemoveTaskObserver(memory_observer_.get());
@@ -1417,7 +1417,8 @@ void RenderThreadImpl::IdleHandler() {
idle_timer_.Stop();
}
- FOR_EACH_OBSERVER(RenderThreadObserver, observers_, IdleNotification());
+ for (auto& observer : observers_)
+ observer.IdleNotification();
}
int64_t RenderThreadImpl::GetIdleNotificationDelayInMs() const {
@@ -2007,7 +2008,8 @@ void RenderThreadImpl::OnPurgePluginListCache(bool reload_pages) {
blink::resetPluginCache(reload_pages);
blink_platform_impl_->set_plugin_refresh_allowed(true);
- FOR_EACH_OBSERVER(RenderThreadObserver, observers_, PluginListChanged());
+ for (auto& observer : observers_)
+ observer.PluginListChanged();
}
#endif
@@ -2016,8 +2018,8 @@ void RenderThreadImpl::OnNetworkConnectionChanged(
double max_bandwidth_mbps) {
bool online = type != net::NetworkChangeNotifier::CONNECTION_NONE;
WebNetworkStateNotifier::setOnLine(online);
- FOR_EACH_OBSERVER(
- RenderThreadObserver, observers_, NetworkStateChanged(online));
+ for (auto& observer : observers_)
+ observer.NetworkStateChanged(online);
WebNetworkStateNotifier::setWebConnection(
NetConnectionTypeToWebConnectionType(type), max_bandwidth_mbps);
}
« no previous file with comments | « content/renderer/render_frame_impl.cc ('k') | content/renderer/render_view_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698