From: John Layt <jlayt@kde.org>
Date: Fri, 1 Jun 2012 20:06:26 +0000 (+0100)
Subject: QtPrintSupport: Move printer list into plugin base class
X-Git-Tag: 071012110112~500
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=20c7fa62caad8f93e3cfa81bcbfed4b753c3e92c;p=profile%2Fivi%2Fqtbase.git

QtPrintSupport: Move printer list into plugin base class

Move the cached printer list from the Windows plugin into the base
class so the CUPS plugin can also use it in a subsequent commit.

Change-Id: Idf5fb7fc49b1820aad58f16e8854ab9120caa193
Reviewed-by: Teemu Katajisto <teemu.katajisto@digia.com>
Reviewed-by: John Layt <jlayt@kde.org>
---

diff --git a/src/plugins/printsupport/windows/qwindowsprintersupport.cpp b/src/plugins/printsupport/windows/qwindowsprintersupport.cpp
index 3715aa1..85b18c2 100644
--- a/src/plugins/printsupport/windows/qwindowsprintersupport.cpp
+++ b/src/plugins/printsupport/windows/qwindowsprintersupport.cpp
@@ -66,7 +66,7 @@ QWindowsPrinterSupport::QWindowsPrinterSupport()
                 QString printerName(QString::fromWCharArray(infoList[i].pPrinterName));
                 bool isDefault = (printerName == defaultPrinterName);
                 QPrinterInfo printerInfo = createPrinterInfo(printerName, QString(), QString(), QString(), isDefault, i);
-                mPrinterList.append(printerInfo);
+                m_printers.append(printerInfo);
             }
         }
         delete [] buffer;
@@ -93,9 +93,4 @@ QList<QPrinter::PaperSize> QWindowsPrinterSupport::supportedPaperSizes(const QPr
     return QWin32PrintEngine::supportedPaperSizes(printerInfo);
 }
 
-QList<QPrinterInfo> QWindowsPrinterSupport::availablePrinters()
-{
-    return mPrinterList;
-}
-
 QT_END_NAMESPACE
diff --git a/src/plugins/printsupport/windows/qwindowsprintersupport.h b/src/plugins/printsupport/windows/qwindowsprintersupport.h
index be8758a..52ff5bb 100644
--- a/src/plugins/printsupport/windows/qwindowsprintersupport.h
+++ b/src/plugins/printsupport/windows/qwindowsprintersupport.h
@@ -42,7 +42,6 @@
 #ifndef WINDOWSPRINTERSUPPORT_H
 #define WINDOWSPRINTERSUPPORT_H
 
-#include <QtCore/QList>
 #include <qpa/qplatformprintersupport.h>
 
 QT_BEGIN_HEADER
@@ -59,12 +58,6 @@ public:
     virtual QPrintEngine *createNativePrintEngine(QPrinter::PrinterMode printerMode);
     virtual QPaintEngine *createPaintEngine(QPrintEngine *printEngine, QPrinter::PrinterMode);
     virtual QList<QPrinter::PaperSize> supportedPaperSizes(const QPrinterInfo &) const;
-
-    virtual QList<QPrinterInfo> availablePrinters();
-private:
-
-    QList<QPrinterInfo> mPrinterList;
-    QPrinter::PrinterMode mCurrentMode;
 };
 
 QT_END_NAMESPACE
diff --git a/src/printsupport/kernel/qplatformprintersupport.h b/src/printsupport/kernel/qplatformprintersupport.h
index 074405f..e922b5f 100644
--- a/src/printsupport/kernel/qplatformprintersupport.h
+++ b/src/printsupport/kernel/qplatformprintersupport.h
@@ -52,11 +52,12 @@
 
 #include <QtPrintSupport/qprinter.h>
 
+#include <QtCore/qlist.h>
+
 QT_BEGIN_HEADER
 
 QT_BEGIN_NAMESPACE
 
-
 #ifndef QT_NO_PRINTER
 
 class QPrintEngine;
@@ -83,6 +84,8 @@ protected:
     static QPrinterInfo createPrinterInfo(const QString &name, const QString &description,
                                           const QString &location, const QString &makeAndModel,
                                           bool isDefault, int index);
+
+    QList<QPrinterInfo> m_printers;
 };
 
 #endif // QT_NO_PRINTER
diff --git a/src/printsupport/kernel/qplatformprintersupport_qpa.cpp b/src/printsupport/kernel/qplatformprintersupport_qpa.cpp
index d6ffd9b..0a386ea 100644
--- a/src/printsupport/kernel/qplatformprintersupport_qpa.cpp
+++ b/src/printsupport/kernel/qplatformprintersupport_qpa.cpp
@@ -84,7 +84,7 @@ QList<QPrinter::PaperSize> QPlatformPrinterSupport::supportedPaperSizes(const QP
 
 QList<QPrinterInfo> QPlatformPrinterSupport::availablePrinters()
 {
-    return QList<QPrinterInfo>();
+    return m_printers;
 }
 
 QPrinterInfo QPlatformPrinterSupport::defaultPrinter()