Make sure we never pass a null Frame to Document::canNavigate()

Source/core/loader/FrameLoader.cpp

Index: Source/core/loader/FrameLoader.cpp
diff --git a/Source/core/loader/FrameLoader.cpp b/Source/core/loader/FrameLoader.cpp
index 944aa686aa7f72da7ec084eab004641cb5c217ba..f8db9cb767fe52ba152ca6fa987cf1a483974d03 100644
--- a/Source/core/loader/FrameLoader.cpp
+++ b/Source/core/loader/FrameLoader.cpp
@@ -1386,7 +1386,7 @@ LocalFrame* FrameLoader::findFrameForNavigation(const AtomicString& name, Docume
 {
     ASSERT(activeDocument);
     LocalFrame* frame = m_frame->tree().find(name);
-    if (!activeDocument->canNavigate(frame))
+    if (!frame || !activeDocument->canNavigate(*frame))

[eseidel, 2014/06/02 05:19:29]

So this is the magic to fix the layout test?

[Inactive, 2014/06/02 12:16:37]

No, I did not do anything to fix the layout test. I think the comment was
outdated. In this part of the code, the function was returning 0 already if
frame was null:
- The if check would frame so it would do "return frame", frame being null.

Now, the if check succeeds, so it explicitly returns 0.

         return 0;
     return frame;
 }