Make the |url| parameter to OnBeforePopup() optional.

git-svn-id: https://chromiumembedded.googlecode.com/svn/trunk@434 5089003a-bbd8-11dd-ad1f-f1f9622dbc98
This commit is contained in:
Marshall Greenblatt 2011-12-20 22:07:45 +00:00
parent 6b195a8739
commit ec797bafe5
3 changed files with 3 additions and 9 deletions

View File

@ -1142,7 +1142,7 @@ public:
// modify the object that |client| points to. To change the settings for the
// new window modify the |settings| structure.
///
/*--cef()--*/
/*--cef(optional_param=url)--*/
virtual bool OnBeforePopup(CefRefPtr<CefBrowser> parentBrowser,
const CefPopupFeatures& popupFeatures,
CefWindowInfo& windowInfo,

View File

@ -40,10 +40,6 @@ int CEF_CALLBACK life_span_handler_on_before_popup(
DCHECK(windowInfo);
if (!windowInfo)
return 0;
// Verify param: url; type: string_byref_const
DCHECK(url);
if (!url)
return 0;
// Verify param: client; type: refptr_same_byref
DCHECK(client);
if (!client)
@ -52,6 +48,7 @@ int CEF_CALLBACK life_span_handler_on_before_popup(
DCHECK(settings);
if (!settings)
return 0;
// Unverified params: url
// Translate param: popupFeatures; type: struct_byref_const
CefPopupFeatures popupFeaturesObj;

View File

@ -31,10 +31,7 @@ bool CefLifeSpanHandlerCToCpp::OnBeforePopup(
DCHECK(parentBrowser.get());
if (!parentBrowser.get())
return false;
// Verify param: url; type: string_byref_const
DCHECK(!url.empty());
if (url.empty())
return false;
// Unverified params: url
// Translate param: client; type: refptr_same_byref
cef_client_t* clientStruct = NULL;