Refactoring: Simplify DocumentWriter by reorganizing its lifetime.

Source/WebKit/chromium/src/WebPagePopupImpl.cpp

 /*
  * Copyright (C) 2012 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 198 matching lines @@
     RefPtr<Frame> frame = Frame::create(m_page.get(), 0, emptyFrameLoaderClient);
     frame->setView(FrameView::create(frame.get()));
     frame->init();
     frame->view()->resize(m_popupClient->contentSize());
     frame->view()->setTransparent(false);
 
     DOMWindowPagePopup::install(frame->document()->domWindow(), m_popupClient);
 
     DocumentWriter* writer = frame->loader()->activeDocumentLoader()->beginWriting("text/html", "UTF-8");
     m_popupClient->writeDocument(*writer);
-    writer->end();
+    frame->loader()->activeDocumentLoader()->endWriting(writer);
     return true;
 }
 
 void WebPagePopupImpl::destroyPage()
 {
     if (!m_page)
         return;
 
     if (m_page->mainFrame())
         m_page->mainFrame()->loader()->frameDetached();
@@ skipping 115 matching lines @@
     // A WebPagePopupImpl instance usually has two references.
     //  - One owned by the instance itself. It represents the visible widget.
     //  - One owned by a WebViewImpl. It's released when the WebViewImpl ask the
     //    WebPagePopupImpl to close.
     // We need them because the closing operation is asynchronous and the widget
     // can be closed while the WebViewImpl is unaware of it.
     return adoptRef(new WebPagePopupImpl(client)).leakRef();
 }
 
 } // namespace WebKit