Wire up the IP address change notification to the PPAPI network state

content/browser/ppapi_plugin_process_host.cc

 // Copyright (c) 2011 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 "content/browser/ppapi_plugin_process_host.h"
 
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/process_util.h"
 #include "base/utf_string_conversions.h"
 #include "content/browser/plugin_service.h"
 #include "content/browser/renderer_host/render_message_filter.h"
 #include "content/common/content_switches.h"
 #include "content/common/pepper_plugin_registry.h"
 #include "ipc/ipc_switches.h"
 #include "ppapi/proxy/ppapi_messages.h"
 
 PpapiPluginProcessHost::PpapiPluginProcessHost(net::HostResolver* host_resolver)
     : BrowserChildProcessHost(ChildProcessInfo::PPAPI_PLUGIN_PROCESS),
       filter_(new PepperMessageFilter(host_resolver)) {
   AddFilter(filter_.get());
+  net::NetworkChangeNotifier::AddIPAddressObserver(this);
   net::NetworkChangeNotifier::AddOnlineStateObserver(this);
 }
 
 PpapiPluginProcessHost::~PpapiPluginProcessHost() {
   net::NetworkChangeNotifier::RemoveOnlineStateObserver(this);
+  net::NetworkChangeNotifier::RemoveIPAddressObserver(this);
   CancelRequests();
 }
 
 bool PpapiPluginProcessHost::Init(const PepperPluginInfo& info) {
   plugin_path_ = info.path;
   set_name(UTF8ToWide(info.name));
   set_version(UTF8ToWide(info.version));
 
   if (!CreateChannel())
     return false;
@@ skipping 114 matching lines @@
   }
   pending_requests_.clear();
 
   while (!sent_requests_.empty()) {
     sent_requests_.front()->OnChannelOpened(base::kNullProcessHandle,
                                             IPC::ChannelHandle());
     sent_requests_.pop();
   }
 }
 
+void PpapiPluginProcessHost::OnIPAddressChanged() {
+  // TODO(brettw) bug 90246: This doesn't seem correct. The online/offline
+  // notification seems like it should be sufficient, but I don't see that when
+  // I unplug and replug my network cable. Sending this notification when
+  // "something" changes seems to make Flash reasonably happy, but seems wrong.
+  // We should really be able to provide the real online state in
+  // OnOnlineStateChanged().
+  Send(new PpapiMsg_SetNetworkState(true));
+}
+
 void PpapiPluginProcessHost::OnOnlineStateChanged(bool online) {
   Send(new PpapiMsg_SetNetworkState(online));
 }
 
 // Called when a new plugin <--> renderer channel has been created.
 void PpapiPluginProcessHost::OnRendererPluginChannelCreated(
     const IPC::ChannelHandle& channel_handle) {
   if (sent_requests_.empty())
     return;
 
@@ skipping 13 matching lines @@
   ::DuplicateHandle(::GetCurrentProcess(), plugin_process,
                     renderer_process, &renderers_plugin_handle,
                     0, FALSE, DUPLICATE_SAME_ACCESS);
 #elif defined(OS_POSIX)
   // Don't need to duplicate anything on POSIX since it's just a PID.
   base::ProcessHandle renderers_plugin_handle = plugin_process;
 #endif
 
   client->OnChannelOpened(renderers_plugin_handle, channel_handle);
 }