Return early from TextAutosizer::updatePageInfo for remote main frame.

Source/core/rendering/TextAutosizer.cpp

 /*
  * Copyright (C) 2013 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 491 matching lines @@
 
     PageInfo previousPageInfo(m_pageInfo);
     m_pageInfo.m_settingEnabled = m_document->settings()->textAutosizingEnabled();
 
     if (!m_pageInfo.m_settingEnabled || m_document->printing()) {
         m_pageInfo.m_pageNeedsAutosizing = false;
     } else {
         RenderView* renderView = m_document->renderView();
         bool horizontalWritingMode = isHorizontalWritingMode(renderView->style()->writingMode());
 
-        LocalFrame* mainFrame = m_document->page()->deprecatedLocalMainFrame();
+        // FIXME: With out-of-process iframes, the top frame can be remote and
+        // doesn't have sizing information. Just return if this is the case.
+        Frame* frame = m_document->frame()->tree().top();
+        if (frame->isRemoteFrame())
+            return;
+
+        LocalFrame* mainFrame = toLocalFrame(frame);

[dcheng, 2014/10/23 21:48:17]

I would probably still use dprecatedLocalMainFrame() here to help us flag it for
later cleanup.

[nasko, 2014/10/23 21:50:44]

Done.

         IntSize frameSize = m_document->settings()->textAutosizingWindowSizeOverride();
         if (frameSize.isEmpty())
             frameSize = windowSize();
 
         m_pageInfo.m_frameWidth = horizontalWritingMode ? frameSize.width() : frameSize.height();
 
         IntSize layoutSize = mainFrame->view()->layoutSize();
         m_pageInfo.m_layoutWidth = horizontalWritingMode ? layoutSize.width() : layoutSize.height();
 
         // Compute the base font scale multiplier based on device and accessibility settings.
@@ skipping 640 matching lines @@
     }
     return computedSize;
 }
 
 void TextAutosizer::trace(Visitor* visitor)
 {
     visitor->trace(m_document);
 }
 
 } // namespace blink