Patch from Adam Treat to implement default icons / resources for Qt. Also does
a small cleanup to QWebPage::icon. This is a great example of how QWebSettings
is not ideal.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@23733 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 5644c6b..f5a0e2e 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,19 @@
+2007-06-21 Adam Treat <adam@staikos.net>
+
+ Reviewed by George Staikos.
+
+ Implement the default resources on Qt.
+
+ * platform/graphics/BitmapImage.h:
+ * platform/graphics/Image.h:
+ * platform/graphics/qt/ImageQt.cpp:
+ (WebCore::Image::loadPlatformResource):
+ (WebCore::BitmapImage::BitmapImage):
+ (WebCore::BitmapImage::initPlatformData):
+ (WebCore::BitmapImage::invalidatePlatformData):
+ (WebCore::BitmapImage::getPixmap):
+ * platform/qt/TemporaryLinkStubs.cpp:
+
2007-06-21 Oliver Hunt <oliver@apple.com>
Reviewed by Darin.
diff --git a/WebCore/platform/graphics/BitmapImage.h b/WebCore/platform/graphics/BitmapImage.h
index d76172e..5a70808 100644
--- a/WebCore/platform/graphics/BitmapImage.h
+++ b/WebCore/platform/graphics/BitmapImage.h
@@ -89,6 +89,9 @@
class BitmapImage : public Image {
friend class GraphicsContext;
public:
+#if PLATFORM(QT)
+ BitmapImage(const QPixmap &pixmap, ImageObserver* = 0);
+#endif
BitmapImage(ImageObserver* = 0);
~BitmapImage();
@@ -187,6 +190,11 @@
mutable bool m_haveSize; // Whether or not our |m_size| member variable has the final overall image size yet.
bool m_sizeAvailable; // Whether or not we can obtain the size of the first image frame yet from ImageIO.
unsigned m_decodedSize; // The current size of all decoded frames.
+
+#if PLATFORM(QT)
+ QPixmap *m_pixmap;
+#endif
+
};
}
diff --git a/WebCore/platform/graphics/Image.h b/WebCore/platform/graphics/Image.h
index 7784f9a..d4f92c8 100644
--- a/WebCore/platform/graphics/Image.h
+++ b/WebCore/platform/graphics/Image.h
@@ -51,7 +51,7 @@
#endif
#if PLATFORM(QT)
-class QPixmap;
+#include <QPixmap>
#endif
namespace WebCore {
diff --git a/WebCore/platform/graphics/qt/ImageQt.cpp b/WebCore/platform/graphics/qt/ImageQt.cpp
index aba4a87..f3cec6c 100644
--- a/WebCore/platform/graphics/qt/ImageQt.cpp
+++ b/WebCore/platform/graphics/qt/ImageQt.cpp
@@ -49,8 +49,8 @@
#include <math.h>
-// This function loads resources from WebKit
-Vector<char> loadResourceIntoArray(const char*);
+// This function loads resources into WebKit
+QPixmap loadResourcePixmap(const char*);
namespace WebCore {
@@ -71,10 +71,7 @@
Image* Image::loadPlatformResource(const char* name)
{
- Vector<char> arr = loadResourceIntoArray(name);
- Image* img = new BitmapImage();
- RefPtr<SharedBuffer> buffer = new SharedBuffer(arr.data(), arr.size());
- img->setData(buffer, true);
+ BitmapImage* img = new BitmapImage(loadResourcePixmap(name));
return img;
}
@@ -85,13 +82,33 @@
notImplemented();
}
+BitmapImage::BitmapImage(const QPixmap &pixmap, ImageObserver *observer)
+ : Image(observer)
+ , m_currentFrame(0)
+ , m_frames(0)
+ , m_frameTimer(0)
+ , m_repetitionCount(0)
+ , m_repetitionsComplete(0)
+ , m_isSolidColor(false)
+ , m_animatingImageType(true)
+ , m_animationFinished(false)
+ , m_allDataReceived(false)
+ , m_haveSize(false)
+ , m_sizeAvailable(false)
+ , m_decodedSize(0)
+{
+ m_pixmap = new QPixmap(pixmap);
+}
void BitmapImage::initPlatformData()
{
+ m_pixmap = 0;
}
void BitmapImage::invalidatePlatformData()
{
+ delete m_pixmap;
+ m_pixmap = 0;
}
// Drawing Routines
@@ -167,7 +184,10 @@
QPixmap* BitmapImage::getPixmap() const
{
- return const_cast<BitmapImage*>(this)->frameAtIndex(0);
+ if (!m_pixmap)
+ return const_cast<BitmapImage*>(this)->frameAtIndex(0);
+ else
+ return m_pixmap;
}
}
diff --git a/WebCore/platform/qt/TemporaryLinkStubs.cpp b/WebCore/platform/qt/TemporaryLinkStubs.cpp
index c6a9b5b..eb46ae5 100644
--- a/WebCore/platform/qt/TemporaryLinkStubs.cpp
+++ b/WebCore/platform/qt/TemporaryLinkStubs.cpp
@@ -128,8 +128,6 @@
bool AXObjectCache::gAccessibilityEnabled = false;
-Vector<char> loadResourceIntoArray(const char*) { return Vector<char>(); }
-
namespace WebCore {
Vector<String> supportedKeySizes() { notImplemented(); return Vector<String>(); }
