Fix crash on clicking the web notification tray

ui/message_center/views/message_center_bubble.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 "ui/message_center/views/message_center_bubble.h"
 
 #include "base/macros.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/message_center/message_center_style.h"
 #include "ui/message_center/views/message_center_view.h"
@@ skipping 89 matching lines @@
   message_center_view_ = new MessageCenterView(
       message_center(), tray(), max_height(), initially_settings_visible_,
       false); /* MessageCenterBubble should be used only on ChromeOS.
                  Message center is never shown top down in ChromeOS. */
   bubble_view()->AddChildView(new ContentsView(this, message_center_view_));
   // Resize the content of the bubble view to the given bubble size. This is
   // necessary in case of the bubble border forcing a bigger size then the
   // |new_bubble_view| actually wants. See crbug.com/169390.
   bubble_view()->Layout();
   UpdateBubbleView();
-  bubble_view()
+  DCHECK(bubble_view()->GetFocusManager());
+  views::View* next_focusable_view = bubble_view()
       ->GetFocusManager()
-      ->GetNextFocusableView(nullptr, nullptr, false, false)
-      ->RequestFocus();
+      ->GetNextFocusableView(nullptr, nullptr, false, false);

[stevenjb, 2016/07/07 16:57:12]

If we are going to DCHECK on bubble_view()->GetFocusManager() we should go ahead
and assign it to a local variable, DCHECK that, then use it here.

[yoshiki, 2016/07/11 18:19:48]

Done.

+  if (next_focusable_view)

[stevenjb, 2016/07/07 16:57:12]

Add a comment explaining why next_focusable_view might be null.

[yoshiki, 2016/07/11 18:19:48]

Done.

+    next_focusable_view->RequestFocus();
 }
 
 void MessageCenterBubble::OnBubbleViewDestroyed() {
   message_center_view_ = NULL;
 }
 
 void MessageCenterBubble::UpdateBubbleView() {
   if (!bubble_view())
     return;  // Could get called after view is closed
   const NotificationList::Notifications& notifications =
@@ skipping 14 matching lines @@
 }
 
 void MessageCenterBubble::OnMouseExitedView() {
 }
 
 size_t MessageCenterBubble::NumMessageViewsForTest() const {
   return message_center_view_->NumMessageViewsForTest();
 }
 
 }  // namespace message_center