[DevTools] Console Message Storage moved from top local frame to frame host

Source/core/frame/FrameConsole.cpp

 /*
  * Copyright (C) 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 147 matching lines @@
 }
 
 void FrameConsole::unmute()
 {
     ASSERT(muteCount > 0);
     muteCount--;
 }
 
 ConsoleMessageStorage* FrameConsole::messageStorage()
 {
-    LocalFrame* curFrame = m_frame;
-    // FIXME: Move the console's messageStorage off the main frame.
-    Frame* topFrame = curFrame->localFrameRoot();
-    ASSERT(topFrame->isLocalFrame());
-    LocalFrame* localTopFrame = toLocalFrame(topFrame);
-    if (localTopFrame != curFrame)
-        return localTopFrame->console().messageStorage();
-    if (!m_consoleMessageStorage)
-        m_consoleMessageStorage = ConsoleMessageStorage::createForFrame(m_frame);
-    return m_consoleMessageStorage.get();
+    ASSERT(m_frame->page());
+    return &m_frame->page()->frameHost().consoleMessageStorage();

[Charlie Reis, 2014/10/15 17:09:17]

nit: Can we use m_frame->host() instead of m_frame->page()->frameHost()?  Page
is eventually going to go away.

 }
 
 void FrameConsole::clearMessages()
 {
     messageStorage()->clear();
 }
 
 void FrameConsole::adoptWorkerMessagesAfterTermination(WorkerGlobalScopeProxy* proxy)
 {
     messageStorage()->adoptWorkerMessagesAfterTermination(proxy);
@@ skipping 14 matching lines @@
     messageStorage()->reportMessage(consoleMessage.release());
 }
 
 void FrameConsole::trace(Visitor* visitor)
 {
     visitor->trace(m_frame);
     visitor->trace(m_consoleMessageStorage);
 }
 
 } // namespace blink