summaryrefslogtreecommitdiffstats
path: root/webkit/tools
diff options
context:
space:
mode:
authorerg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2010-09-01 17:08:20 +0000
committererg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2010-09-01 17:08:20 +0000
commit601858c0f72ea9693a4796fe6b48558efde46ed7 (patch)
treef9102a4e748ce2f2257f86a748ba6ecfb7087141 /webkit/tools
parenta0709c0dafd07a463a1e9b3553554a0be6cec862 (diff)
downloadchromium_src-601858c0f72ea9693a4796fe6b48558efde46ed7.zip
chromium_src-601858c0f72ea9693a4796fe6b48558efde46ed7.tar.gz
chromium_src-601858c0f72ea9693a4796fe6b48558efde46ed7.tar.bz2
FBTF: Move more ctors/dtors from headers to implementation.
- Adding/Moving dtors of objects that have CancelableRequestConsumers shaves three megs off browser.a. - Adding/Moving dtors of objects that have ScopedRunnableMethodFactories only shaved 100k off browser.a/renderer.a. - Adding/Moving dtors of objects that used some form of base::*Timer<> was negligible; there were only a few classes that had a Timer but had a ctor/dtor in the header after the previous cleanups. BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3278006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58192 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/tools')
-rw-r--r--webkit/tools/test_shell/event_sending_controller.cc3
-rw-r--r--webkit/tools/test_shell/event_sending_controller.h1
-rw-r--r--webkit/tools/test_shell/layout_test_controller.cc6
-rw-r--r--webkit/tools/test_shell/layout_test_controller.h2
-rw-r--r--webkit/tools/test_shell/test_shell_devtools_agent.cc3
-rw-r--r--webkit/tools/test_shell/test_shell_devtools_agent.h2
6 files changed, 16 insertions, 1 deletions
diff --git a/webkit/tools/test_shell/event_sending_controller.cc b/webkit/tools/test_shell/event_sending_controller.cc
index 02d8078..29e7295 100644
--- a/webkit/tools/test_shell/event_sending_controller.cc
+++ b/webkit/tools/test_shell/event_sending_controller.cc
@@ -305,6 +305,9 @@ EventSendingController::EventSendingController(TestShell* shell)
#endif
}
+EventSendingController::~EventSendingController() {
+}
+
void EventSendingController::Reset() {
// The test should have finished a drag and the mouse button state.
DCHECK(current_drag_data.isNull());
diff --git a/webkit/tools/test_shell/event_sending_controller.h b/webkit/tools/test_shell/event_sending_controller.h
index e7db529..95ddc05 100644
--- a/webkit/tools/test_shell/event_sending_controller.h
+++ b/webkit/tools/test_shell/event_sending_controller.h
@@ -36,6 +36,7 @@ class EventSendingController : public CppBoundClass {
// Builds the property and method lists needed to bind this class to a JS
// object.
EventSendingController(TestShell* shell);
+ ~EventSendingController();
// Resets some static variable state.
void Reset();
diff --git a/webkit/tools/test_shell/layout_test_controller.cc b/webkit/tools/test_shell/layout_test_controller.cc
index 32f5188..e250886 100644
--- a/webkit/tools/test_shell/layout_test_controller.cc
+++ b/webkit/tools/test_shell/layout_test_controller.cc
@@ -199,6 +199,12 @@ LayoutTestController::LayoutTestController(TestShell* shell) :
BindProperty("webHistoryItemCount", &webHistoryItemCount_);
}
+LayoutTestController::~LayoutTestController() {
+}
+
+LayoutTestController::WorkQueue::WorkQueue() {
+}
+
LayoutTestController::WorkQueue::~WorkQueue() {
Reset();
}
diff --git a/webkit/tools/test_shell/layout_test_controller.h b/webkit/tools/test_shell/layout_test_controller.h
index 28ae5ea..d048d0f 100644
--- a/webkit/tools/test_shell/layout_test_controller.h
+++ b/webkit/tools/test_shell/layout_test_controller.h
@@ -26,6 +26,7 @@ class LayoutTestController : public CppBoundClass {
// Builds the property and method lists needed to bind this class to a JS
// object.
LayoutTestController(TestShell* shell);
+ ~LayoutTestController();
// This function sets a flag that tells the test_shell to dump pages as
// plain text, rather than as a text representation of the renderer's state.
@@ -352,6 +353,7 @@ class LayoutTestController : public CppBoundClass {
// queueScript.
class WorkQueue {
public:
+ WorkQueue();
virtual ~WorkQueue();
void ProcessWorkSoon();
diff --git a/webkit/tools/test_shell/test_shell_devtools_agent.cc b/webkit/tools/test_shell/test_shell_devtools_agent.cc
index 356f59c..d13cb9f 100644
--- a/webkit/tools/test_shell/test_shell_devtools_agent.cc
+++ b/webkit/tools/test_shell/test_shell_devtools_agent.cc
@@ -61,6 +61,9 @@ TestShellDevToolsAgent::TestShellDevToolsAgent()
&TestShellDevToolsAgent::DispatchMessageLoop);
}
+TestShellDevToolsAgent::~TestShellDevToolsAgent() {
+}
+
void TestShellDevToolsAgent::SetWebView(WebKit::WebView* web_view) {
web_view_ = web_view;
}
diff --git a/webkit/tools/test_shell/test_shell_devtools_agent.h b/webkit/tools/test_shell/test_shell_devtools_agent.h
index 8ea16de..e0f5276 100644
--- a/webkit/tools/test_shell/test_shell_devtools_agent.h
+++ b/webkit/tools/test_shell/test_shell_devtools_agent.h
@@ -24,7 +24,7 @@ class TestShellDevToolsAgent : public WebKit::WebDevToolsAgentClient {
public:
TestShellDevToolsAgent();
- virtual ~TestShellDevToolsAgent() {}
+ virtual ~TestShellDevToolsAgent();
void SetWebView(WebKit::WebView* web_view);