PrintPreview: Print Preview is not staying associated with initiator renderer.

chrome/browser/ui/webui/print_preview_ui.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/print_preview_ui.h"
 
+#include "base/string_util.h"
 #include "base/values.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/webui/print_preview_handler.h"
-#include "chrome/browser/ui/webui/print_preview_ui_html_source.h"
-#include "content/browser/browser_thread.h"
 #include "content/browser/tab_contents/tab_contents.h"
 
-PrintPreviewUI::PrintPreviewUI(TabContents* contents)
-    : WebUI(contents),
-      html_source_(new PrintPreviewUIHTMLSource()) {
+PrintPreviewUI::PrintPreviewUI(TabContents* contents) : WebUI(contents) {
   // PrintPreviewUI owns |handler|.
   PrintPreviewHandler* handler = new PrintPreviewHandler();
   AddMessageHandler(handler->Attach(this));
 
-  // Set up the chrome://print/ source.
-  contents->profile()->GetChromeURLDataManager()->AddDataSource(html_source_);
+  // Set up the chrome://print/ data source.
+  data_source_ = PrintPreviewDataSource::GetInstance();
+  contents->profile()->GetChromeURLDataManager()->AddDataSource(data_source_);
+
+  // Store the PrintPreviewUIAddress as a string.
+  char preview_ui_addr[128];

[Lei Zhang, 2011/05/26 01:48:31]

Since we are using the memory address as the identifier, I think the string size
should be: (2 * sizeof(this)) + 3, where 3 = '0x' + null terminator.

[kmadhusu, 2011/05/26 15:51:22]

Done.

+  base::snprintf(preview_ui_addr, 128, "%p", this);
+  preview_ui_addr_str_.reset(new std::string(preview_ui_addr));
 }
 
 PrintPreviewUI::~PrintPreviewUI() {
+  data_source_->RemoveEntry(*preview_ui_addr_str_);
 }
 
-PrintPreviewUIHTMLSource* PrintPreviewUI::html_source() {
-  return html_source_.get();
+void PrintPreviewUI::GetPrintPreviewData(
+    PrintPreviewDataSource::PrintPreviewData* data) {
+  data_source_->GetDataEntry(*preview_ui_addr_str_, data);
+}
+
+void PrintPreviewUI::SetPrintPreviewData(
+    const PrintPreviewDataSource::PrintPreviewData& data) {
+  data_source_->SetDataEntry(*preview_ui_addr_str_, data);
 }
 
 void PrintPreviewUI::OnInitiatorTabClosed(
     const std::string& initiator_url) {
   StringValue initiator_tab_url(initiator_url);
   CallJavascriptFunction("onInitiatorTabClosed", initiator_tab_url);
 }
 
 void PrintPreviewUI::OnPreviewDataIsAvailable(int expected_pages_count,
                                               const string16& job_title,
                                               bool modifiable) {
   VLOG(1) << "Print preview request finished with "
           << expected_pages_count << " pages";
   FundamentalValue pages_count(expected_pages_count);
   StringValue title(job_title);
   FundamentalValue is_preview_modifiable(modifiable);
+  StringValue tab_identifier(*preview_ui_addr_str_);
   CallJavascriptFunction("updatePrintPreview", pages_count, title,
-                         is_preview_modifiable);
+                         is_preview_modifiable, tab_identifier);
 }