Fix idle handlers not being called. The problem was that RenderThread::AddRoute was assuming that e…

content/renderer/render_widget.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 "content/renderer/render_widget.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/debug/trace_event_synthetic_delay.h"
@@ skipping 447 matching lines @@
     opener_id_ = opener_id;
 
   webwidget_ = web_widget;
 
   bool result = RenderThread::Get()->Send(create_widget_message);
   if (result) {
     RenderThread::Get()->AddRoute(routing_id_, this);
     // Take a reference on behalf of the RenderThread.  This will be balanced
     // when we receive ViewMsg_Close.
     AddRef();
-    if (is_hidden_)
-      RenderThread::Get()->WidgetHidden();
+    if (RenderThreadImpl::current()) {
+      RenderThreadImpl::current()->WidgetCreated();
+      if (is_hidden_)
+        RenderThreadImpl::current()->WidgetHidden();
+    }
     return true;
   } else {
     // The above Send can fail when the tab is closing.
     return false;
   }
 }
 
 // This is used to complete pending inits and non-pending inits.
 void RenderWidget::CompleteInit() {
   DCHECK(routing_id_ != MSG_ROUTING_NONE);
@@ skipping 235 matching lines @@
     initial_pos_ = new_position;
 }
 
 void RenderWidget::OnClose() {
   if (closing_)
     return;
   closing_ = true;
 
   // Browser correspondence is no longer needed at this point.
   if (routing_id_ != MSG_ROUTING_NONE) {
+    if (RenderThreadImpl::current())
+      RenderThreadImpl::current()->WidgetDestroyed();
     RenderThread::Get()->RemoveRoute(routing_id_);
     SetHidden(false);
   }
 
   // If there is a Send call on the stack, then it could be dangerous to close
   // now.  Post a task that only gets invoked when there are no nested message
   // loops.
   base::MessageLoop::current()->PostNonNestableTask(
       FROM_HERE, base::Bind(&RenderWidget::Close, this));
 
@@ skipping 1641 matching lines @@
   return gfx::Vector2d();
 }
 
 void RenderWidget::SetHidden(bool hidden) {
   if (is_hidden_ == hidden)
     return;
 
   // The status has changed.  Tell the RenderThread about it.
   is_hidden_ = hidden;
   if (is_hidden_)
-    RenderThread::Get()->WidgetHidden();
+    RenderThreadImpl::current()->WidgetHidden();
   else
-    RenderThread::Get()->WidgetRestored();
+    RenderThreadImpl::current()->WidgetRestored();
 }
 
 void RenderWidget::WillToggleFullscreen() {
   if (!webwidget_)
     return;
 
   if (is_fullscreen_) {
     webwidget_->willExitFullScreen();
   } else {
     webwidget_->willEnterFullScreen();
@@ skipping 471 matching lines @@
           surface_id(),
           GetURLForGraphicsContext3D(),
           gpu_channel_host.get(),
           attributes,
           false /* bind generates resources */,
           limits));
   return context.Pass();
 }
 
 }  // namespace content