Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(331)

Unified Diff: chrome/browser/renderer_host/resource_message_filter_gtk.cc

Issue 203062: Linux: print page to file rather than using shared memory to send it to the b... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: style Created 11 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/renderer_host/resource_message_filter.cc ('k') | chrome/common/render_messages_internal.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/renderer_host/resource_message_filter_gtk.cc
===================================================================
--- chrome/browser/renderer_host/resource_message_filter_gtk.cc (revision 26282)
+++ chrome/browser/renderer_host/resource_message_filter_gtk.cc (working copy)
@@ -4,11 +4,20 @@
#include "chrome/browser/renderer_host/resource_message_filter.h"
+#include <fcntl.h>
+#include <map>
+
+#include "app/l10n_util.h"
#include "base/clipboard.h"
+#include "base/file_util.h"
#include "base/gfx/gtk_native_view_id_manager.h"
+#include "base/path_service.h"
+#include "base/singleton.h"
#include "chrome/browser/chrome_thread.h"
+#include "chrome/common/chrome_paths.h"
#include "chrome/common/render_messages.h"
#include "chrome/common/x11_util.h"
+#include "grit/generated_resources.h"
#include "webkit/api/public/WebScreenInfo.h"
#include "webkit/api/public/x11/WebScreenInfoFactory.h"
@@ -16,6 +25,16 @@
using WebKit::WebScreenInfo;
using WebKit::WebScreenInfoFactory;
+namespace {
+
+typedef std::map<int, FilePath> FdMap;
+
+struct PrintingFileDescriptorMap {
+ FdMap map;
+};
+
+} // namespace
+
// We get null window_ids passed into the two functions below; please see
// http://crbug.com/9060 for more details.
@@ -210,3 +229,47 @@
this, &ResourceMessageFilter::DoOnClipboardReadHTML, buffer,
reply_msg));
}
+
+// Called on the IO thread.
+void ResourceMessageFilter::OnAllocateTempFileForPrinting(
+ base::FileDescriptor* temp_file_fd, int* fd_in_browser) {
+ temp_file_fd->fd = *fd_in_browser = -1;
+
+ FilePath path;
+ if (!file_util::CreateTemporaryFile(&path))
+ return;
+
+ int fd = open(path.value().c_str(), O_WRONLY);
+ if (fd < 0)
+ return;
+
+ // We need to remember the FilePath of the temporary file because we need
+ // it when we want to rename/move it, and more importantly, to print it
+ // when we print by using gtk_print_job_set_source_file().
+ FdMap* map = &Singleton<PrintingFileDescriptorMap>::get()->map;
+ FdMap::iterator it = map->find(fd);
+ if (it != map->end()) {
+ NOTREACHED() << "The file descriptor is in use. fd=" << fd;
+ return;
+ }
+
+ (*map)[fd] = path;
+ temp_file_fd->fd = *fd_in_browser = fd;
+ temp_file_fd->auto_close = true;
+}
+
+// Called on the IO thread.
+void ResourceMessageFilter::OnTempFileForPrintingWritten(int fd_in_browser) {
+ FdMap* map = &Singleton<PrintingFileDescriptorMap>::get()->map;
+ FdMap::iterator it = map->find(fd_in_browser);
+ if (it == map->end()) {
+ NOTREACHED() << "Got a file descriptor that we didn't pass to the "
+ "renderer: " << fd_in_browser;
+ return;
+ }
+
+ // TODO(estade): print it.
+
+ // Erase the entry in the map.
+ map->erase(it);
+}
« no previous file with comments | « chrome/browser/renderer_host/resource_message_filter.cc ('k') | chrome/common/render_messages_internal.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698