Fixing crash Report - Magic Signature: views::View::ConvertPointToScreen

chrome/browser/ui/views/frame/immersive_mode_controller_ash.cc

Index: chrome/browser/ui/views/frame/immersive_mode_controller_ash.cc
diff --git a/chrome/browser/ui/views/frame/immersive_mode_controller_ash.cc b/chrome/browser/ui/views/frame/immersive_mode_controller_ash.cc
index 5aa2b0f83de7170ac837d2ff9ec5701d60e31c06..bd1c9ebd68d279c082e90c205c38457cff9e633d 100644
--- a/chrome/browser/ui/views/frame/immersive_mode_controller_ash.cc
+++ b/chrome/browser/ui/views/frame/immersive_mode_controller_ash.cc
@@ -586,8 +586,8 @@ void ImmersiveModeControllerAsh::OnAddTransientChild(aura::Window* window,
                                                      aura::Window* transient) {
   views::BubbleDelegateView* bubble_delegate = AsBubbleDelegate(transient);
   if (bubble_delegate &&
-      bubble_delegate->anchor_view() &&
-      top_container_->Contains(bubble_delegate->anchor_view())) {
+      bubble_delegate->AnchorView() &&
+      top_container_->Contains(bubble_delegate->AnchorView())) {
     // Observe the aura::Window because the BubbleDelegateView may not be
     // parented to the widget's root view yet so |bubble_delegate->GetWidget()|
     // may still return NULL.
@@ -837,10 +837,12 @@ void ImmersiveModeControllerAsh::UpdateFocusRevealedLock() {
         native_window_->GetRootWindow())->GetActiveWindow();
     views::BubbleDelegateView* bubble_delegate =
         AsBubbleDelegate(active_window);
-    if (bubble_delegate && bubble_delegate->anchor_view()) {
+    if (bubble_delegate && bubble_delegate->has_anchor_view()) {
       // BubbleManager will already have locked the top-of-window views if the
       // bubble is anchored to a child of |top_container_|. Don't acquire
       // |focus_revealed_lock_| here for the sake of simplicity.
+      // Note that even thogh the view might already be gone, the owning

[msw, 2013/09/27 18:20:31]

nit: "though". I don't quite understand the point of this comment.

[Mr4D (OOO till 08-26), 2013/09/27 20:29:35]

Done.

+      // anchor widget does still exist.
     } else {
       // The currently active window is not |native_window_| and it is not a
       // bubble with an anchor view. The top-of-window views should be revealed
@@ -1170,8 +1172,8 @@ void ImmersiveModeControllerAsh::RecreateBubbleManager() {
     views::BubbleDelegateView* bubble_delegate =
         AsBubbleDelegate(transient_child);
     if (bubble_delegate &&
-        bubble_delegate->anchor_view() &&
-        top_container_->Contains(bubble_delegate->anchor_view())) {
+        bubble_delegate->AnchorView() &&
+        top_container_->Contains(bubble_delegate->AnchorView())) {
       bubble_manager_->StartObserving(transient_child);
     }
   }