DCHECK FrameView in LayoutSelection::SetSelection()

third_party/WebKit/Source/core/editing/LayoutSelection.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights
  * reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 130 matching lines @@
   // Check www.msnbc.com and try clicking around to find the case where this
   // happened.
   if ((start && !end) || (end && !start))
     return;
 
   // Just return if the selection hasn't changed.
   if (selection_start_ == start && selection_start_pos_ == start_pos &&
       selection_end_ == end && selection_end_pos_ == end_pos)
     return;
 
+  DCHECK(frame_selection_->GetDocument().GetLayoutView()->GetFrameView());
+
   // Record the old selected objects. These will be used later when we compare
   // against the new selected objects.
   int old_start_pos = selection_start_pos_;
   int old_end_pos = selection_end_pos_;
 
   // Objects each have a single selection rect to examine.
   typedef HashMap<LayoutObject*, SelectionState> SelectedObjectMap;
   SelectedObjectMap old_selected_objects;
   // FIXME: |newSelectedObjects| doesn't really need to store the
   // SelectionState, it's just more convenient to have it use the same data
@@ skipping 88 matching lines @@
         if (!result.is_new_entry)
           break;
         cb = cb->ContainingBlock();
       }
     }
 
     o = GetNextOrPrevLayoutObjectBasedOnDirection(o, stop, continue_exploring,
                                                   exploring_backwards);
   }
 
-  // TODO(yoichio): DCHECK(frame_selection_->,,,->GetFrameView());
-  if (!frame_selection_->GetDocument().GetLayoutView()->GetFrameView())
-    return;
-
   // Have any of the old selected objects changed compared to the new selection?
   for (SelectedObjectMap::iterator i = old_selected_objects.begin();
        i != old_objects_end; ++i) {
     LayoutObject* obj = i->key;
     SelectionState new_selection_state = obj->GetSelectionState();
     SelectionState old_selection_state = i->value;
     if (new_selection_state != old_selection_state ||
         (selection_start_ == obj && old_start_pos != selection_start_pos_) ||
         (selection_end_ == obj && old_end_pos != selection_end_pos_)) {
       obj->SetShouldInvalidateSelection();
@@ skipping 169 matching lines @@
 
     o->SetShouldInvalidateSelection();
   }
 }
 
 DEFINE_TRACE(LayoutSelection) {
   visitor->Trace(frame_selection_);
 }
 
 }  // namespace blink