Restore CefRenderProcessHandler::OnBeforeNavigation (issue #1076). This method is still needed for some use cases due to issue #1129.
git-svn-id: https://chromiumembedded.googlecode.com/svn/trunk@1519 5089003a-bbd8-11dd-ad1f-f1f9622dbc98
This commit is contained in:
parent
f5bc72b234
commit
7428e45dd2
1
cef.gyp
1
cef.gyp
|
@ -1306,6 +1306,7 @@
|
|||
'tests/unittests/navigation_unittest.cc',
|
||||
'tests/unittests/process_message_unittest.cc',
|
||||
'tests/unittests/request_handler_unittest.cc',
|
||||
'tests/unittests/request_unittest.cc',
|
||||
'tests/unittests/scheme_handler_unittest.cc',
|
||||
'tests/unittests/urlrequest_unittest.cc',
|
||||
'tests/unittests/test_handler.cc',
|
||||
|
|
|
@ -94,6 +94,17 @@ typedef struct _cef_render_process_handler_t {
|
|||
struct _cef_load_handler_t* (CEF_CALLBACK *get_load_handler)(
|
||||
struct _cef_render_process_handler_t* self);
|
||||
|
||||
///
|
||||
// Called before browser navigation. Return true (1) to cancel the navigation
|
||||
// or false (0) to allow the navigation to proceed. The |request| object
|
||||
// cannot be modified in this callback.
|
||||
///
|
||||
int (CEF_CALLBACK *on_before_navigation)(
|
||||
struct _cef_render_process_handler_t* self,
|
||||
struct _cef_browser_t* browser, struct _cef_frame_t* frame,
|
||||
struct _cef_request_t* request,
|
||||
enum cef_navigation_type_t navigation_type, int is_redirect);
|
||||
|
||||
///
|
||||
// Called immediately after the V8 context for a frame has been created. To
|
||||
// retrieve the JavaScript 'window' object use the
|
||||
|
|
|
@ -55,6 +55,8 @@
|
|||
/*--cef(source=client)--*/
|
||||
class CefRenderProcessHandler : public virtual CefBase {
|
||||
public:
|
||||
typedef cef_navigation_type_t NavigationType;
|
||||
|
||||
///
|
||||
// Called after the render process main thread has been created. |extra_info|
|
||||
// is a read-only value originating from
|
||||
|
@ -92,6 +94,18 @@ class CefRenderProcessHandler : public virtual CefBase {
|
|||
return NULL;
|
||||
}
|
||||
|
||||
///
|
||||
// Called before browser navigation. Return true to cancel the navigation or
|
||||
// false to allow the navigation to proceed. The |request| object cannot be
|
||||
// modified in this callback.
|
||||
///
|
||||
/*--cef()--*/
|
||||
virtual bool OnBeforeNavigation(CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame,
|
||||
CefRefPtr<CefRequest> request,
|
||||
NavigationType navigation_type,
|
||||
bool is_redirect) { return false; }
|
||||
|
||||
///
|
||||
// Called immediately after the V8 context for a frame has been created. To
|
||||
// retrieve the JavaScript 'window' object use the CefV8Context::GetGlobal()
|
||||
|
|
|
@ -1479,6 +1479,18 @@ enum cef_focus_source_t {
|
|||
FOCUS_SOURCE_SYSTEM,
|
||||
};
|
||||
|
||||
///
|
||||
// Navigation types.
|
||||
///
|
||||
enum cef_navigation_type_t {
|
||||
NAVIGATION_LINK_CLICKED = 0,
|
||||
NAVIGATION_FORM_SUBMITTED,
|
||||
NAVIGATION_BACK_FORWARD,
|
||||
NAVIGATION_RELOAD,
|
||||
NAVIGATION_FORM_RESUBMITTED,
|
||||
NAVIGATION_OTHER,
|
||||
};
|
||||
|
||||
///
|
||||
// Supported XML encoding types. The parser supports ASCII, ISO-8859-1, and
|
||||
// UTF16 (LE and BE) by default. All other types must be translated to UTF8
|
||||
|
|
|
@ -20,6 +20,7 @@ MSVC_POP_WARNING();
|
|||
#include "libcef/common/cef_messages.h"
|
||||
#include "libcef/common/cef_switches.h"
|
||||
#include "libcef/common/content_client.h"
|
||||
#include "libcef/common/request_impl.h"
|
||||
#include "libcef/common/values_impl.h"
|
||||
#include "libcef/renderer/browser_impl.h"
|
||||
#include "libcef/renderer/chrome_bindings.h"
|
||||
|
@ -535,6 +536,62 @@ bool CefContentRendererClient::OverrideCreatePlugin(
|
|||
return false;
|
||||
}
|
||||
|
||||
bool CefContentRendererClient::HandleNavigation(
|
||||
blink::WebFrame* frame,
|
||||
const blink::WebURLRequest& request,
|
||||
blink::WebNavigationType type,
|
||||
blink::WebNavigationPolicy default_policy,
|
||||
bool is_redirect) {
|
||||
CefRefPtr<CefApp> application = CefContentClient::Get()->application();
|
||||
if (application.get()) {
|
||||
CefRefPtr<CefRenderProcessHandler> handler =
|
||||
application->GetRenderProcessHandler();
|
||||
if (handler.get()) {
|
||||
CefRefPtr<CefBrowserImpl> browserPtr =
|
||||
CefBrowserImpl::GetBrowserForMainFrame(frame->top());
|
||||
DCHECK(browserPtr.get());
|
||||
if (browserPtr.get()) {
|
||||
CefRefPtr<CefFrameImpl> framePtr = browserPtr->GetWebFrameImpl(frame);
|
||||
CefRefPtr<CefRequest> requestPtr(CefRequest::Create());
|
||||
CefRequestImpl* requestImpl =
|
||||
static_cast<CefRequestImpl*>(requestPtr.get());
|
||||
requestImpl->Set(request);
|
||||
requestImpl->SetReadOnly(true);
|
||||
|
||||
cef_navigation_type_t navigation_type = NAVIGATION_OTHER;
|
||||
switch (type) {
|
||||
case blink::WebNavigationTypeLinkClicked:
|
||||
navigation_type = NAVIGATION_LINK_CLICKED;
|
||||
break;
|
||||
case blink::WebNavigationTypeFormSubmitted:
|
||||
navigation_type = NAVIGATION_FORM_SUBMITTED;
|
||||
break;
|
||||
case blink::WebNavigationTypeBackForward:
|
||||
navigation_type = NAVIGATION_BACK_FORWARD;
|
||||
break;
|
||||
case blink::WebNavigationTypeReload:
|
||||
navigation_type = NAVIGATION_RELOAD;
|
||||
break;
|
||||
case blink::WebNavigationTypeFormResubmitted:
|
||||
navigation_type = NAVIGATION_FORM_RESUBMITTED;
|
||||
break;
|
||||
case blink::WebNavigationTypeOther:
|
||||
navigation_type = NAVIGATION_OTHER;
|
||||
break;
|
||||
}
|
||||
|
||||
if (handler->OnBeforeNavigation(browserPtr.get(), framePtr.get(),
|
||||
requestPtr.get(), navigation_type,
|
||||
is_redirect)) {
|
||||
return true;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
return false;
|
||||
}
|
||||
|
||||
void CefContentRendererClient::DidCreateScriptContext(
|
||||
blink::WebFrame* frame, v8::Handle<v8::Context> context,
|
||||
int extension_group, int world_id) {
|
||||
|
|
|
@ -81,6 +81,11 @@ class CefContentRendererClient : public content::ContentRendererClient,
|
|||
blink::WebFrame* frame,
|
||||
const blink::WebPluginParams& params,
|
||||
blink::WebPlugin** plugin) OVERRIDE;
|
||||
virtual bool HandleNavigation(blink::WebFrame* frame,
|
||||
const blink::WebURLRequest& request,
|
||||
blink::WebNavigationType type,
|
||||
blink::WebNavigationPolicy default_policy,
|
||||
bool is_redirect) OVERRIDE;
|
||||
virtual void DidCreateScriptContext(blink::WebFrame* frame,
|
||||
v8::Handle<v8::Context> context,
|
||||
int extension_group,
|
||||
|
|
|
@ -17,6 +17,7 @@
|
|||
#include "libcef_dll/ctocpp/frame_ctocpp.h"
|
||||
#include "libcef_dll/ctocpp/list_value_ctocpp.h"
|
||||
#include "libcef_dll/ctocpp/process_message_ctocpp.h"
|
||||
#include "libcef_dll/ctocpp/request_ctocpp.h"
|
||||
#include "libcef_dll/ctocpp/v8context_ctocpp.h"
|
||||
#include "libcef_dll/ctocpp/v8exception_ctocpp.h"
|
||||
#include "libcef_dll/ctocpp/v8stack_trace_ctocpp.h"
|
||||
|
@ -104,6 +105,40 @@ cef_load_handler_t* CEF_CALLBACK render_process_handler_get_load_handler(
|
|||
return CefLoadHandlerCppToC::Wrap(_retval);
|
||||
}
|
||||
|
||||
int CEF_CALLBACK render_process_handler_on_before_navigation(
|
||||
struct _cef_render_process_handler_t* self, cef_browser_t* browser,
|
||||
cef_frame_t* frame, struct _cef_request_t* request,
|
||||
enum cef_navigation_type_t navigation_type, int is_redirect) {
|
||||
// AUTO-GENERATED CONTENT - DELETE THIS COMMENT BEFORE MODIFYING
|
||||
|
||||
DCHECK(self);
|
||||
if (!self)
|
||||
return 0;
|
||||
// Verify param: browser; type: refptr_diff
|
||||
DCHECK(browser);
|
||||
if (!browser)
|
||||
return 0;
|
||||
// Verify param: frame; type: refptr_diff
|
||||
DCHECK(frame);
|
||||
if (!frame)
|
||||
return 0;
|
||||
// Verify param: request; type: refptr_diff
|
||||
DCHECK(request);
|
||||
if (!request)
|
||||
return 0;
|
||||
|
||||
// Execute
|
||||
bool _retval = CefRenderProcessHandlerCppToC::Get(self)->OnBeforeNavigation(
|
||||
CefBrowserCToCpp::Wrap(browser),
|
||||
CefFrameCToCpp::Wrap(frame),
|
||||
CefRequestCToCpp::Wrap(request),
|
||||
navigation_type,
|
||||
is_redirect?true:false);
|
||||
|
||||
// Return type: bool
|
||||
return _retval;
|
||||
}
|
||||
|
||||
void CEF_CALLBACK render_process_handler_on_context_created(
|
||||
struct _cef_render_process_handler_t* self, cef_browser_t* browser,
|
||||
cef_frame_t* frame, struct _cef_v8context_t* context) {
|
||||
|
@ -265,6 +300,8 @@ CefRenderProcessHandlerCppToC::CefRenderProcessHandlerCppToC(
|
|||
struct_.struct_.on_browser_destroyed =
|
||||
render_process_handler_on_browser_destroyed;
|
||||
struct_.struct_.get_load_handler = render_process_handler_get_load_handler;
|
||||
struct_.struct_.on_before_navigation =
|
||||
render_process_handler_on_before_navigation;
|
||||
struct_.struct_.on_context_created =
|
||||
render_process_handler_on_context_created;
|
||||
struct_.struct_.on_context_released =
|
||||
|
|
|
@ -15,6 +15,7 @@
|
|||
#include "libcef_dll/cpptoc/frame_cpptoc.h"
|
||||
#include "libcef_dll/cpptoc/list_value_cpptoc.h"
|
||||
#include "libcef_dll/cpptoc/process_message_cpptoc.h"
|
||||
#include "libcef_dll/cpptoc/request_cpptoc.h"
|
||||
#include "libcef_dll/cpptoc/v8context_cpptoc.h"
|
||||
#include "libcef_dll/cpptoc/v8exception_cpptoc.h"
|
||||
#include "libcef_dll/cpptoc/v8stack_trace_cpptoc.h"
|
||||
|
@ -98,6 +99,40 @@ CefRefPtr<CefLoadHandler> CefRenderProcessHandlerCToCpp::GetLoadHandler() {
|
|||
return CefLoadHandlerCToCpp::Wrap(_retval);
|
||||
}
|
||||
|
||||
bool CefRenderProcessHandlerCToCpp::OnBeforeNavigation(
|
||||
CefRefPtr<CefBrowser> browser, CefRefPtr<CefFrame> frame,
|
||||
CefRefPtr<CefRequest> request, NavigationType navigation_type,
|
||||
bool is_redirect) {
|
||||
if (CEF_MEMBER_MISSING(struct_, on_before_navigation))
|
||||
return false;
|
||||
|
||||
// AUTO-GENERATED CONTENT - DELETE THIS COMMENT BEFORE MODIFYING
|
||||
|
||||
// Verify param: browser; type: refptr_diff
|
||||
DCHECK(browser.get());
|
||||
if (!browser.get())
|
||||
return false;
|
||||
// Verify param: frame; type: refptr_diff
|
||||
DCHECK(frame.get());
|
||||
if (!frame.get())
|
||||
return false;
|
||||
// Verify param: request; type: refptr_diff
|
||||
DCHECK(request.get());
|
||||
if (!request.get())
|
||||
return false;
|
||||
|
||||
// Execute
|
||||
int _retval = struct_->on_before_navigation(struct_,
|
||||
CefBrowserCppToC::Wrap(browser),
|
||||
CefFrameCppToC::Wrap(frame),
|
||||
CefRequestCppToC::Wrap(request),
|
||||
navigation_type,
|
||||
is_redirect);
|
||||
|
||||
// Return type: bool
|
||||
return _retval?true:false;
|
||||
}
|
||||
|
||||
void CefRenderProcessHandlerCToCpp::OnContextCreated(
|
||||
CefRefPtr<CefBrowser> browser, CefRefPtr<CefFrame> frame,
|
||||
CefRefPtr<CefV8Context> context) {
|
||||
|
|
|
@ -40,6 +40,9 @@ class CefRenderProcessHandlerCToCpp
|
|||
virtual void OnBrowserCreated(CefRefPtr<CefBrowser> browser) OVERRIDE;
|
||||
virtual void OnBrowserDestroyed(CefRefPtr<CefBrowser> browser) OVERRIDE;
|
||||
virtual CefRefPtr<CefLoadHandler> GetLoadHandler() OVERRIDE;
|
||||
virtual bool OnBeforeNavigation(CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame, CefRefPtr<CefRequest> request,
|
||||
NavigationType navigation_type, bool is_redirect) OVERRIDE;
|
||||
virtual void OnContextCreated(CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame, CefRefPtr<CefV8Context> context) OVERRIDE;
|
||||
virtual void OnContextReleased(CefRefPtr<CefBrowser> browser,
|
||||
|
|
|
@ -289,6 +289,22 @@ CefRefPtr<CefLoadHandler> ClientApp::GetLoadHandler() {
|
|||
return load_handler;
|
||||
}
|
||||
|
||||
bool ClientApp::OnBeforeNavigation(CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame,
|
||||
CefRefPtr<CefRequest> request,
|
||||
NavigationType navigation_type,
|
||||
bool is_redirect) {
|
||||
RenderDelegateSet::iterator it = render_delegates_.begin();
|
||||
for (; it != render_delegates_.end(); ++it) {
|
||||
if ((*it)->OnBeforeNavigation(this, browser, frame, request,
|
||||
navigation_type, is_redirect)) {
|
||||
return true;
|
||||
}
|
||||
}
|
||||
|
||||
return false;
|
||||
}
|
||||
|
||||
void ClientApp::OnContextCreated(CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame,
|
||||
CefRefPtr<CefV8Context> context) {
|
||||
|
|
|
@ -55,6 +55,15 @@ class ClientApp : public CefApp,
|
|||
return NULL;
|
||||
}
|
||||
|
||||
virtual bool OnBeforeNavigation(CefRefPtr<ClientApp> app,
|
||||
CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame,
|
||||
CefRefPtr<CefRequest> request,
|
||||
cef_navigation_type_t navigation_type,
|
||||
bool is_redirect) {
|
||||
return false;
|
||||
}
|
||||
|
||||
virtual void OnContextCreated(CefRefPtr<ClientApp> app,
|
||||
CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame,
|
||||
|
@ -147,6 +156,11 @@ class ClientApp : public CefApp,
|
|||
virtual void OnBrowserCreated(CefRefPtr<CefBrowser> browser) OVERRIDE;
|
||||
virtual void OnBrowserDestroyed(CefRefPtr<CefBrowser> browser) OVERRIDE;
|
||||
virtual CefRefPtr<CefLoadHandler> GetLoadHandler() OVERRIDE;
|
||||
virtual bool OnBeforeNavigation(CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame,
|
||||
CefRefPtr<CefRequest> request,
|
||||
NavigationType navigation_type,
|
||||
bool is_redirect) OVERRIDE;
|
||||
virtual void OnContextCreated(CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame,
|
||||
CefRefPtr<CefV8Context> context) OVERRIDE;
|
||||
|
|
|
@ -42,6 +42,10 @@ void ClientApp::CreateRenderDelegates(RenderDelegateSet& delegates) {
|
|||
extern void CreateNavigationRendererTests(RenderDelegateSet& delegates);
|
||||
CreateNavigationRendererTests(delegates);
|
||||
|
||||
// Bring in the Request tests.
|
||||
extern void CreateRequestRendererTests(RenderDelegateSet& delegates);
|
||||
CreateRequestRendererTests(delegates);
|
||||
|
||||
// Bring in the RequestHandler tests.
|
||||
extern void CreateRequestHandlerRendererTests(RenderDelegateSet& delegates);
|
||||
CreateRequestHandlerRendererTests(delegates);
|
||||
|
|
|
@ -166,6 +166,40 @@ class HistoryNavRendererTest : public ClientApp::RenderDelegate,
|
|||
SendTestResultsIfDone(browser);
|
||||
}
|
||||
|
||||
virtual bool OnBeforeNavigation(CefRefPtr<ClientApp> app,
|
||||
CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame,
|
||||
CefRefPtr<CefRequest> request,
|
||||
cef_navigation_type_t navigation_type,
|
||||
bool is_redirect) OVERRIDE {
|
||||
if (!run_test_)
|
||||
return false;
|
||||
|
||||
const NavListItem& item = kHNavList[nav_];
|
||||
|
||||
std::string url = request->GetURL();
|
||||
EXPECT_STREQ(item.target, url.c_str());
|
||||
|
||||
EXPECT_EQ(RT_SUB_RESOURCE, request->GetResourceType());
|
||||
EXPECT_EQ(TT_EXPLICIT, request->GetTransitionType());
|
||||
|
||||
if (item.action == NA_LOAD)
|
||||
EXPECT_EQ(NAVIGATION_OTHER, navigation_type);
|
||||
else if (item.action == NA_BACK || item.action == NA_FORWARD)
|
||||
EXPECT_EQ(NAVIGATION_BACK_FORWARD, navigation_type);
|
||||
|
||||
if (nav_ > 0) {
|
||||
const NavListItem& last_item = kHNavList[nav_ - 1];
|
||||
EXPECT_EQ(last_item.can_go_back, browser->CanGoBack());
|
||||
EXPECT_EQ(last_item.can_go_forward, browser->CanGoForward());
|
||||
} else {
|
||||
EXPECT_FALSE(browser->CanGoBack());
|
||||
EXPECT_FALSE(browser->CanGoForward());
|
||||
}
|
||||
|
||||
return false;
|
||||
}
|
||||
|
||||
protected:
|
||||
void SendTestResultsIfDone(CefRefPtr<CefBrowser> browser) {
|
||||
if (got_load_end_ && got_loading_state_end_)
|
||||
|
|
|
@ -6,6 +6,7 @@
|
|||
|
||||
#include "include/cef_request.h"
|
||||
#include "include/cef_runnable.h"
|
||||
#include "tests/cefclient/client_app.h"
|
||||
#include "tests/unittests/test_handler.h"
|
||||
#include "tests/unittests/test_util.h"
|
||||
#include "testing/gtest/include/gtest/gtest.h"
|
||||
|
@ -200,46 +201,52 @@ TEST(RequestTest, SendRecv) {
|
|||
|
||||
namespace {
|
||||
|
||||
const char kTypeTestCompleteMsg[] = "RequestTest.Type";
|
||||
const char kTypeTestOrigin[] = "http://tests-requesttt.com/";
|
||||
|
||||
static struct TypeExpected {
|
||||
const char* file;
|
||||
bool browser_side; // True if this expectation applies to the browser side.
|
||||
bool navigation; // True if this expectation represents a navigation.
|
||||
cef_transition_type_t transition_type;
|
||||
cef_resource_type_t resource_type;
|
||||
int expected_count;
|
||||
} g_type_expected[] = {
|
||||
// Initial main frame load due to browser creation.
|
||||
{"main.html", true, TT_EXPLICIT, RT_MAIN_FRAME, 1},
|
||||
{"main.html", true, true, TT_EXPLICIT, RT_MAIN_FRAME, 1},
|
||||
{"main.html", false, true, TT_EXPLICIT, RT_SUB_RESOURCE, 1},
|
||||
|
||||
// Sub frame load.
|
||||
{"sub.html", true, TT_LINK, RT_SUB_FRAME, 1},
|
||||
{"sub.html", true, true, TT_LINK, RT_SUB_FRAME, 1},
|
||||
{"sub.html", false, true, TT_EXPLICIT, RT_SUB_RESOURCE, 1},
|
||||
|
||||
// Stylesheet load.
|
||||
{"style.css", false, TT_LINK, RT_STYLESHEET, 1},
|
||||
{"style.css", true, false, TT_LINK, RT_STYLESHEET, 1},
|
||||
|
||||
// Script load.
|
||||
{"script.js", false, TT_LINK, RT_SCRIPT, 1},
|
||||
{"script.js", true, false, TT_LINK, RT_SCRIPT, 1},
|
||||
|
||||
// Image load.
|
||||
{"image.png", false, TT_LINK, RT_IMAGE, 1},
|
||||
{"image.png", true, false, TT_LINK, RT_IMAGE, 1},
|
||||
|
||||
// Font load.
|
||||
{"font.ttf", false, TT_LINK, RT_FONT_RESOURCE, 1},
|
||||
{"font.ttf", true, false, TT_LINK, RT_FONT_RESOURCE, 1},
|
||||
|
||||
// XHR load.
|
||||
{"xhr.html", false, TT_LINK, RT_XHR, 1},
|
||||
{"xhr.html", true, false, TT_LINK, RT_XHR, 1},
|
||||
};
|
||||
|
||||
class TypeExpectations {
|
||||
public:
|
||||
explicit TypeExpectations(bool navigation)
|
||||
: navigation_(navigation) {
|
||||
TypeExpectations(bool browser_side, bool navigation)
|
||||
: browser_side_(browser_side),
|
||||
navigation_(navigation) {
|
||||
// Build the map of relevant requests.
|
||||
for (int i = 0;
|
||||
i < static_cast<int>(sizeof(g_type_expected) / sizeof(TypeExpected));
|
||||
++i) {
|
||||
if (navigation_ && g_type_expected[i].navigation != navigation_)
|
||||
if (g_type_expected[i].browser_side != browser_side_ ||
|
||||
(navigation_ && g_type_expected[i].navigation != navigation_))
|
||||
continue;
|
||||
|
||||
request_count_.insert(std::make_pair(i, 0));
|
||||
|
@ -260,6 +267,7 @@ class TypeExpectations {
|
|||
const int index = GetExpectedIndex(file, transition_type, resource_type);
|
||||
EXPECT_GE(index, 0)
|
||||
<< "File: " << file.c_str()
|
||||
<< "; Browser Side: " << browser_side_
|
||||
<< "; Navigation: " << navigation_
|
||||
<< "; Transition Type: " << transition_type
|
||||
<< "; Resource Type: " << resource_type;
|
||||
|
@ -271,6 +279,7 @@ class TypeExpectations {
|
|||
const int expected_count = g_type_expected[index].expected_count;
|
||||
EXPECT_LE(actual_count, expected_count)
|
||||
<< "File: " << file.c_str()
|
||||
<< "; Browser Side: " << browser_side_
|
||||
<< "; Navigation: " << navigation_
|
||||
<< "; Transition Type: " << transition_type
|
||||
<< "; Resource Type: " << resource_type;
|
||||
|
@ -283,7 +292,8 @@ class TypeExpectations {
|
|||
for (int i = 0;
|
||||
i < static_cast<int>(sizeof(g_type_expected) / sizeof(TypeExpected));
|
||||
++i) {
|
||||
if (navigation_ && g_type_expected[i].navigation != navigation_)
|
||||
if (g_type_expected[i].browser_side != browser_side_ ||
|
||||
(navigation_ && g_type_expected[i].navigation != navigation_))
|
||||
continue;
|
||||
|
||||
RequestCount::const_iterator it = request_count_.find(i);
|
||||
|
@ -292,6 +302,7 @@ class TypeExpectations {
|
|||
if (assert) {
|
||||
EXPECT_EQ(g_type_expected[i].expected_count, it->second)
|
||||
<< "File: " << g_type_expected[i].file
|
||||
<< "; Browser Side: " << browser_side_
|
||||
<< "; Navigation: " << navigation_
|
||||
<< "; Transition Type: " << g_type_expected[i].transition_type
|
||||
<< "; Resource Type: " << g_type_expected[i].resource_type;
|
||||
|
@ -311,6 +322,7 @@ class TypeExpectations {
|
|||
i < static_cast<int>(sizeof(g_type_expected) / sizeof(TypeExpected));
|
||||
++i) {
|
||||
if (g_type_expected[i].file == file &&
|
||||
g_type_expected[i].browser_side == browser_side_ &&
|
||||
(!navigation_ || g_type_expected[i].navigation == navigation_) &&
|
||||
g_type_expected[i].transition_type == transition_type &&
|
||||
g_type_expected[i].resource_type == resource_type) {
|
||||
|
@ -320,6 +332,7 @@ class TypeExpectations {
|
|||
return -1;
|
||||
}
|
||||
|
||||
bool browser_side_;
|
||||
bool navigation_;
|
||||
|
||||
// Map of TypeExpected index to actual request count.
|
||||
|
@ -327,14 +340,52 @@ class TypeExpectations {
|
|||
RequestCount request_count_;
|
||||
};
|
||||
|
||||
// Renderer side.
|
||||
class TypeRendererTest : public ClientApp::RenderDelegate {
|
||||
public:
|
||||
TypeRendererTest() :
|
||||
expectations_(false, true) {}
|
||||
|
||||
virtual bool OnBeforeNavigation(CefRefPtr<ClientApp> app,
|
||||
CefRefPtr<CefBrowser> browser,
|
||||
CefRefPtr<CefFrame> frame,
|
||||
CefRefPtr<CefRequest> request,
|
||||
cef_navigation_type_t navigation_type,
|
||||
bool is_redirect) OVERRIDE {
|
||||
if (expectations_.GotRequest(request) && expectations_.IsDone(false))
|
||||
SendTestResults(browser);
|
||||
return false;
|
||||
}
|
||||
|
||||
private:
|
||||
// Send the test results.
|
||||
void SendTestResults(CefRefPtr<CefBrowser> browser) {
|
||||
// Check if the test has failed.
|
||||
bool result = !TestFailed();
|
||||
|
||||
// Return the result to the browser process.
|
||||
CefRefPtr<CefProcessMessage> return_msg =
|
||||
CefProcessMessage::Create(kTypeTestCompleteMsg);
|
||||
CefRefPtr<CefListValue> args = return_msg->GetArgumentList();
|
||||
EXPECT_TRUE(args.get());
|
||||
EXPECT_TRUE(args->SetBool(0, result));
|
||||
EXPECT_TRUE(browser->SendProcessMessage(PID_BROWSER, return_msg));
|
||||
}
|
||||
|
||||
TypeExpectations expectations_;
|
||||
|
||||
IMPLEMENT_REFCOUNTING(TypeRendererTest);
|
||||
};
|
||||
|
||||
// Browser side.
|
||||
class TypeTestHandler : public TestHandler {
|
||||
public:
|
||||
TypeTestHandler() :
|
||||
browse_expectations_(true),
|
||||
load_expectations_(false),
|
||||
get_expectations_(false),
|
||||
browse_expectations_(true, true),
|
||||
load_expectations_(true, false),
|
||||
get_expectations_(true, false),
|
||||
completed_browser_side_(false),
|
||||
completed_render_side_(false),
|
||||
destroyed_(false) {}
|
||||
|
||||
virtual void RunTest() OVERRIDE {
|
||||
|
@ -411,13 +462,41 @@ class TypeTestHandler : public TestHandler {
|
|||
completed_browser_side_ = true;
|
||||
// Destroy the test on the UI thread.
|
||||
CefPostTask(TID_UI,
|
||||
NewCefRunnableMethod(this, &TypeTestHandler::DestroyTest));
|
||||
NewCefRunnableMethod(this, &TypeTestHandler::DestroyTestIfComplete));
|
||||
}
|
||||
|
||||
return TestHandler::GetResourceHandler(browser, frame, request);
|
||||
}
|
||||
|
||||
virtual bool OnProcessMessageReceived(
|
||||
CefRefPtr<CefBrowser> browser,
|
||||
CefProcessId source_process,
|
||||
CefRefPtr<CefProcessMessage> message) OVERRIDE {
|
||||
const std::string& msg_name = message->GetName();
|
||||
if (msg_name == kTypeTestCompleteMsg) {
|
||||
// Test that the renderer side succeeded.
|
||||
CefRefPtr<CefListValue> args = message->GetArgumentList();
|
||||
EXPECT_TRUE(args.get());
|
||||
EXPECT_TRUE(args->GetBool(0));
|
||||
|
||||
completed_render_side_ = true;
|
||||
DestroyTestIfComplete();
|
||||
return true;
|
||||
}
|
||||
|
||||
// Message not handled.
|
||||
return false;
|
||||
}
|
||||
|
||||
private:
|
||||
void DestroyTestIfComplete() {
|
||||
if (destroyed_)
|
||||
return;
|
||||
|
||||
if (completed_browser_side_ && completed_render_side_)
|
||||
DestroyTest();
|
||||
}
|
||||
|
||||
virtual void DestroyTest() OVERRIDE {
|
||||
if (destroyed_)
|
||||
return;
|
||||
|
@ -425,6 +504,7 @@ class TypeTestHandler : public TestHandler {
|
|||
|
||||
// Verify test expectations.
|
||||
EXPECT_TRUE(completed_browser_side_);
|
||||
EXPECT_TRUE(completed_render_side_);
|
||||
EXPECT_TRUE(browse_expectations_.IsDone(true));
|
||||
EXPECT_TRUE(load_expectations_.IsDone(true));
|
||||
EXPECT_TRUE(get_expectations_.IsDone(true));
|
||||
|
@ -437,6 +517,7 @@ class TypeTestHandler : public TestHandler {
|
|||
TypeExpectations get_expectations_;
|
||||
|
||||
bool completed_browser_side_;
|
||||
bool completed_render_side_;
|
||||
bool destroyed_;
|
||||
};
|
||||
|
||||
|
@ -448,3 +529,10 @@ TEST(RequestTest, ResourceAndTransitionType) {
|
|||
new TypeTestHandler();
|
||||
handler->ExecuteTest();
|
||||
}
|
||||
|
||||
|
||||
// Entry point for creating request renderer test objects.
|
||||
// Called from client_app_delegates.cc.
|
||||
void CreateRequestRendererTests(ClientApp::RenderDelegateSet& delegates) {
|
||||
delegates.insert(new TypeRendererTest);
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue