Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5672)

Unified Diff: chrome_frame/test/proxy_factory_mock.h

Issue 3061036: Handle automation server crashes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome_frame/test/mock_ie_event_sink_test.cc ('k') | chrome_frame/test/proxy_factory_mock.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_frame/test/proxy_factory_mock.h
===================================================================
--- chrome_frame/test/proxy_factory_mock.h (revision 55540)
+++ chrome_frame/test/proxy_factory_mock.h (working copy)
@@ -1,8 +1,8 @@
// Copyright (c) 2006-2010 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CHROME_FRAME_PROXY_FACTORY_MOCK_H_
-#define CHROME_FRAME_PROXY_FACTORY_MOCK_H_
+#ifndef CHROME_FRAME_TEST_PROXY_FACTORY_MOCK_H_
+#define CHROME_FRAME_TEST_PROXY_FACTORY_MOCK_H_
#include <windows.h>
#include <string>
@@ -10,16 +10,20 @@
#include "gmock/gmock.h"
#include "chrome_frame/chrome_frame_automation.h"
-struct LaunchDelegateMock : public ProxyFactory::LaunchDelegate {
+struct LaunchDelegateMock : public LaunchDelegate {
MOCK_METHOD2(LaunchComplete, void(ChromeFrameAutomationProxy*,
- AutomationLaunchResult));
+ AutomationLaunchResult));
+ MOCK_METHOD0(AutomationServerDied, void());
};
class MockProxyFactory : public ProxyFactory {
public:
- MOCK_METHOD3(GetAutomationServer, void (ProxyFactory::LaunchDelegate*,
- const ChromeFrameLaunchParams& params, void** automation_server_id));
- MOCK_METHOD1(ReleaseAutomationServer, bool(void* id));
+ MOCK_METHOD3(GetAutomationServer,
+ void (LaunchDelegate*, // NOLINT
+ ChromeFrameLaunchParams* params,
+ void** automation_server_id));
+ MOCK_METHOD2(ReleaseAutomationServer, bool(void* server_id,
+ LaunchDelegate* delegate));
MockProxyFactory() : thread_("mock factory worker") {
thread_.Start();
@@ -31,7 +35,7 @@
void* proxy_id,
AutomationLaunchResult result,
LaunchDelegate* d,
- const ChromeFrameLaunchParams& params,
+ ChromeFrameLaunchParams* params,
void** automation_server_id);
base::Thread thread_;
@@ -39,5 +43,5 @@
};
-#endif // CHROME_FRAME_PROXY_FACTORY_MOCK_H_
+#endif // CHROME_FRAME_TEST_PROXY_FACTORY_MOCK_H_
« no previous file with comments | « chrome_frame/test/mock_ie_event_sink_test.cc ('k') | chrome_frame/test/proxy_factory_mock.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698