Use IsRenderViewLive() to test for SadTab in ZoomController.

chrome/browser/ui/zoom/zoom_controller.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 "chrome/browser/ui/zoom/zoom_controller.h"
 
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/zoom/zoom_event_manager.h"
@@ skipping 75 matching lines @@
   // An extension did not initiate this zoom change.
   return SetZoomLevelByExtension(zoom_level, NULL);
 }
 
 bool ZoomController::SetZoomLevelByExtension(
     double zoom_level,
     const scoped_refptr<const extensions::Extension>& extension) {
   // Cannot zoom in disabled mode. Also, don't allow changing zoom level on
   // a crashed tab.
   if (zoom_mode_ == ZOOM_MODE_DISABLED ||
-      !web_contents()->GetRenderProcessHost()->HasConnection())
+    !web_contents()->GetRenderViewHost()->IsRenderViewLive())

[Fady Samuel, 2014/09/02 14:27:24]

nit: indentation is off

[wjmaclean, 2014/09/02 14:42:54]

Done.

     return false;
 
   // Store extension data so that |extension| can be attributed when the zoom
   // change completes. We expect that by the time this function returns that
   // any observers that require this information will have requested it.
   last_extension_ = extension;
 
   // Do not actually rescale the page in manual mode.
   if (zoom_mode_ == ZOOM_MODE_MANUAL) {
     double old_zoom_level = zoom_level_;
@@ skipping 191 matching lines @@
   } else {
     // TODO(wjmaclean) Should we consider having HostZoomMap send both old and
     // new zoom levels here?
     double zoom_level = GetZoomLevel();
     ZoomChangedEventData zoom_change_data(
         web_contents(), zoom_level, zoom_level, zoom_mode_, can_show_bubble);
     FOR_EACH_OBSERVER(
         ZoomObserver, observers_, OnZoomChanged(zoom_change_data));
   }
 }