Clean up cloud printing aura stub code.

printing/printed_document.cc

Index: printing/printed_document.cc
diff --git a/printing/printed_document.cc b/printing/printed_document.cc
index 28a70e9ac73f567d6ba8d9c5fae71c70eef4c89c..8cfcee51a4878dddf64e3b8aa68c8062d0eab5f0 100644
--- a/printing/printed_document.cc
+++ b/printing/printed_document.cc
@@ -230,4 +230,12 @@ PrintedDocument::Immutable::Immutable(const PrintSettings& settings,
 PrintedDocument::Immutable::~Immutable() {
 }
 
+#if defined(OS_POSIX) && defined(USE_AURA)

[Albert Bodenhamer, 2011/12/01 01:09:05]

Shouldn't this be OS_CHROMEOS || USE_AURA?

[jennyz, 2011/12/02 14:40:18]

This function signature is only declared for OS_POSIX in printed_document.h, so
defined(OS_POSIX) is needed, are you sure you want this for OS_CHROMEOS non-aura
build? I thought  that will use gtk implementation.

[Albert Bodenhamer, 2011/12/02 17:57:17]

On chrome os there is no native printing, only cloud print, so no need to use
the GTK version.

You can do OS_POSIX || USE_AURA, but I'm afraid you'll get a symbol collision on
Linux since it needs to pull in the GTK version.  You don't want && since you'll
want this for the non-aura chromeos build.

On 2011/12/02 14:40:18, jennyz wrote:

+// This function is not used on aura linux/chromeos.
+void PrintedDocument::RenderPrintedPage(const PrintedPage& page,
+                                        PrintingContext* context) const {
+  NOTIMPLEMENTED();

[Albert Bodenhamer, 2011/12/01 00:47:23]

I don't think NOTIMPLEMENTED is correct here.  NOTIMPLEMENTED will log an error.
 Doing nothing for this function is the CORRECT thing to do.  See logging.h line
913 for a discussion of NOTIMPLEMENTED

[jennyz, 2011/12/02 14:40:18]

Done.

+}
+#endif
+
 }  // namespace printing