Reviewed/landed by Adam.

        Fixes: http://bugzilla.opendarwin.org/show_bug.cgi?id=10559
        Confirm to WebKit style guide - last fixes :-)

        * platform/ResourceLoaderClient.h:
        * platform/qt/ComboBoxQt.cpp:
        * platform/qt/FrameQt.cpp:
        (WebCore::doScroll):
        (WebCore::FrameQt::FrameQt):
        (WebCore::FrameQt::openURL):
        (WebCore::FrameQt::submitForm):
        (WebCore::FrameQt::setTitle):
        (WebCore::FrameQt::passSubframeEventToSubframe):
        (WebCore::FrameQt::registerCommandForUndo):
        (WebCore::FrameQt::registerCommandForRedo):
        (WebCore::FrameQt::keyEvent):
        (WebCore::FrameQt::setFrameGeometry):
        * platform/qt/GlyphMapQt.cpp:
        (WebCore::GlyphMap::fillPage):
        * platform/qt/GraphicsContextQt.cpp:
        (WebCore::toQtCompositionMode):
        (WebCore::toQtLineCap):
        (WebCore::toQtLineJoin):
        (WebCore::TextShadow::TextShadow):
        (WebCore::GraphicsContextPlatformPrivate::p):
        * platform/qt/ImageQt.cpp:
        (WebCore::FrameData::clear):
        (WebCore::Image::supportsType):
        * platform/qt/IntSizeQt.cpp:
        * platform/qt/LineEditQt.cpp:
        * platform/qt/ListBoxQt.cpp:
        * platform/qt/PageQt.cpp:
        (WebCore::Page::windowRect):
        * platform/qt/PathQt.cpp:
        (WebCore::Path::~Path):
        * platform/qt/ResourceLoaderCurl.cpp:
        (WebCore::ResourceLoader::assembleResponseHeaders):
        (WebCore::ResourceLoader::retrieveCharset):
        (WebCore::ResourceLoader::receivedResponse):
        * platform/qt/ResourceLoaderManager.cpp:
        (WebCore::headerCallback):
        (WebCore::ResourceLoaderManager::downloadTimerCallback):
        (WebCore::ResourceLoaderManager::add):
        * platform/qt/ScreenQt.cpp:
        (WebCore::screenRect):
        (WebCore::usableScreenRect):
        * platform/qt/ScrollViewQt.cpp:
        (WebCore::ScrollView::ScrollView):
        (WebCore::ScrollView::~ScrollView):
        (WebCore::ScrollView::setParentWidget):
        (WebCore::ScrollView::addChild):
        * platform/qt/SharedTimerQt.cpp:
        (WebCore::setSharedTimerFiredFunction):
        * platform/qt/SharedTimerQt.h:
        (WebCore::SharedTimerQt::SharedTimerQt):
        (WebCore::SharedTimerQt::fire):
        * platform/qt/SystemTimeQt.cpp:
        (WebCore::currentTime):
        * platform/qt/TextEditQt.cpp:
        (WebCore::PlatformTextEdit::setParentWidget):
        (WebCore::PlatformTextEdit::text):
        (WebCore::PlatformTextEdit::sizeWithColumnsAndRows):



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@16033 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/platform/ResourceLoaderClient.h b/WebCore/platform/ResourceLoaderClient.h
index 92c1d1d..3f64626 100644
--- a/WebCore/platform/ResourceLoaderClient.h
+++ b/WebCore/platform/ResourceLoaderClient.h
@@ -36,6 +36,10 @@
 #endif
 #endif
 
+#if PLATFORM(QT)
+#include <QString>
+#endif
+
 namespace WebCore {
 
 #if PLATFORM(MAC)
@@ -43,7 +47,7 @@
     typedef NSURLResponse* PlatformResponse;
 #elif PLATFORM(QT)
     typedef void* PlatformData;
-    typedef char* PlatformResponse;
+    typedef QString PlatformResponse;
 #else
     // Not sure what the strategy for this will be on other platforms.
     typedef struct PlatformDataStruct *PlatformData;
diff --git a/WebCore/platform/qt/ComboBoxQt.cpp b/WebCore/platform/qt/ComboBoxQt.cpp
index 44ebffe..c957ee8 100644
--- a/WebCore/platform/qt/ComboBoxQt.cpp
+++ b/WebCore/platform/qt/ComboBoxQt.cpp
@@ -26,6 +26,8 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
+#include "config.h"
+
 #include <QComboBox>
 
 #include "PlatformComboBox.h"
diff --git a/WebCore/platform/qt/FrameQt.cpp b/WebCore/platform/qt/FrameQt.cpp
index 0e0bb6c..201d92a 100644
--- a/WebCore/platform/qt/FrameQt.cpp
+++ b/WebCore/platform/qt/FrameQt.cpp
@@ -67,18 +67,18 @@
 {
     // FIXME: The scrolling done here should be done in the default handlers
     // of the elements rather than here in the part.
-    if(!r)
+    if (!r)
         return;
 
     //broken since it calls scroll on scrollbars
     //and we have none now
     //r->scroll(direction, KWQScrollWheel, multiplier);
-    if(!r->layer())
+    if (!r->layer())
         return;
 
     int x = r->layer()->scrollXOffset();
     int y = r->layer()->scrollYOffset();
-    if(isHorizontal)
+    if (isHorizontal)
         x += multiplier;
     else
         y += multiplier;
@@ -94,7 +94,6 @@
 FrameQt::FrameQt()
     : Frame(new Page, 0)
 {
-    qDebug("FrameQt::FrameQt");
     init();
 
     page()->setMainFrame(this);
@@ -141,7 +140,6 @@
 
 bool FrameQt::openURL(const KURL& url)
 {
-    qDebug("openURL(%s)", url.url().latin1());
     didOpenURL(url);
 
     // FIXME: Use mimetype logic from KIO!
@@ -157,16 +155,14 @@
 
 void FrameQt::submitForm(const ResourceRequest& request)
 {
-    qDebug("FrameQt::submitForm()");
-
     // FIXME: this is a hack inherited from FrameMac, and should be pushed into Frame
-    if(d->m_submittedFormURL == request.url())
+    if (d->m_submittedFormURL == request.url())
         return;
 
     d->m_submittedFormURL = request.url();
 
     /* FIXME: Once we have a KPart - named "FramePartQt" - we can let that inherit from FrameQtClient and implement the functions...)
-    if(m_client)
+    if (m_client)
         m_client->submitForm(request.doPost() ? "POST" : "GET", request.url(), &request.postData);
     */
 
@@ -212,7 +208,7 @@
 
 void FrameQt::setTitle(const String& title)
 {
-    if(view() && view()->parentWidget())
+    if (view() && view()->parentWidget())
         view()->parentWidget()->setWindowTitle(title);
 }
 
@@ -230,7 +226,7 @@
 
 bool FrameQt::passSubframeEventToSubframe(MouseEventWithHitTestResults& mev, Frame*)
 {
-    if(mev.targetNode() == 0)
+    if (mev.targetNode() == 0)
         return true;
  
     return false;
@@ -381,12 +377,12 @@
     return 0;
 }
 
-void FrameQt::registerCommandForUndo(const EditCommandPtr&)
+void FrameQt::registerCommandForUndo(PassRefPtr<EditCommand>)
 {
     notImplemented();
 }
 
-void FrameQt::registerCommandForRedo(const EditCommandPtr&)
+void FrameQt::registerCommandForRedo(PassRefPtr<EditCommand>)
 {
     notImplemented();
 }
@@ -499,21 +495,21 @@
     // Check for cases where we are too early for events -- possible unmatched key up
     // from pressing return in the location bar.
     Document* doc = document();
-    if(!doc)
+    if (!doc)
         return false;
 
     Node* node = doc->focusNode();
-    if(!node) {
+    if (!node) {
         if (doc->isHTMLDocument())
             node = doc->body();
         else
             node = doc->documentElement();
 
-        if(!node)
+        if (!node)
             return false;
     }
 
-    if(!keyEvent.isKeyUp())
+    if (!keyEvent.isKeyUp())
         prepareForUserAction();
 
     result = !EventTargetNodeCast(node)->dispatchKeyEvent(keyEvent);
@@ -539,7 +535,7 @@
 
 void FrameQt::setFrameGeometry(const IntRect& r)
 {
-    setFrameGeometry((QRect) r);
+    setFrameGeometry(QRect(r));
 }
 
 }
diff --git a/WebCore/platform/qt/GlyphMapQt.cpp b/WebCore/platform/qt/GlyphMapQt.cpp
index ad22ab3..8542092 100644
--- a/WebCore/platform/qt/GlyphMapQt.cpp
+++ b/WebCore/platform/qt/GlyphMapQt.cpp
@@ -36,9 +36,8 @@
 
 bool GlyphMap::fillPage(GlyphPage* page, UChar* buffer, unsigned bufferLength, const FontData* fontData)
 {
-    for (unsigned i = 0; i < bufferLength; i++) {
+    for (unsigned i = 0; i < bufferLength; i++)
         page->setGlyphDataForIndex(i, buffer[i], fontData);
-    }
 
     return true;
 }
diff --git a/WebCore/platform/qt/GraphicsContextQt.cpp b/WebCore/platform/qt/GraphicsContextQt.cpp
index 617befd..952aa2b 100644
--- a/WebCore/platform/qt/GraphicsContextQt.cpp
+++ b/WebCore/platform/qt/GraphicsContextQt.cpp
@@ -30,6 +30,8 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include "config.h"
+
 #include "GraphicsContext.h"
 #include "Path.h"
 
@@ -53,34 +55,34 @@
 static QPainter::CompositionMode toQtCompositionMode(CompositeOperator op)
 {
     switch (op) {
-    case CompositeClear:
-        return QPainter::CompositionMode_Clear;
-    case CompositeCopy:
-        return QPainter::CompositionMode_Source;
-    case CompositeSourceOver:
-        return QPainter::CompositionMode_SourceOver;
-    case CompositeSourceIn:
-        return QPainter::CompositionMode_SourceIn;
-    case CompositeSourceOut:
-        return QPainter::CompositionMode_SourceOut;
-    case CompositeSourceAtop:
-        return QPainter::CompositionMode_SourceAtop;
-    case CompositeDestinationOver:
-        return QPainter::CompositionMode_DestinationOver;
-    case CompositeDestinationIn:
-        return QPainter::CompositionMode_DestinationIn;
-    case CompositeDestinationOut:
-        return QPainter::CompositionMode_DestinationOut;
-    case CompositeDestinationAtop:
-        return QPainter::CompositionMode_DestinationAtop;
-    case CompositeXOR:
-        return QPainter::CompositionMode_Xor;
-    case CompositePlusDarker:
-        return QPainter::CompositionMode_SourceOver;
-    case CompositeHighlight:
-        return QPainter::CompositionMode_SourceOver;
-    case CompositePlusLighter:
-        return QPainter::CompositionMode_SourceOver;
+        case CompositeClear:
+            return QPainter::CompositionMode_Clear;
+        case CompositeCopy:
+            return QPainter::CompositionMode_Source;
+        case CompositeSourceOver:
+            return QPainter::CompositionMode_SourceOver;
+        case CompositeSourceIn:
+            return QPainter::CompositionMode_SourceIn;
+        case CompositeSourceOut:
+            return QPainter::CompositionMode_SourceOut;
+        case CompositeSourceAtop:
+            return QPainter::CompositionMode_SourceAtop;
+        case CompositeDestinationOver:
+            return QPainter::CompositionMode_DestinationOver;
+        case CompositeDestinationIn:
+            return QPainter::CompositionMode_DestinationIn;
+        case CompositeDestinationOut:
+            return QPainter::CompositionMode_DestinationOut;
+        case CompositeDestinationAtop:
+            return QPainter::CompositionMode_DestinationAtop;
+        case CompositeXOR:
+            return QPainter::CompositionMode_Xor;
+        case CompositePlusDarker:
+            return QPainter::CompositionMode_SourceOver;
+        case CompositeHighlight:
+            return QPainter::CompositionMode_SourceOver;
+        case CompositePlusLighter:
+            return QPainter::CompositionMode_SourceOver;
     }
 
     return QPainter::CompositionMode_SourceOver;
@@ -89,12 +91,12 @@
 static Qt::PenCapStyle toQtLineCap(LineCap lc)
 {
     switch (lc) {
-    case ButtCap:
-        return Qt::FlatCap;
-    case RoundCap:
-        return Qt::RoundCap;
-    case SquareCap:
-        return Qt::SquareCap;
+        case ButtCap:
+            return Qt::FlatCap;
+        case RoundCap:
+            return Qt::RoundCap;
+        case SquareCap:
+            return Qt::SquareCap;
     }
 
     return Qt::FlatCap;
@@ -103,12 +105,12 @@
 static Qt::PenJoinStyle toQtLineJoin(LineJoin lj)
 {
     switch (lj) {
-    case MiterJoin:
-        return Qt::SvgMiterJoin;
-    case RoundJoin:
-        return Qt::RoundJoin;
-    case BevelJoin:
-        return Qt::BevelJoin;
+        case MiterJoin:
+            return Qt::SvgMiterJoin;
+        case RoundJoin:
+            return Qt::RoundJoin;
+        case BevelJoin:
+            return Qt::BevelJoin;
     }
 
     return Qt::MiterJoin;
@@ -144,9 +146,9 @@
 struct TextShadow
 {
     TextShadow()
-    : x(0)
-    , y(0)
-    , blur(0)
+        : x(0)
+        , y(0)
+        , blur(0)
     {
     }
 
@@ -165,7 +167,7 @@
     GraphicsContextPlatformPrivate(QPainter* painter);
     ~GraphicsContextPlatformPrivate();
 
-    inline QPainter& p()
+    QPainter& p()
     {
         if (layers.isEmpty()) {
             if (redirect)
diff --git a/WebCore/platform/qt/ImageQt.cpp b/WebCore/platform/qt/ImageQt.cpp
index 18aa014..53a92ee 100644
--- a/WebCore/platform/qt/ImageQt.cpp
+++ b/WebCore/platform/qt/ImageQt.cpp
@@ -51,7 +51,7 @@
 {
     if (m_frame) {
         m_frame = 0;
-        m_duration = 0.;
+        m_duration = 0.0f;
         m_hasAlpha = true;
     }
 }
@@ -82,8 +82,6 @@
 
 bool Image::supportsType(const String& type)
 {
-    qDebug() << "does support image format? "<< type;
-
     QList<QByteArray> formats = QImageReader::supportedImageFormats();
     foreach(QByteArray format, formats) {
         if (format == (QString) type)
diff --git a/WebCore/platform/qt/IntSizeQt.cpp b/WebCore/platform/qt/IntSizeQt.cpp
index e164208..4f2bf35 100644
--- a/WebCore/platform/qt/IntSizeQt.cpp
+++ b/WebCore/platform/qt/IntSizeQt.cpp
@@ -25,9 +25,10 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include "config.h"
+
 #include <QSize>
 
-#include "config.h"
 #include "IntSize.h"
 
 namespace WebCore {
diff --git a/WebCore/platform/qt/LineEditQt.cpp b/WebCore/platform/qt/LineEditQt.cpp
index 424f993..b07c671 100644
--- a/WebCore/platform/qt/LineEditQt.cpp
+++ b/WebCore/platform/qt/LineEditQt.cpp
@@ -26,9 +26,10 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
+#include "config.h"
+
 #include <QLineEdit>
 
-#include <config.h>
 #include "GraphicsTypes.h"
 #include "TextDirection.h"
 #include "Widget.h"
diff --git a/WebCore/platform/qt/ListBoxQt.cpp b/WebCore/platform/qt/ListBoxQt.cpp
index 5502b8f..4707b4c 100644
--- a/WebCore/platform/qt/ListBoxQt.cpp
+++ b/WebCore/platform/qt/ListBoxQt.cpp
@@ -26,6 +26,8 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
+#include "config.h"
+
 #include <QScrollBar>
 #include <QListWidget>
 
diff --git a/WebCore/platform/qt/PageQt.cpp b/WebCore/platform/qt/PageQt.cpp
index b534bbc..80a1824 100644
--- a/WebCore/platform/qt/PageQt.cpp
+++ b/WebCore/platform/qt/PageQt.cpp
@@ -27,11 +27,11 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
+#include "config.h"
+
 #include <QWidget>
 
-#include "config.h"
 #include "Page.h"
-
 #include "IntRect.h"
 #include "FloatRect.h"
 #include "Frame.h"
@@ -54,7 +54,7 @@
     if (!widget)
         return FloatRect();
 
-    return (IntRect) widget->geometry();
+    return IntRect(widget->geometry());
 }
 
 void Page::setWindowRect(const FloatRect& r)
diff --git a/WebCore/platform/qt/PathQt.cpp b/WebCore/platform/qt/PathQt.cpp
index 01314f0..41ba37f 100644
--- a/WebCore/platform/qt/PathQt.cpp
+++ b/WebCore/platform/qt/PathQt.cpp
@@ -44,7 +44,6 @@
 Path::~Path()
 {
     delete m_path;
-    m_path = 0;
 }
 
 Path::Path(const Path& other)
diff --git a/WebCore/platform/qt/ResourceLoaderCurl.cpp b/WebCore/platform/qt/ResourceLoaderCurl.cpp
index 0aeafef..9e8de77 100644
--- a/WebCore/platform/qt/ResourceLoaderCurl.cpp
+++ b/WebCore/platform/qt/ResourceLoaderCurl.cpp
@@ -27,7 +27,6 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
-#include <QDebug>
 #include "config.h"
 #include "ResourceLoader.h"
 
@@ -67,9 +66,8 @@
  
         // FIXME: Move the client activation to receivedResponse(), once
         // we use KIO, and receivedResponse() is called only once.
-        if (d->client) {
-            d->client->receivedResponse(const_cast<ResourceLoader *>(this), (char *) d->response.data());
-        }
+        if (d->client)
+            d->client->receivedResponse(const_cast<ResourceLoader*>(this), d->response);
 
         d->response = QString(); // Reset
     }
@@ -77,9 +75,8 @@
 
 void ResourceLoader::retrieveCharset() const
 {
-    if (!d->retrievedCharset) {
+    if (!d->retrievedCharset)
         d->retrievedCharset = true;
-    }
 
     // FIXME: We can just parse the headers here, but once we use KIO
     // we can set the response parameter to sth. else than a "char*".
@@ -87,7 +84,7 @@
     notImplemented();
 }
 
-void ResourceLoader::receivedResponse(char* response)
+void ResourceLoader::receivedResponse(QString response)
 {
     Q_ASSERT(method() == "POST");
 
@@ -102,7 +99,7 @@
     //
     // Anyway, let's collect the response data here, as the ResourceLoaderManager
     // calls us for every line of the header it receives.
-    d->response += QString::fromLatin1(response);
+    d->response += response;
 }
 
 } // namespace WebCore
diff --git a/WebCore/platform/qt/ResourceLoaderManager.cpp b/WebCore/platform/qt/ResourceLoaderManager.cpp
index bb03095..d5207bf 100644
--- a/WebCore/platform/qt/ResourceLoaderManager.cpp
+++ b/WebCore/platform/qt/ResourceLoaderManager.cpp
@@ -74,9 +74,8 @@
     ResourceLoader* job = static_cast<ResourceLoader*>(obj);
     ResourceLoaderInternal* d = job->getInternal();
  
-    if (job->method() == "POST") {
+    if (job->method() == "POST")
         job->receivedResponse(ptr);
-    }
     
     int totalSize = size * nmemb;
     return totalSize;
@@ -140,12 +139,12 @@
                 // find the node which has same d->m_handle as completed transfer
                 CURL* chandle = msg->easy_handle;
                 assert(chandle);
-                ResourceLoader *job;
+                ResourceLoader* job;
                 curl_easy_getinfo(chandle, CURLINFO_PRIVATE, &job);
                 assert(job); //fixme: assert->if ?
                 // if found, delete it
                 if (job) {
-                    ResourceLoaderInternal *d = job->getInternal();
+                    ResourceLoaderInternal* d = job->getInternal();
                     switch (msg->data.result) {
                         case CURLE_OK: {
                             // use this to authenticate
@@ -211,7 +210,7 @@
     if (job->method() == "POST") {
         DeprecatedString postData = job->postData().flattenToString();
 
-        char *postDataString = (char *) malloc(postData.length() + 1);
+        char* postDataString = (char*)malloc(postData.length() + 1);
         strncpy(postDataString, postData.ascii(), postData.length());
         postDataString[postData.length()] = '\0';
 
diff --git a/WebCore/platform/qt/ScreenQt.cpp b/WebCore/platform/qt/ScreenQt.cpp
index 391ab6a..bec308d 100644
--- a/WebCore/platform/qt/ScreenQt.cpp
+++ b/WebCore/platform/qt/ScreenQt.cpp
@@ -49,7 +49,7 @@
     if (!dw)
         return FloatRect();
 
-    return (IntRect) dw->screenGeometry(qw);
+    return IntRect(dw->screenGeometry(qw));
 }
 
 int screenDepth(Widget* widget)
@@ -72,7 +72,7 @@
     if (!dw)
         return FloatRect();
 
-    return (IntRect) dw->availableGeometry(qw);
+    return IntRect(dw->availableGeometry(qw));
 }
 
 }
diff --git a/WebCore/platform/qt/ScrollViewQt.cpp b/WebCore/platform/qt/ScrollViewQt.cpp
index 166173f..01a0ece 100644
--- a/WebCore/platform/qt/ScrollViewQt.cpp
+++ b/WebCore/platform/qt/ScrollViewQt.cpp
@@ -47,17 +47,14 @@
 ScrollView::ScrollView()
     : m_area(new QScrollArea(0))
 {
-    qDebug("ScrollView::ScrollView()");
 }
 
 ScrollView::~ScrollView()
 {
-    qDebug("ScrollView::~ScrollView()");
 }
 
 void ScrollView::setParentWidget(QWidget* parent)
 {
-    qDebug("ScrollView::setParentWidget(), parent=%p", parent);
     Widget::setParentWidget(parent);
 
     // 'isFrameView()' can not be called yet in the constructor!
@@ -229,7 +226,6 @@
 
 void ScrollView::addChild(Widget* child, int x, int y)
 {
-    qDebug("ScrollView::addChild(), child=%p, x=%i, y=%i", child, x, y);
     Q_ASSERT(child != 0);
     Q_ASSERT(m_area && m_area->widget());
 
diff --git a/WebCore/platform/qt/SharedTimerQt.cpp b/WebCore/platform/qt/SharedTimerQt.cpp
index 8ef421b..cceb05a 100644
--- a/WebCore/platform/qt/SharedTimerQt.cpp
+++ b/WebCore/platform/qt/SharedTimerQt.cpp
@@ -34,7 +34,7 @@
 
 void setSharedTimerFiredFunction(void (*f)())
 {
-    SharedTimerQt::inst()->_timerFunction = f;
+    SharedTimerQt::inst()->m_timerFunction = f;
 }
 
 void setSharedTimerFireTime(double fireTime)
diff --git a/WebCore/platform/qt/SharedTimerQt.h b/WebCore/platform/qt/SharedTimerQt.h
index cda9c4e..ac40324 100644
--- a/WebCore/platform/qt/SharedTimerQt.h
+++ b/WebCore/platform/qt/SharedTimerQt.h
@@ -38,11 +38,12 @@
 class SharedTimerQt : public QTimer {
 Q_OBJECT
 protected:
-    SharedTimerQt() : QTimer()
+    SharedTimerQt()
+        : QTimer()
+        , m_timerFunction(0)
     {
         connect(this, SIGNAL(timeout()), this, SLOT(fire()));
         setSingleShot(true);
-        _timerFunction = 0;
     }
 
     ~SharedTimerQt()
@@ -58,13 +59,13 @@
         return s_self;
     }
 
-    void (*_timerFunction)();
+    void (*m_timerFunction)();
 
 public Q_SLOTS:
     void fire()
     {
-        if (_timerFunction)
-            (_timerFunction)();
+        if (m_timerFunction)
+            (m_timerFunction)();
     }
 
 private:
diff --git a/WebCore/platform/qt/SystemTimeQt.cpp b/WebCore/platform/qt/SystemTimeQt.cpp
index bb55d57..5a32316 100644
--- a/WebCore/platform/qt/SystemTimeQt.cpp
+++ b/WebCore/platform/qt/SystemTimeQt.cpp
@@ -36,7 +36,7 @@
 {
     time_t ttime;
     time(&ttime);
-    return (double) ttime;
+    return (double)ttime;
 }
 
 }
diff --git a/WebCore/platform/qt/TextEditQt.cpp b/WebCore/platform/qt/TextEditQt.cpp
index 023905c..fd003aa 100644
--- a/WebCore/platform/qt/TextEditQt.cpp
+++ b/WebCore/platform/qt/TextEditQt.cpp
@@ -52,7 +52,7 @@
 {
     Widget::setParentWidget(parent);
 
-    QTextEdit *widget = new QTextEdit(parent, "");
+    QTextEdit* widget = new QTextEdit(parent, "");
     setQWidget(widget);
 }
 
@@ -122,6 +122,7 @@
 String PlatformTextEdit::text() const
 {
     notImplemented();
+    return String();
 }
 
 String PlatformTextEdit::textWithHardLineBreaks() const
@@ -189,6 +190,7 @@
 IntSize PlatformTextEdit::sizeWithColumnsAndRows(int numColumns, int numRows) const
 {
     notImplemented();
+    return IntSize();
 }
 
 bool PlatformTextEdit::checksDescendantsForFocus() const