2011-05-20 Kent Tamura <tkent@chromium.org>
authortkent@chromium.org <tkent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 May 2011 08:48:12 +0000 (08:48 +0000)
committertkent@chromium.org <tkent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 May 2011 08:48:12 +0000 (08:48 +0000)
        Reviewed by Ryosuke Niwa.

        Fix style errors in DumpRenderTree/chromium/.
        https://bugs.webkit.org/show_bug.cgi?id=61172

        * DumpRenderTree/chromium/CppBoundClass.cpp:
        (CppBoundClass::getAsCppVariant):
        (CppBoundClass::bindToJavascript):
        * DumpRenderTree/chromium/CppBoundClass.h:
        (CppBoundClass::GetterCallback::~GetterCallback):
        (CppBoundClass::CppBoundClass):
        (CppBoundClass::Callback::~Callback):
        (CppBoundClass::MemberCallback::MemberCallback):
        (CppBoundClass::MemberCallback::~MemberCallback):
        (CppBoundClass::MemberGetterCallback::MemberGetterCallback):
        (CppBoundClass::MemberGetterCallback::~MemberGetterCallback):
        * DumpRenderTree/chromium/CppVariant.h:
        * DumpRenderTree/chromium/DRTDevToolsAgent.h:
        (DRTDevToolsAgent::~DRTDevToolsAgent):
        * DumpRenderTree/chromium/DRTDevToolsClient.cpp:
        (DRTDevToolsClient::sendFrontendLoaded):
        * DumpRenderTree/chromium/DRTDevToolsClient.h:
        * DumpRenderTree/chromium/DumpRenderTree.cpp:
        (main):
        * DumpRenderTree/chromium/EventSender.cpp:
        (SavedEvent::SavedEvent):
        (EventSender::EventSender):
        (EventSender::keyDown):
        * DumpRenderTree/chromium/EventSender.h:
        * DumpRenderTree/chromium/ImageDiff.cpp:
        (Image::Image):
        * DumpRenderTree/chromium/LayoutTestController.cpp:
        (LayoutTestController::LayoutTestController):
        (LayoutTestController::WorkQueue::reset):
        (WorkItemBackForward::WorkItemBackForward):
        (WorkItemLoadingScript::WorkItemLoadingScript):
        (WorkItemNonLoadingScript::WorkItemNonLoadingScript):
        (WorkItemLoad::WorkItemLoad):
        (WorkItemLoadHTMLString::WorkItemLoadHTMLString):
        (LayoutTestController::pathToLocalResource):
        * DumpRenderTree/chromium/LayoutTestController.h:
        (LayoutTestController::WorkItem::~WorkItem):
        (LayoutTestController::WorkQueue::WorkQueue):
        * DumpRenderTree/chromium/MockSpellCheck.cpp:
        (MockSpellCheck::MockSpellCheck):
        (MockSpellCheck::~MockSpellCheck):
        * DumpRenderTree/chromium/MockSpellCheck.h:
        * DumpRenderTree/chromium/NotificationPresenter.h:
        (NotificationPresenter::NotificationPresenter):
        * DumpRenderTree/chromium/Task.cpp:
        (WebTask::WebTask):
        * DumpRenderTree/chromium/Task.h:
        (TaskList::TaskList):
        * DumpRenderTree/chromium/TestEventPrinter.cpp:
        (DRTPrinter::DRTPrinter):
        (TestShellPrinter::TestShellPrinter):
        * DumpRenderTree/chromium/TestNavigationController.cpp:
        (TestNavigationEntry::TestNavigationEntry):
        (TestNavigationEntry::~TestNavigationEntry):
        (TestNavigationController::TestNavigationController):
        (TestNavigationController::reload):
        (TestNavigationController::loadEntry):
        (TestNavigationController::didNavigateToEntry):
        * DumpRenderTree/chromium/TestNavigationController.h:
        (TestShellExtraData::TestShellExtraData):
        * DumpRenderTree/chromium/TestShell.cpp:
        (dumpHistoryItem):
        * DumpRenderTree/chromium/TestShell.h:
        (TestParams::TestParams):
        * DumpRenderTree/chromium/TestShellWin.cpp:
        (TestShell::waitTestFinished):
        * DumpRenderTree/chromium/TestWebWorker.h:
        (TestWebWorker::startWorkerContext):
        (TestWebWorker::terminateWorkerContext):
        (TestWebWorker::postMessageToWorkerContext):
        (TestWebWorker::clientDestroyed):
        (TestWebWorker::postMessageToWorkerObject):
        (TestWebWorker::postExceptionToWorkerObject):
        (TestWebWorker::postConsoleMessageToWorkerObject):
        (TestWebWorker::confirmMessageFromWorkerObject):
        (TestWebWorker::reportPendingActivity):
        (TestWebWorker::workerContextClosed):
        (TestWebWorker::~TestWebWorker):
        * DumpRenderTree/chromium/TextInputController.cpp:
        * DumpRenderTree/chromium/WebThemeControlDRTWin.h:
        * DumpRenderTree/chromium/WebThemeEngineDRTMac.h:
        * DumpRenderTree/chromium/WebThemeEngineDRTWin.cpp:
        (WebThemeEngineDRTWin::paintButton):
        (WebThemeEngineDRTWin::paintMenuList):
        (WebThemeEngineDRTWin::paintTrackbar):
        * DumpRenderTree/chromium/WebThemeEngineDRTWin.h:
        (WebThemeEngineDRTWin::WebThemeEngineDRTWin):
        * DumpRenderTree/chromium/WebViewHost.cpp:
        (WebViewHost::navigate):
        (WebViewHost::updateSessionHistory):
        (WebViewHost::paintInvalidatedRegion):
        * DumpRenderTree/chromium/WebViewHost.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@86930 268f45cc-cd09-0410-ab3c-d52691b4dbfc

32 files changed:
Tools/ChangeLog
Tools/DumpRenderTree/chromium/CppBoundClass.cpp
Tools/DumpRenderTree/chromium/CppBoundClass.h
Tools/DumpRenderTree/chromium/CppVariant.h
Tools/DumpRenderTree/chromium/DRTDevToolsAgent.h
Tools/DumpRenderTree/chromium/DRTDevToolsClient.cpp
Tools/DumpRenderTree/chromium/DRTDevToolsClient.h
Tools/DumpRenderTree/chromium/DumpRenderTree.cpp
Tools/DumpRenderTree/chromium/EventSender.cpp
Tools/DumpRenderTree/chromium/EventSender.h
Tools/DumpRenderTree/chromium/ImageDiff.cpp
Tools/DumpRenderTree/chromium/LayoutTestController.cpp
Tools/DumpRenderTree/chromium/LayoutTestController.h
Tools/DumpRenderTree/chromium/MockSpellCheck.cpp
Tools/DumpRenderTree/chromium/MockSpellCheck.h
Tools/DumpRenderTree/chromium/NotificationPresenter.h
Tools/DumpRenderTree/chromium/Task.cpp
Tools/DumpRenderTree/chromium/Task.h
Tools/DumpRenderTree/chromium/TestEventPrinter.cpp
Tools/DumpRenderTree/chromium/TestNavigationController.cpp
Tools/DumpRenderTree/chromium/TestNavigationController.h
Tools/DumpRenderTree/chromium/TestShell.cpp
Tools/DumpRenderTree/chromium/TestShell.h
Tools/DumpRenderTree/chromium/TestShellWin.cpp
Tools/DumpRenderTree/chromium/TestWebWorker.h
Tools/DumpRenderTree/chromium/TextInputController.cpp
Tools/DumpRenderTree/chromium/WebThemeControlDRTWin.h
Tools/DumpRenderTree/chromium/WebThemeEngineDRTMac.h
Tools/DumpRenderTree/chromium/WebThemeEngineDRTWin.cpp
Tools/DumpRenderTree/chromium/WebThemeEngineDRTWin.h
Tools/DumpRenderTree/chromium/WebViewHost.cpp
Tools/DumpRenderTree/chromium/WebViewHost.h

index 529a221..806c583 100644 (file)
@@ -1,3 +1,103 @@
+2011-05-20  Kent Tamura  <tkent@chromium.org>
+
+        Reviewed by Ryosuke Niwa.
+
+        Fix style errors in DumpRenderTree/chromium/.
+        https://bugs.webkit.org/show_bug.cgi?id=61172
+
+        * DumpRenderTree/chromium/CppBoundClass.cpp:
+        (CppBoundClass::getAsCppVariant):
+        (CppBoundClass::bindToJavascript):
+        * DumpRenderTree/chromium/CppBoundClass.h:
+        (CppBoundClass::GetterCallback::~GetterCallback):
+        (CppBoundClass::CppBoundClass):
+        (CppBoundClass::Callback::~Callback):
+        (CppBoundClass::MemberCallback::MemberCallback):
+        (CppBoundClass::MemberCallback::~MemberCallback):
+        (CppBoundClass::MemberGetterCallback::MemberGetterCallback):
+        (CppBoundClass::MemberGetterCallback::~MemberGetterCallback):
+        * DumpRenderTree/chromium/CppVariant.h:
+        * DumpRenderTree/chromium/DRTDevToolsAgent.h:
+        (DRTDevToolsAgent::~DRTDevToolsAgent):
+        * DumpRenderTree/chromium/DRTDevToolsClient.cpp:
+        (DRTDevToolsClient::sendFrontendLoaded):
+        * DumpRenderTree/chromium/DRTDevToolsClient.h:
+        * DumpRenderTree/chromium/DumpRenderTree.cpp:
+        (main):
+        * DumpRenderTree/chromium/EventSender.cpp:
+        (SavedEvent::SavedEvent):
+        (EventSender::EventSender):
+        (EventSender::keyDown):
+        * DumpRenderTree/chromium/EventSender.h:
+        * DumpRenderTree/chromium/ImageDiff.cpp:
+        (Image::Image):
+        * DumpRenderTree/chromium/LayoutTestController.cpp:
+        (LayoutTestController::LayoutTestController):
+        (LayoutTestController::WorkQueue::reset):
+        (WorkItemBackForward::WorkItemBackForward):
+        (WorkItemLoadingScript::WorkItemLoadingScript):
+        (WorkItemNonLoadingScript::WorkItemNonLoadingScript):
+        (WorkItemLoad::WorkItemLoad):
+        (WorkItemLoadHTMLString::WorkItemLoadHTMLString):
+        (LayoutTestController::pathToLocalResource):
+        * DumpRenderTree/chromium/LayoutTestController.h:
+        (LayoutTestController::WorkItem::~WorkItem):
+        (LayoutTestController::WorkQueue::WorkQueue):
+        * DumpRenderTree/chromium/MockSpellCheck.cpp:
+        (MockSpellCheck::MockSpellCheck):
+        (MockSpellCheck::~MockSpellCheck):
+        * DumpRenderTree/chromium/MockSpellCheck.h:
+        * DumpRenderTree/chromium/NotificationPresenter.h:
+        (NotificationPresenter::NotificationPresenter):
+        * DumpRenderTree/chromium/Task.cpp:
+        (WebTask::WebTask):
+        * DumpRenderTree/chromium/Task.h:
+        (TaskList::TaskList):
+        * DumpRenderTree/chromium/TestEventPrinter.cpp:
+        (DRTPrinter::DRTPrinter):
+        (TestShellPrinter::TestShellPrinter):
+        * DumpRenderTree/chromium/TestNavigationController.cpp:
+        (TestNavigationEntry::TestNavigationEntry):
+        (TestNavigationEntry::~TestNavigationEntry):
+        (TestNavigationController::TestNavigationController):
+        (TestNavigationController::reload):
+        (TestNavigationController::loadEntry):
+        (TestNavigationController::didNavigateToEntry):
+        * DumpRenderTree/chromium/TestNavigationController.h:
+        (TestShellExtraData::TestShellExtraData):
+        * DumpRenderTree/chromium/TestShell.cpp:
+        (dumpHistoryItem):
+        * DumpRenderTree/chromium/TestShell.h:
+        (TestParams::TestParams):
+        * DumpRenderTree/chromium/TestShellWin.cpp:
+        (TestShell::waitTestFinished):
+        * DumpRenderTree/chromium/TestWebWorker.h:
+        (TestWebWorker::startWorkerContext):
+        (TestWebWorker::terminateWorkerContext):
+        (TestWebWorker::postMessageToWorkerContext):
+        (TestWebWorker::clientDestroyed):
+        (TestWebWorker::postMessageToWorkerObject):
+        (TestWebWorker::postExceptionToWorkerObject):
+        (TestWebWorker::postConsoleMessageToWorkerObject):
+        (TestWebWorker::confirmMessageFromWorkerObject):
+        (TestWebWorker::reportPendingActivity):
+        (TestWebWorker::workerContextClosed):
+        (TestWebWorker::~TestWebWorker):
+        * DumpRenderTree/chromium/TextInputController.cpp:
+        * DumpRenderTree/chromium/WebThemeControlDRTWin.h:
+        * DumpRenderTree/chromium/WebThemeEngineDRTMac.h:
+        * DumpRenderTree/chromium/WebThemeEngineDRTWin.cpp:
+        (WebThemeEngineDRTWin::paintButton):
+        (WebThemeEngineDRTWin::paintMenuList):
+        (WebThemeEngineDRTWin::paintTrackbar):
+        * DumpRenderTree/chromium/WebThemeEngineDRTWin.h:
+        (WebThemeEngineDRTWin::WebThemeEngineDRTWin):
+        * DumpRenderTree/chromium/WebViewHost.cpp:
+        (WebViewHost::navigate):
+        (WebViewHost::updateSessionHistory):
+        (WebViewHost::paintInvalidatedRegion):
+        * DumpRenderTree/chromium/WebViewHost.h:
+
 2011-05-19  Dmitry Lomov  <dslomov@google.com>
 
         Reviewed by Adam Roben.
index 7e24bd3..e68f100 100644 (file)
@@ -90,7 +90,7 @@ private:
     OwnPtr<CppBoundClass::GetterCallback> m_callback;
 };
 
-// Our special NPObject type.  We extend an NPObject with a pointer to a
+// Our special NPObject type. We extend an NPObject with a pointer to a
 // CppBoundClass, which is just a C++ interface that we forward all NPObject
 // callbacks to.
 struct CppNPObject {
@@ -113,27 +113,27 @@ struct CppNPObject {
     static void deallocate(NPObject*);
 
     // Returns true if the C++ class associated with this NPObject exposes the
-    // given property.  Called by the JS runtime.
+    // given property. Called by the JS runtime.
     static bool hasProperty(NPObject*, NPIdentifier);
 
     // Returns true if the C++ class associated with this NPObject exposes the
-    // given method.  Called by the JS runtime.
+    // given method. Called by the JS runtime.
     static bool hasMethod(NPObject*, NPIdentifier);
 
     // If the given method is exposed by the C++ class associated with this
-    // NPObject, invokes it with the given arguments and returns a result.  Otherwise,
-    // returns "undefined" (in the JavaScript sense).  Called by the JS runtime.
+    // NPObject, invokes it with the given arguments and returns a result. Otherwise,
+    // returns "undefined" (in the JavaScript sense). Called by the JS runtime.
     static bool invoke(NPObject*, NPIdentifier,
                        const NPVariant* arguments, uint32_t argumentCount,
                        NPVariant* result);
 
     // If the given property is exposed by the C++ class associated with this
-    // NPObject, returns its value.  Otherwise, returns "undefined" (in the
-    // JavaScript sense).  Called by the JS runtime.
+    // NPObject, returns its value. Otherwise, returns "undefined" (in the
+    // JavaScript sense). Called by the JS runtime.
     static bool getProperty(NPObject*, NPIdentifier, NPVariant* result);
 
     // If the given property is exposed by the C++ class associated with this
-    // NPObject, sets its value.  Otherwise, does nothing. Called by the JS
+    // NPObject, sets its value. Otherwise, does nothing. Called by the JS
     // runtime.
     static bool setProperty(NPObject*, NPIdentifier, const NPVariant* value);
 };
@@ -329,7 +329,7 @@ bool CppBoundClass::isMethodRegistered(const string& name) const
 CppVariant* CppBoundClass::getAsCppVariant()
 {
     if (!m_selfVariant.isObject()) {
-        // Create an NPObject using our static NPClass.  The first argument (a
+        // Create an NPObject using our static NPClass. The first argument (a
         // plugin's instance handle) is passed through to the allocate function
         // directly, and we don't use it, so it's ok to be 0.
         NPObject* npObj = WebBindings::createObject(0, &CppNPObject::npClass);
@@ -345,7 +345,7 @@ CppVariant* CppBoundClass::getAsCppVariant()
 void CppBoundClass::bindToJavascript(WebFrame* frame, const WebString& classname)
 {
     // BindToWindowObject will take its own reference to the NPObject, and clean
-    // up after itself.  It will also (indirectly) register the object with V8,
+    // up after itself. It will also (indirectly) register the object with V8,
     // so we must remember this so we can unregister it when we're destroyed.
     frame->bindToWindowObject(classname, NPVARIANT_TO_OBJECT(*getAsCppVariant()));
     m_boundToFrame = true;
index f7b198a..4fb5361 100644 (file)
@@ -77,14 +77,14 @@ public:
     // Callback class for "void function(CppVariant*)"
     class GetterCallback {
     public:
-        virtual ~GetterCallback() {}
+        virtual ~GetterCallback() { }
         virtual void run(CppVariant*) = 0;
     };
 
     // The constructor should call BindMethod, BindProperty, and
     // SetFallbackMethod as needed to set up the methods, properties, and
     // fallback method.
-    CppBoundClass() : m_boundToFrame(false) {}
+    CppBoundClass() : m_boundToFrame(false) { }
     virtual ~CppBoundClass();
 
     // Return a CppVariant representing this class, for use with BindProperty().
@@ -92,7 +92,7 @@ public:
     CppVariant* getAsCppVariant();
 
     // Given a WebFrame, BindToJavascript builds the NPObject that will represent
-    // the class and binds it to the frame's window under the given name.  This
+    // the class and binds it to the frame's window under the given name. This
     // should generally be called from the WebView delegate's
     // WindowObjectCleared(). A class so bound will be accessible to JavaScript
     // as window.<classname>. The owner of the CppBoundObject is responsible for
@@ -100,7 +100,7 @@ public:
     // afterwards.
     void bindToJavascript(WebKit::WebFrame*, const WebKit::WebString& classname);
 
-    // Used by a test.  Returns true if a method with name |name| exists,
+    // Used by a test. Returns true if a method with the specified name exists,
     // regardless of whether a fallback is registered.
     bool isMethodRegistered(const std::string&) const;
 
@@ -108,7 +108,7 @@ protected:
     // Callback for "void function(const CppArguemntList&, CppVariant*)"
     class Callback {
     public:
-        virtual ~Callback() {}
+        virtual ~Callback() { }
         virtual void run(const CppArgumentList&, CppVariant*) = 0;
     };
 
@@ -118,13 +118,14 @@ protected:
         typedef void (T::*MethodType)(const CppArgumentList&, CppVariant*);
         MemberCallback(T* object, MethodType method)
             : m_object(object)
-            , m_method(method) {}
-        virtual ~MemberCallback() {}
+            , m_method(method) { }
+        virtual ~MemberCallback() { }
 
         virtual void run(const CppArgumentList& arguments, CppVariant* result)
         {
             (m_object->*m_method)(arguments, result);
         }
+
     private:
         T* m_object;
         MethodType m_method;
@@ -136,10 +137,11 @@ protected:
         typedef void (T::*MethodType)(CppVariant*);
         MemberGetterCallback(T* object, MethodType method)
             : m_object(object)
-            , m_method(method) {}
-        virtual ~MemberGetterCallback() {}
+            , m_method(method) { }
+        virtual ~MemberGetterCallback() { }
 
         virtual void run(CppVariant* result) { (m_object->*m_method)(result); }
+
     private:
         T* m_object;
         MethodType m_method;
@@ -149,7 +151,7 @@ protected:
     void bindCallback(const std::string&, Callback*);
 
     // A wrapper for bindCallback, to simplify the common case of binding a
-    // method on the current object.  Though not verified here, |method|
+    // method on the current object. Though not verified here, the method parameter
     // must be a method of this CppBoundClass subclass.
     template<class T>
     void bindMethod(const std::string& name, void (T::*method)(const CppArgumentList&, CppVariant*))
@@ -163,7 +165,7 @@ protected:
     void bindGetterCallback(const std::string&, PassOwnPtr<GetterCallback>);
 
     // A wrapper for BindGetterCallback, to simplify the common case of binding a
-    // property on the current object.  Though not verified here, |method|
+    // property on the current object. Though not verified here, the method parameter
     // must be a method of this CppBoundClass subclass.
     template<class T>
     void bindProperty(const std::string& name, void (T::*method)(CppVariant*))
@@ -195,7 +197,7 @@ protected:
     }
 
     // A wrapper for BindFallbackCallback, to simplify the common case of
-    // binding a method on the current object.  Though not verified here,
+    // binding a method on the current object. Though not verified here,
     // |method| must be a method of this CppBoundClass subclass.
     // Passing 0 for |method| clears out any existing binding.
     template<class T>
@@ -231,7 +233,7 @@ private:
     bool getProperty(NPIdentifier, NPVariant* result) const;
     bool setProperty(NPIdentifier, const NPVariant*);
 
-    // A lazily-initialized CppVariant representing this class.  We retain 1
+    // A lazily-initialized CppVariant representing this class. We retain 1
     // reference to this object, and it is released on deletion.
     CppVariant m_selfVariant;
 
index 3032310..9d5edca 100644 (file)
@@ -58,7 +58,7 @@ public:
 
     // Note that setting a CppVariant to a string value involves copying the
     // string data, which must be freed with a call to freeData() when the
-    // CppVariant is set to a different value or is no longer needed.  Normally
+    // CppVariant is set to a different value or is no longer needed. Normally
     // this is handled by the other set() methods and by the destructor.
     void set(const char*); // Must be a null-terminated string.
     void set(const std::string&);
@@ -72,7 +72,7 @@ public:
     // the NP runtime nor CppVariant will ever free it.
     void set(NPObject*_value);
 
-    // These three methods all perform deep copies of any string data.  This
+    // These three methods all perform deep copies of any string data. This
     // allows local CppVariants to be released by the destructor without
     // corrupting their sources. In performance-critical code, or when strings
     // are very long, avoid creating new CppVariants.
@@ -89,8 +89,8 @@ public:
     // the ref-count (releases when ref-count becomes 0)
     void freeData();
 
-    // Compares this CppVariant's type and value to another's.  They must be
-    // identical in both type and value to be considered equal.  For string and
+    // Compares this CppVariant's type and value to another's. They must be
+    // identical in both type and value to be considered equal. For string and
     // object types, a deep comparison is performed; that is, the contents of the
     // strings, or the classes and refcounts of the objects, must be the same,
     // but they need not be the same pointers.
@@ -111,7 +111,7 @@ public:
     bool isEmpty() const { return (isVoid() || isNull()); }
     bool isObject() const { return (type == NPVariantType_Object); }
 
-    // Converters.  The CppVariant must be of a type convertible to these values.
+    // Converters. The CppVariant must be of a type convertible to these values.
     // For example, toInt32() works only if isNumber() is true.
     std::string toString() const;
     int32_t toInt32() const;
@@ -123,7 +123,7 @@ public:
 
     // Invoke method of the given name on an object with the supplied arguments.
     // The first argument should be the object on which the method is to be
-    // invoked.  Returns whether the method was successfully invoked.  If the
+    // invoked. Returns whether the method was successfully invoked. If the
     // method was invoked successfully, any return value is stored in the
     // CppVariant specified by result.
     bool invoke(const std::string&, const CppVariant* arguments,
index 0ecf61b..a973f81 100644 (file)
@@ -52,7 +52,7 @@ class DRTDevToolsAgent : public WebKit::WebDevToolsAgentClient {
     WTF_MAKE_NONCOPYABLE(DRTDevToolsAgent);
 public:
     DRTDevToolsAgent();
-    virtual ~DRTDevToolsAgent() {}
+    virtual ~DRTDevToolsAgent() { }
     void reset();
 
     void setWebView(WebKit::WebView*);
@@ -70,7 +70,7 @@ public:
     void frontendLoaded();
 
     bool evaluateInWebInspector(long callID, const std::string& script);
-    bool setTimelineProfilingEnabled(bool enable);
+    bool setTimelineProfilingEnabled(bool);
     TaskList* taskList() { return &m_taskList; }
 
 private:
@@ -82,14 +82,15 @@ private:
     class AsyncCallTask: public MethodTask<DRTDevToolsAgent> {
     public:
         AsyncCallTask(DRTDevToolsAgent* object, const WebKit::WebString& args)
-            : MethodTask<DRTDevToolsAgent>(object), m_args(args) {}
+            : MethodTask<DRTDevToolsAgent>(object), m_args(args) { }
         virtual void runIfValid() { m_object->call(m_args); }
+
     private:
         WebKit::WebString m_args;
     };
 
     struct DelayedFrontendLoadedTask: public MethodTask<DRTDevToolsAgent> {
-        DelayedFrontendLoadedTask(DRTDevToolsAgent* object) : MethodTask<DRTDevToolsAgent>(object) {}
+        DelayedFrontendLoadedTask(DRTDevToolsAgent* object) : MethodTask<DRTDevToolsAgent>(object) { }
         virtual void runIfValid() { m_object->delayedFrontendLoaded(); }
     };
 
index bf7aa3e..cfcd003 100644 (file)
@@ -64,7 +64,8 @@ void DRTDevToolsClient::reset()
     m_taskList.revokeAll();
 }
 
-void DRTDevToolsClient::sendFrontendLoaded() {
+void DRTDevToolsClient::sendFrontendLoaded()
+{
     if (m_drtDevToolsAgent)
         m_drtDevToolsAgent->frontendLoaded();
 }
index 8a93ee0..79f7d95 100644 (file)
@@ -73,8 +73,9 @@ public:
     class AsyncCallTask: public MethodTask<DRTDevToolsClient> {
     public:
         AsyncCallTask(DRTDevToolsClient* object, const WebKit::WebString& args)
-            : MethodTask<DRTDevToolsClient>(object), m_args(args) {}
+            : MethodTask<DRTDevToolsClient>(object), m_args(args) { }
         virtual void runIfValid() { m_object->call(m_args); }
+
     private:
         WebKit::WebString m_args;
     };
index 4c2e131..d348547 100644 (file)
@@ -259,8 +259,8 @@ int main(int argc, char* argv[])
         shell.callJSGC();
 
         // When we finish the last test, cleanup the LayoutTestController.
-        // It may have references to not-yet-cleaned up windows.  By
-        // cleaning up here we help purify reports.
+        // It may have references to not-yet-cleaned up windows. By cleaning up
+        // here we help purify reports.
         shell.resetTestController();
     }
 
index c800c43..9e55589 100644 (file)
 // This file contains the definition for EventSender.
 //
 // Some notes about drag and drop handling:
-// Windows drag and drop goes through a system call to doDragDrop.  At that
+// Windows drag and drop goes through a system call to doDragDrop. At that
 // point, program control is given to Windows which then periodically makes
-// callbacks into the webview.  This won't work for layout tests, so instead,
-// we queue up all the mouse move and mouse up events.  When the test tries to
+// callbacks into the webview. This won't work for layout tests, so instead,
+// we queue up all the mouse move and mouse up events. When the test tries to
 // start a drag (by calling EvenSendingController::doDragDrop), we take the
 // events in the queue and replay them.
 // The behavior of queuing events and replaying them can be disabled by a
@@ -84,7 +84,7 @@ struct SavedEvent {
     SavedEvent()
         : type(Unspecified)
         , buttonType(WebMouseEvent::ButtonNone)
-        , milliseconds(0) {}
+        , milliseconds(0) { }
 };
 
 static WebDragData currentDragData;
@@ -115,7 +115,7 @@ inline bool outsideMultiClickRadius(const WebPoint& a, const WebPoint& b)
         multipleClickRadiusPixels * multipleClickRadiusPixels;
 }
 
-// Used to offset the time the event hander things an event happened.  This is
+// Used to offset the time the event hander things an event happened. This is
 // done so tests can run without a delay, but bypass checks that are time
 // dependent (e.g., dragging has a timeout vs selection).
 static uint32 timeOffsetMs = 0;
@@ -248,7 +248,7 @@ EventSender::EventSender(TestShell* shell)
     : m_shell(shell)
 {
     // Initialize the map that associates methods of this class with the names
-    // they will use when called by JavaScript.  The actual binding of those
+    // they will use when called by JavaScript. The actual binding of those
     // names to their methods will be done by calling bindToJavaScript() (defined
     // by CppBoundClass, the parent to EventSender).
     bindMethod("addTouchPoint", &EventSender::addTouchPoint);
@@ -498,7 +498,7 @@ void EventSender::keyDown(const CppArgumentList& arguments, CppVariant* result)
     // FIXME: Should we also generate a KEY_UP?
     string codeStr = arguments[0].toString();
 
-    // Convert \n -> VK_RETURN.  Some layout tests use \n to mean "Enter", when
+    // Convert \n -> VK_RETURN. Some layout tests use \n to mean "Enter", when
     // Windows uses \r for "Enter".
     int code = 0;
     int text = 0;
@@ -795,8 +795,9 @@ void EventSender::contextClick(const CppArgumentList& arguments, CppVariant* res
 class MouseDownTask: public MethodTask<EventSender> {
 public:
     MouseDownTask(EventSender* obj, const CppArgumentList& arg)
-        : MethodTask<EventSender>(obj), m_arguments(arg) {}
+        : MethodTask<EventSender>(obj), m_arguments(arg) { }
     virtual void runIfValid() { m_object->mouseDown(m_arguments, 0); }
+
 private:
     CppArgumentList m_arguments;
 };
@@ -804,8 +805,9 @@ private:
 class MouseUpTask: public MethodTask<EventSender> {
 public:
     MouseUpTask(EventSender* obj, const CppArgumentList& arg)
-        : MethodTask<EventSender>(obj), m_arguments(arg) {}
+        : MethodTask<EventSender>(obj), m_arguments(arg) { }
     virtual void runIfValid() { m_object->mouseUp(m_arguments, 0); }
+
 private:
     CppArgumentList m_arguments;
 };
index 118509b..de1ca05 100644 (file)
@@ -118,7 +118,7 @@ private:
     bool isDragMode() { return dragMode.isBool() && dragMode.toBoolean(); }
 
     // Sometimes we queue up mouse move and mouse up events for drag drop
-    // handling purposes.  These methods dispatch the event.
+    // handling purposes. These methods dispatch the event.
     void doMouseMove(const WebKit::WebMouseEvent&);
     void doMouseUp(const WebKit::WebMouseEvent&);
     static void doLeapForward(int milliseconds);
@@ -145,7 +145,7 @@ private:
 
     TaskList m_taskList;
 
-    // Non-owning pointer.  The EventSender is owned by the TestShell.
+    // Non-owning pointer. The EventSender is owned by the TestShell.
     TestShell* m_shell;
 
     // Location of last mouseMoveTo event.
index d676430..20f9e93 100644 (file)
@@ -69,12 +69,12 @@ class Image {
 public:
     Image()
         : m_width(0)
-        , m_height(0) {}
+        , m_height(0) { }
 
     Image(const Image& image)
         : m_width(image.m_width)
         , m_height(image.m_height)
-        , m_data(image.m_data) {}
+        , m_data(image.m_data) { }
 
     bool hasImage() const { return m_width > 0 && m_height > 0; }
     int width() const { return m_width; }
index 9f7e949..33a5580 100644 (file)
@@ -78,7 +78,7 @@ LayoutTestController::LayoutTestController(TestShell* shell)
 {
 
     // Initialize the map that associates methods of this class with the names
-    // they will use when called by JavaScript.  The actual binding of those
+    // they will use when called by JavaScript. The actual binding of those
     // names to their methods will be done by calling bindToJavaScript() (defined
     // by CppBoundClass, the parent to LayoutTestController).
     bindMethod("addFileToPasteboardOnDrag", &LayoutTestController::addFileToPasteboardOnDrag);
@@ -265,9 +265,8 @@ void LayoutTestController::WorkQueue::processWork()
 void LayoutTestController::WorkQueue::reset()
 {
     m_frozen = false;
-    while (!m_queue.isEmpty()) {
+    while (!m_queue.isEmpty())
         delete m_queue.takeFirst();
-    }
 }
 
 void LayoutTestController::WorkQueue::addWork(WorkItem* work)
@@ -394,12 +393,13 @@ void LayoutTestController::completeNotifyDone(bool isTimeout)
 
 class WorkItemBackForward : public LayoutTestController::WorkItem {
 public:
-    WorkItemBackForward(int distance) : m_distance(distance) {}
+    WorkItemBackForward(int distance) : m_distance(distance) { }
     bool run(TestShell* shell)
     {
         shell->goToOffset(m_distance);
         return true; // FIXME: Did it really start a navigation?
     }
+
 private:
     int m_distance;
 };
@@ -435,24 +435,26 @@ void LayoutTestController::queueReload(const CppArgumentList&, CppVariant* resul
 
 class WorkItemLoadingScript : public LayoutTestController::WorkItem {
 public:
-    WorkItemLoadingScript(const string& script) : m_script(script) {}
+    WorkItemLoadingScript(const string& script) : m_script(script) { }
     bool run(TestShell* shell)
     {
         shell->webView()->mainFrame()->executeScript(WebScriptSource(WebString::fromUTF8(m_script)));
         return true; // FIXME: Did it really start a navigation?
     }
+
 private:
     string m_script;
 };
 
 class WorkItemNonLoadingScript : public LayoutTestController::WorkItem {
 public:
-    WorkItemNonLoadingScript(const string& script) : m_script(script) {}
+    WorkItemNonLoadingScript(const string& script) : m_script(script) { }
     bool run(TestShell* shell)
     {
         shell->webView()->mainFrame()->executeScript(WebScriptSource(WebString::fromUTF8(m_script)));
         return false;
     }
+
 private:
     string m_script;
 };
@@ -475,12 +477,13 @@ class WorkItemLoad : public LayoutTestController::WorkItem {
 public:
     WorkItemLoad(const WebURL& url, const WebString& target)
         : m_url(url)
-        , m_target(target) {}
+        , m_target(target) { }
     bool run(TestShell* shell)
     {
         shell->webViewHost()->loadURLForFrame(m_url, m_target);
         return true; // FIXME: Did it really start a navigation?
     }
+
 private:
     WebURL m_url;
     WebString m_target;
@@ -506,17 +509,18 @@ class WorkItemLoadHTMLString : public LayoutTestController::WorkItem  {
 public:
     WorkItemLoadHTMLString(const std::string& html, const WebURL& baseURL)
         : m_html(html)
-        , m_baseURL(baseURL) {}
+        , m_baseURL(baseURL) { }
     WorkItemLoadHTMLString(const std::string& html, const WebURL& baseURL, const WebURL& unreachableURL)
         : m_html(html)
         , m_baseURL(baseURL)
-        , m_unreachableURL(unreachableURL) {}
+        , m_unreachableURL(unreachableURL) { }
     bool run(TestShell* shell)
     {
         shell->webView()->mainFrame()->loadHTMLString(
             WebKit::WebData(m_html.data(), m_html.length()), m_baseURL, m_unreachableURL);
         return true;
     }
+
 private:
     std::string m_html;
     WebURL m_baseURL;
@@ -926,7 +930,7 @@ void LayoutTestController::pathToLocalResource(const CppArgumentList& arguments,
     }
 #endif
 
-    // Some layout tests use file://// which we resolve as a UNC path.  Normalize
+    // Some layout tests use file://// which we resolve as a UNC path. Normalize
     // them to just file:///.
     string lowerUrl = url;
     transform(lowerUrl.begin(), lowerUrl.end(), lowerUrl.begin(), ::tolower);
index 3c74549..c4e95a7 100644 (file)
@@ -72,33 +72,33 @@ public:
     void dumpAsText(const CppArgumentList&, CppVariant*);
 
     // This function should set a flag that tells the test_shell to print a line
-    // of descriptive text for each database command.  It should take no
+    // of descriptive text for each database command. It should take no
     // arguments, and ignore any that may be present. However, at the moment, we
     // don't have any DB function that prints messages, so for now this function
     // doesn't do anything.
     void dumpDatabaseCallbacks(const CppArgumentList&, CppVariant*);
 
     // This function sets a flag that tells the test_shell to print a line of
-    // descriptive text for each editing command.  It takes no arguments, and
+    // descriptive text for each editing command. It takes no arguments, and
     // ignores any that may be present.
     void dumpEditingCallbacks(const CppArgumentList&, CppVariant*);
 
     // This function sets a flag that tells the test_shell to print a line of
-    // descriptive text for each frame load callback.  It takes no arguments, and
+    // descriptive text for each frame load callback. It takes no arguments, and
     // ignores any that may be present.
     void dumpFrameLoadCallbacks(const CppArgumentList&, CppVariant*);
 
     // This function sets a flag that tells the test_shell to print a line of
-    // user gesture status text for some frame load callbacks.  It takes no
+    // user gesture status text for some frame load callbacks. It takes no
     // arguments, and ignores any that may be present.
     void dumpUserGestureInFrameLoadCallbacks(const CppArgumentList&, CppVariant*);
 
     // This function sets a flag that tells the test_shell to print out a text
-    // representation of the back/forward list.  It ignores all arguments.
+    // representation of the back/forward list. It ignores all arguments.
     void dumpBackForwardList(const CppArgumentList&, CppVariant*);
 
     // This function sets a flag that tells the test_shell to print out the
-    // scroll offsets of the child frames.  It ignores all.
+    // scroll offsets of the child frames. It ignores all.
     void dumpChildFrameScrollPositions(const CppArgumentList&, CppVariant*);
 
     // This function sets a flag that tells the test_shell to recursively
@@ -126,17 +126,17 @@ public:
     // attempt is made. It ignores any additional arguments.
     void setAcceptsEditing(const CppArgumentList&, CppVariant*);
 
-    // Functions for dealing with windows.  By default we block all new windows.
+    // Functions for dealing with windows. By default we block all new windows.
     void windowCount(const CppArgumentList&, CppVariant*);
     void setCanOpenWindows(const CppArgumentList&, CppVariant*);
     void setCloseRemainingWindowsWhenComplete(const CppArgumentList&, CppVariant*);
 
-    // By default, tests end when page load is complete.  These methods are used
+    // By default, tests end when page load is complete. These methods are used
     // to delay the completion of the test until notifyDone is called.
     void waitUntilDone(const CppArgumentList&, CppVariant*);
     void notifyDone(const CppArgumentList&, CppVariant*);
 
-    // Methods for adding actions to the work queue.  Used in conjunction with
+    // Methods for adding actions to the work queue. Used in conjunction with
     // waitUntilDone/notifyDone above.
     void queueBackNavigation(const CppArgumentList&, CppVariant*);
     void queueForwardNavigation(const CppArgumentList&, CppVariant*);
@@ -219,7 +219,7 @@ public:
     // the test.
     void setStopProvisionalFrameLoads(const CppArgumentList&, CppVariant*);
 
-    // Enable or disable smart insert/delete.  This is enabled by default.
+    // Enable or disable smart insert/delete. This is enabled by default.
     void setSmartInsertDeleteEnabled(const CppArgumentList&, CppVariant*);
 
     // Enable or disable trailing whitespace selection on double click.
@@ -245,8 +245,8 @@ public:
     void setDeferMainResourceDataLoad(const CppArgumentList&, CppVariant*);
     void setEditingBehavior(const CppArgumentList&, CppVariant*);
 
-    // The following are only stubs.  TODO(pamg): Implement any of these that
-    // are needed to pass the layout tests.
+    // The following are only stubs.
+    // FIXME: Implement any of these that are needed to pass the layout tests.
     void dumpAsWebArchive(const CppArgumentList&, CppVariant*);
     void dumpTitleChanges(const CppArgumentList&, CppVariant*);
     void setMainFrameIsFirstResponder(const CppArgumentList&, CppVariant*);
@@ -419,18 +419,17 @@ public:
     // waitForPolicyDelegate was called.
     void policyDelegateDone();
 
-    // Reinitializes all static values.  The reset() method should be called
-    // before the start of each test (currently from
-    // TestShell::runFileTest).
+    // Reinitializes all static values. The reset() method should be called
+    // before the start of each test (currently from TestShell::runFileTest).
     void reset();
 
     // A single item in the work queue.
     class WorkItem {
     public:
-        virtual ~WorkItem() {}
+        virtual ~WorkItem() { }
 
         // Returns true if this started a load.
-        virtual bool run(TestShell* shell) = 0;
+        virtual bool run(TestShell*) = 0;
     };
 
     TaskList* taskList() { return &m_taskList; }
@@ -443,14 +442,14 @@ private:
     // queueScript.
     class WorkQueue {
     public:
-        WorkQueue(LayoutTestController* controller) : m_frozen(false), m_controller(controller) {}
+        WorkQueue(LayoutTestController* controller) : m_frozen(false), m_controller(controller) { }
         virtual ~WorkQueue();
         void processWorkSoon();
 
         // Reset the state of the class between tests.
         void reset();
 
-        void addWork(WorkItem* work);
+        void addWork(WorkItem*);
 
         void setFrozen(bool frozen) { m_frozen = frozen; }
         bool isEmpty() { return m_queue.isEmpty(); }
@@ -460,7 +459,7 @@ private:
         void processWork();
         class WorkQueueTask: public MethodTask<WorkQueue> {
         public:
-            WorkQueueTask(WorkQueue* object): MethodTask<WorkQueue>(object) {}
+            WorkQueueTask(WorkQueue* object): MethodTask<WorkQueue>(object) { }
             virtual void runIfValid() { m_object->processWork(); }
         };
 
@@ -479,7 +478,7 @@ private:
     void completeNotifyDone(bool isTimeout);
     class NotifyDoneTimedOutTask: public MethodTask<LayoutTestController> {
     public:
-        NotifyDoneTimedOutTask(LayoutTestController* object): MethodTask<LayoutTestController>(object) {}
+        NotifyDoneTimedOutTask(LayoutTestController* object): MethodTask<LayoutTestController>(object) { }
         virtual void runIfValid() { m_object->completeNotifyDone(true); }
     };
 
@@ -494,7 +493,7 @@ private:
     // Used for test timeouts.
     TaskList m_taskList;
 
-    // Non-owning pointer.  The LayoutTestController is owned by the host.
+    // Non-owning pointer. The LayoutTestController is owned by the host.
     TestShell* m_shell;
 
     // If true, the test_shell will produce a plain text dump rather than a
@@ -546,18 +545,18 @@ private:
     // If true, the test_shell will generate pixel results in dumpAsText mode
     bool m_generatePixelResults;
 
-    // If true, the element will be treated as editable.  This value is returned
+    // If true, the element will be treated as editable. This value is returned
     // from various editing callbacks that are called just before edit operations
     // are allowed.
     bool m_acceptsEditing;
 
-    // If true, new windows can be opened via javascript or by plugins.  By
+    // If true, new windows can be opened via javascript or by plugins. By
     // default, set to false and can be toggled to true using
     // setCanOpenWindows().
     bool m_canOpenWindows;
 
     // When reset is called, go through and close all but the main test shell
-    // window.  By default, set to true but toggled to false using
+    // window. By default, set to true but toggled to false using
     // setCloseRemainingWindowsWhenComplete().
     bool m_closeRemainingWindows;
 
index 7243152..860176b 100644 (file)
@@ -39,9 +39,9 @@
 using namespace WebKit;
 
 MockSpellCheck::MockSpellCheck()
-    : m_initialized(false) {}
+    : m_initialized(false) { }
 
-MockSpellCheck::~MockSpellCheck() {}
+MockSpellCheck::~MockSpellCheck() { }
 
 static bool isNotASCIIAlpha(UChar ch) { return !isASCIIAlpha(ch); }
 
index dcd37f3..bce05c7 100644 (file)
@@ -63,6 +63,7 @@ public:
                         int* misspelledLength);
 
     void fillSuggestionList(const WebKit::WebString& word, Vector<WebKit::WebString>* suggestions);
+
 private:
     // Initialize the internal resources if we need to initialize it.
     // Initializing this object may take long time. To prevent from hurting
index 689a908..d476509 100644 (file)
@@ -43,7 +43,7 @@ class TestShell;
 // A class that implements WebNotificationPresenter for DRT.
 class NotificationPresenter : public WebKit::WebNotificationPresenter {
 public:
-    explicit NotificationPresenter(TestShell* shell) : m_shell(shell) {}
+    explicit NotificationPresenter(TestShell* shell) : m_shell(shell) { }
 
     // Called by the LayoutTestController to simulate a user granting permission.
     void grantPermission(const WebKit::WebString& origin);
index 8638915..1e7b938 100644 (file)
 #include "WebKit.h"
 #include "WebKitClient.h"
 
-WebTask::WebTask(TaskList* list): m_taskList(list) { m_taskList->registerTask(this); }
+WebTask::WebTask(TaskList* list)
+    : m_taskList(list)
+{
+    m_taskList->registerTask(this);
+}
+
 WebTask::~WebTask()
 {
     if (m_taskList)
index 57d27ea..0b32c47 100644 (file)
@@ -57,11 +57,12 @@ protected:
 
 class TaskList {
 public:
-    TaskList() {}
+    TaskList() { }
     ~TaskList() { revokeAll(); }
     void registerTask(WebTask* task) { m_tasks.append(task); }
-    void unregisterTask(WebTask* task);
+    void unregisterTask(WebTask*);
     void revokeAll();
+
 private:
     Vector<WebTask*> m_tasks;
 };
@@ -71,7 +72,7 @@ private:
 // Class T must have "TaskList* taskList()".
 template<class T> class MethodTask: public WebTask {
 public:
-    MethodTask(T* object): WebTask(object->taskList()), m_object(object) {}
+    MethodTask(T* object): WebTask(object->taskList()), m_object(object) { }
     virtual void run()
     {
         if (m_object)
@@ -84,11 +85,12 @@ public:
         m_taskList = 0;
     }
     virtual void runIfValid() = 0;
+
 protected:
     T* m_object;
 };
 
-void postTask(WebTask* task);
-void postDelayedTask(WebTask* task, int64_t ms);
+void postTask(WebTask*);
+void postDelayedTask(WebTask*, int64_t ms);
 
 #endif // Task_h
index c23e1d4..15de5e6 100644 (file)
@@ -37,7 +37,7 @@
 
 class DRTPrinter : public TestEventPrinter {
 public:
-    DRTPrinter() {}
+    DRTPrinter() { }
     void handleTestHeader(const char* url) const;
     void handleTimedOut() const;
     void handleTextHeader() const;
@@ -49,7 +49,7 @@ public:
 
 class TestShellPrinter : public TestEventPrinter {
 public:
-    TestShellPrinter() {}
+    TestShellPrinter() { }
     void handleTestHeader(const char* url) const;
     void handleTimedOut() const;
     void handleTextHeader() const;
index 9653c07..ad6fcff 100644 (file)
@@ -52,16 +52,16 @@ PassRefPtr<TestNavigationEntry> TestNavigationEntry::create(
 }
 
 TestNavigationEntry::TestNavigationEntry()
-    : m_pageID(-1) {}
+    : m_pageID(-1) { }
 
 TestNavigationEntry::TestNavigationEntry(
     int pageID, const WebURL& url, const WebString& title, const WebString& targetFrame)
     : m_pageID(pageID)
     , m_url(url)
     , m_title(title)
-    , m_targetFrame(targetFrame) {}
+    , m_targetFrame(targetFrame) { }
 
-TestNavigationEntry::~TestNavigationEntry() {}
+TestNavigationEntry::~TestNavigationEntry() { }
 
 void TestNavigationEntry::setContentState(const WebHistoryItem& state)
 {
@@ -76,7 +76,7 @@ TestNavigationController::TestNavigationController(NavigationHost* host)
     , m_lastCommittedEntryIndex(-1)
     , m_pendingEntryIndex(-1)
     , m_host(host)
-    , m_maxPageID(-1) {}
+    , m_maxPageID(-1) { }
 
 TestNavigationController::~TestNavigationController()
 {
@@ -96,8 +96,8 @@ void TestNavigationController::reload()
     // Base the navigation on where we are now...
     int currentIndex = currentEntryIndex();
 
-    // If we are no where, then we can't reload.  TODO(darin): We should add a
-    // CanReload method.
+    // If we are no where, then we can't reload.
+    // FIXME: We should add a CanReload method.
     if (currentIndex == -1)
         return;
 
@@ -130,7 +130,7 @@ void TestNavigationController::goToIndex(int index)
 void TestNavigationController::loadEntry(TestNavigationEntry* entry)
 {
     // When navigating to a new page, we don't know for sure if we will actually
-    // end up leaving the current page.  The new page load could for example
+    // end up leaving the current page. The new page load could for example
     // result in a download or a 'no content' response (e.g., a mailto: URL).
     discardPendingEntry();
     m_pendingEntry = entry;
@@ -185,7 +185,7 @@ void TestNavigationController::didNavigateToEntry(TestNavigationEntry* entry)
 
     // Otherwise, we just need to update an existing entry with matching PageID.
     // If the existing entry corresponds to the entry which is pending, then we
-    // must update the current entry index accordingly.  When navigating to the
+    // must update the current entry index accordingly. When navigating to the
     // same URL, a new PageID is not created.
 
     int existingEntryIndex = entryIndexWithPageID(entry->pageID());
index 8502a96..c4ba335 100644 (file)
@@ -46,7 +46,7 @@ class TestShellExtraData : public WebKit::WebDataSource::ExtraData {
 public:
     TestShellExtraData(int32_t pendingPageID)
         : pendingPageID(pendingPageID)
-        , requestCommitted(false) {}
+        , requestCommitted(false) { }
 
     // Contains the page_id for this navigation or -1 if there is none yet.
     int32_t pendingPageID;
@@ -142,7 +142,7 @@ public:
     int entryCount() const { return static_cast<int>(m_entries.size()); }
 
     // Returns the active entry, which is the pending entry if a navigation is in
-    // progress or the last committed entry otherwise.  NOTE: This can be 0!!
+    // progress or the last committed entry otherwise. NOTE: This can be 0!!
     //
     // If you are trying to get the current state of the NavigationControllerBase,
     // this is the method you will typically want to call.
@@ -153,8 +153,7 @@ public:
     // it is the m_pendingEntryIndex.
     int currentEntryIndex() const;
 
-    // Returns the entry at the specified index.  Returns 0 if out of
-    // bounds.
+    // Returns the entry at the specified index. Returns 0 if out of bounds.
     TestNavigationEntry* entryAtIndex(int) const;
 
     // Return the entry with the corresponding type and page ID, or 0 if
@@ -192,8 +191,8 @@ private:
     typedef NavigationEntryList::iterator NavigationEntryListIterator;
     NavigationEntryList m_entries;
 
-    // An entry we haven't gotten a response for yet.  This will be discarded
-    // when we navigate again.  It's used only so we know what the currently
+    // An entry we haven't gotten a response for yet. This will be discarded
+    // when we navigate again. It's used only so we know what the currently
     // displayed tab is.
     RefPtr<TestNavigationEntry> m_pendingEntry;
 
index 41afa10..e88f40f 100644 (file)
@@ -385,9 +385,8 @@ static string dumpHistoryItem(const WebHistoryItem& item, int indent, bool isCur
     if (isCurrent) {
         result.append("curr->");
         result.append(indent - 6, ' '); // 6 == "curr->".length()
-    } else {
+    } else
         result.append(indent, ' ');
-    }
 
     string url = item.urlString().utf8();
     size_t pos;
index 2a68b24..2db4c94 100644 (file)
@@ -75,7 +75,7 @@ struct TestParams {
         , dumpPixels(false)
         , debugRenderTree(false)
         , debugLayerTree(false)
-        , printSeparators(false) {}
+        , printSeparators(false) { }
 };
 
 class TestShell {
@@ -103,7 +103,7 @@ public:
     void waitTestFinished();
 
     // Operations to the main window.
-    void loadURL(const WebKit::WebURL& url);
+    void loadURL(const WebKit::WebURL&);
     void reload();
     void goToOffset(int offset);
     int navigationEntryCount() const;
@@ -132,7 +132,7 @@ public:
     void setLegacyAccelerated2dCanvasEnabled(bool enabled) { m_legacyAccelerated2dCanvasEnabled = enabled; }
     void setAcceleratedDrawingEnabled(bool enabled) { m_acceleratedDrawingEnabled = enabled; }
 #if defined(OS_WIN)
-    // Access to the finished event.  Used by the static WatchDog thread.
+    // Access to the finished event. Used by the static WatchDog thread.
     HANDLE finishedEvent() { return m_finishedEvent; }
 #endif
 
index f82771f..850e5de 100644 (file)
@@ -83,9 +83,9 @@ void TestShell::waitTestFinished()
     m_testIsPending = true;
 
     // Create a watchdog thread which just sets a timer and
-    // kills the process if it times out.  This catches really
+    // kills the process if it times out. This catches really
     // bad hangs where the shell isn't coming back to the
-    // message loop.  If the watchdog is what catches a
+    // message loop. If the watchdog is what catches a
     // timeout, it can't do anything except terminate the test
     // shell, which is unfortunate.
     m_finishedEvent = CreateEvent(0, TRUE, FALSE, 0);
index 89e19af..71bffce 100644 (file)
@@ -58,23 +58,23 @@ public:
     }
 
     // WebWorker methods:
-    virtual void startWorkerContext(const WebKit::WebURL&, const WebKit::WebString&, const WebKit::WebString&) {}
-    virtual void terminateWorkerContext() {}
-    virtual void postMessageToWorkerContext(const WebKit::WebString&, const WebKit::WebMessagePortChannelArray&) {}
+    virtual void startWorkerContext(const WebKit::WebURL&, const WebKit::WebString&, const WebKit::WebString&) { }
+    virtual void terminateWorkerContext() { }
+    virtual void postMessageToWorkerContext(const WebKit::WebString&, const WebKit::WebMessagePortChannelArray&) { }
     virtual void workerObjectDestroyed()
     {
         // Releases the reference held for worker object.
         deref();
     }
-    virtual void clientDestroyed() {}
+    virtual void clientDestroyed() { }
 
     // WebWorkerClient methods:
-    virtual void postMessageToWorkerObject(const WebKit::WebString&, const WebKit::WebMessagePortChannelArray&) {}
-    virtual void postExceptionToWorkerObject(const WebKit::WebString&, int, const WebKit::WebString&) {}
-    virtual void postConsoleMessageToWorkerObject(int, int, int, int, const WebKit::WebString&, int, const WebKit::WebString&) {}
-    virtual void confirmMessageFromWorkerObject(bool) {}
-    virtual void reportPendingActivity(bool) {}
-    virtual void workerContextClosed() {}
+    virtual void postMessageToWorkerObject(const WebKit::WebString&, const WebKit::WebMessagePortChannelArray&) { }
+    virtual void postExceptionToWorkerObject(const WebKit::WebString&, int, const WebKit::WebString&) { }
+    virtual void postConsoleMessageToWorkerObject(int, int, int, int, const WebKit::WebString&, int, const WebKit::WebString&) { }
+    virtual void confirmMessageFromWorkerObject(bool) { }
+    virtual void reportPendingActivity(bool) { }
+    virtual void workerContextClosed() { }
     virtual void workerContextDestroyed()
     {
         // Releases the reference held for worker context object.
@@ -86,7 +86,7 @@ public:
     virtual bool allowDatabase(WebKit::WebFrame*, const WebKit::WebString&, const WebKit::WebString&, unsigned long) { return true; }
 
 private:
-    ~TestWebWorker() {}
+    ~TestWebWorker() { }
     friend class WTF::RefCounted<TestWebWorker>;
 };
 
index 1d27dd6..22aa4c6 100644 (file)
@@ -39,8 +39,8 @@
 #include "WebString.h"
 #include "WebVector.h"
 #include "WebView.h"
-#include <wtf/StringExtras.h>
 #include <string>
+#include <wtf/StringExtras.h>
 
 using namespace WebKit;
 
index 6448dc6..397faf3 100644 (file)
@@ -132,7 +132,7 @@ public:
     // for drawing them is dependent on what WebKit tells us to do.
     // If drawEdges is true, draw an edge around the control. If
     // fillContentArea is true, fill the content area with the given color.
-    void drawTextField(bool drawEdges, bool fillContentArea, SkColor color);
+    void drawTextField(bool drawEdges, bool fillContentArea, SkColor);
 
     // Use this for drawing ProgressBar controls instead, since we
     // need to know the rect to fill inside the bar.
@@ -141,26 +141,26 @@ public:
 private:
     // Draws a box of size specified by irect, filled with the given color.
     // The box will have a border drawn in the default edge color.
-    void box(const SkIRect& irect, SkColor color);
+    void box(const SkIRect& irect, SkColor);
 
 
     // Draws a triangle of size specified by the three pairs of coordinates,
     // filled with the given color. The box will have an edge drawn in the
     // default edge color.
-    void triangle(int x0, int y0, int x1, int y1, int x2, int y2, SkColor color);
+    void triangle(int x0, int y0, int x1, int y1, int x2, int y2, SkColor);
 
     // Draws a rectangle the size of the control with rounded corners, filled
     // with the specified color (and with a border in the default edge color).
-    void roundRect(SkColor color);
+    void roundRect(SkColor);
 
     // Draws an oval the size of the control, filled with the specified color
     // and with a border in the default edge color.
-    void oval(SkColor color);
+    void oval(SkColor);
 
     // Draws a circle centered in the control with the specified radius,
     // filled with the specified color, and with a border draw in the
     // default edge color.
-    void circle(SkScalar radius, SkColor color);
+    void circle(SkScalar radius, SkColor);
 
     // Draws a box the size of the control, filled with the outerColor and
     // with a border in the default edge color, and then draws another box
@@ -174,7 +174,7 @@ private:
                      SkColor innerColor);
 
     // Draws a line between the two points in the given color.
-    void line(int x0, int y0, int x1, int y1, SkColor color);
+    void line(int x0, int y0, int x1, int y1, SkColor);
 
     // Draws a distinctive mark on the control for each state, so that the
     // state of the control can be determined without needing to know which
index 5b54394..877c081 100644 (file)
@@ -40,6 +40,7 @@ public:
         WebKit::WebThemeEngine::Size,
         const WebKit::WebRect&,
         const WebKit::WebThemeEngine::ScrollbarInfo&);
+
 private:
     virtual void paintHIThemeScrollbarThumb(
         WebKit::WebCanvas*,
index 103b295..1f8a422 100755 (executable)
@@ -267,9 +267,8 @@ void WebThemeEngineDRTWin::paintButton(WebCanvas* canvas,
             ASSERT_NOT_REACHED();
             break;
         }
-    } else {
+    } else
         ASSERT_NOT_REACHED();
-    }
 
     drawControl(canvas, rect, ctype, cstate);
 }
@@ -311,9 +310,8 @@ void WebThemeEngineDRTWin::paintMenuList(WebCanvas* canvas,
             CRASH();
             break;
         }
-    } else {
+    } else
         CRASH();
-    }
 
     drawControl(canvas, rect, ctype, cstate);
 }
@@ -736,9 +734,8 @@ void WebThemeEngineDRTWin::paintTrackbar(WebCanvas* canvas,
         ASSERT(part == TUS_NORMAL);
         ASSERT(classicState == dfcsNormal);
         cstate = WebThemeControlDRTWin::NormalState;
-    } else {
+    } else
         ASSERT_NOT_REACHED();
-    }
 
     drawControl(canvas, rect, ctype, cstate);
 }
index 9b1e817..69a4668 100644 (file)
@@ -53,7 +53,7 @@
 class WebThemeEngineDRTWin : public WebKit::WebThemeEngine {
     WTF_MAKE_NONCOPYABLE(WebThemeEngineDRTWin);
 public:
-    WebThemeEngineDRTWin() {}
+    WebThemeEngineDRTWin() { }
 
     // WebThemeEngine methods:
     virtual void paintButton(
index c01cdd6..7ef47b3 100644 (file)
@@ -124,7 +124,7 @@ static string descriptionSuitableForTestResult(const string& url)
     return url.substr(pos + 1);
 }
 
-// Adds a file called "DRTFakeFile" to |data_object| (CF_HDROP).  Use to fake
+// Adds a file called "DRTFakeFile" to dragData (CF_HDROP). Use to fake
 // dragging a file.
 static void addDRTFakeFileToDataObject(WebDragData* dragData)
 {
@@ -277,7 +277,7 @@ void WebViewHost::didStopLoading()
 }
 
 // The output from these methods in layout test mode should match that
-// expected by the layout tests.  See EditingDelegate.m in DumpRenderTree.
+// expected by the layout tests. See EditingDelegate.m in DumpRenderTree.
 
 bool WebViewHost::shouldBeginEditing(const WebRange& range)
 {
@@ -1283,7 +1283,7 @@ bool WebViewHost::navigate(const TestNavigationEntry& entry, bool reload)
     // back/forward navigations maintain the target frame?
 
     // A navigation resulting from loading a javascript URL should not be
-    // treated as a browser initiated event.  Instead, we want it to look as if
+    // treated as a browser initiated event. Instead, we want it to look as if
     // the page initiated any load resulting from JS execution.
     if (!GURL(entry.URL()).SchemeIs("javascript"))
         setPendingExtraData(adoptPtr(new TestShellExtraData(entry.pageID())));
@@ -1390,7 +1390,7 @@ void WebViewHost::updateURL(WebFrame* frame)
 void WebViewHost::updateSessionHistory(WebFrame* frame)
 {
     // If we have a valid page ID at this point, then it corresponds to the page
-    // we are navigating away from.  Otherwise, this is the first navigation, so
+    // we are navigating away from. Otherwise, this is the first navigation, so
     // there is no past session history to record.
     if (m_pageId == -1)
         return;
@@ -1492,7 +1492,7 @@ void WebViewHost::paintInvalidatedRegion()
     WebSize widgetSize = webWidget()->size();
     WebRect clientRect(0, 0, widgetSize.width, widgetSize.height);
 
-    // Paint the canvas if necessary.  Allow painting to generate extra rects
+    // Paint the canvas if necessary. Allow painting to generate extra rects
     // for the first two calls. This is necessary because some WebCore rendering
     // objects update their layout only when painted.
     // Store the total area painted in total_paint. Then tell the gdk window
index 67abb5b..92e7d86 100644 (file)
@@ -66,7 +66,7 @@ struct WebWindowFeatures;
 
 class WebViewHost : public WebKit::WebSpellCheckClient, public WebKit::WebViewClient, public WebKit::WebFrameClient, public NavigationHost {
  public:
-    WebViewHost(TestShell* shell);
+    WebViewHost(TestShell*);
     ~WebViewHost();
     void setWebWidget(WebKit::WebWidget*);
     WebKit::WebView* webView() const;
@@ -253,7 +253,7 @@ private:
     void setAddressBarURL(const WebKit::WebURL&);
 
     // In the Mac code, this is called to trigger the end of a test after the
-    // page has finished loading.  From here, we can generate the dump for the
+    // page has finished loading. From here, we can generate the dump for the
     // test.
     void locationChangeDone(WebKit::WebFrame*);
 
@@ -276,8 +276,8 @@ private:
     // don't actually want to open the mail program.
     bool m_policyDelegateEnabled;
 
-    // Toggles the behavior of the policy delegate.  If true, then navigations
-    // will be allowed.  Otherwise, they will be ignored (dropped).
+    // Toggles the behavior of the policy delegate. If true, then navigations
+    // will be allowed. Otherwise, they will be ignored (dropped).
     bool m_policyDelegateIsPermissive;
 
     // If true, the policy delegate will signal layout test completion.
@@ -292,7 +292,7 @@ private:
     // This is non-0 IFF a load is in progress.
     WebKit::WebFrame* m_topLoadingFrame;
 
-    // For tracking session history.  See RenderView.
+    // For tracking session history. See RenderView.
     int m_pageId;
     int m_lastPageIdUpdated;