Fix crash in KeyboardController when widget is being destroyed or crashed

widget->GetView() may return NULL so adding check for that.

BUG=381091

Review URL: https://codereview.chromium.org/314113002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275201 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 17 insertions, 12 deletions

diff --git a/ui/keyboard/keyboard_controller.cc b/ui/keyboard/keyboard_controller.cc
index 99bddd8..e43e036 100644
--- a/ui/keyboard/keyboard_controller.cc
+++ b/ui/keyboard/keyboard_controller.cc
@@ -251,18 +251,23 @@ void KeyboardController::NotifyKeyboardBoundsChanging(
       aura::Window *root_window = keyboard_window->GetRootWindow();
       while (content::RenderWidgetHost* widget = widgets->GetNextHost()) {
         content::RenderWidgetHostView* view = widget->GetView();
-        aura::Window *window = view->GetNativeView();
-        if (window != keyboard_window && window->GetRootWindow() == root_window)
-        {
-          gfx::Rect window_bounds = window->GetBoundsInScreen();
-          gfx::Rect intersect = gfx::IntersectRects(window_bounds, new_bounds);
-          int overlap = intersect.height();
-          if (overlap > 0 && overlap < window_bounds.height())
-            view->SetInsets(gfx::Insets(0, 0, overlap, 0));
-          else
-            view->SetInsets(gfx::Insets(0, 0, 0, 0));
-          // TODO(kevers): Add window observer to native window to update insets
-          // on a window move or resize.
+        // Can be NULL, e.g. if the RenderWidget is being destroyed or
+        // the render process crashed.
+        if (view) {
+          aura::Window *window = view->GetNativeView();
+          if (window != keyboard_window &&
+              window->GetRootWindow() == root_window) {
+            gfx::Rect window_bounds = window->GetBoundsInScreen();
+            gfx::Rect intersect = gfx::IntersectRects(window_bounds,
+                                                      new_bounds);
+            int overlap = intersect.height();
+            if (overlap > 0 && overlap < window_bounds.height())
+              view->SetInsets(gfx::Insets(0, 0, overlap, 0));
+            else
+              view->SetInsets(gfx::Insets(0, 0, 0, 0));
+            // TODO(kevers): Add window observer to native window to update
+            // insets on a window move or resize.
+          }
         }
       }
     }