bcarrella/cef
1
0
Fork 0
mirror of https://bitbucket.org/chromiumembedded/cef synced 2025-02-16 20:20:51 +01:00
Code Issues Packages Projects Releases Wiki Activity

Add asserts to verify correct DOM object usage.

Browse Source 
git-svn-id: https://chromiumembedded.googlecode.com/svn/trunk@192 5089003a-bbd8-11dd-ad1f-f1f9622dbc98
This commit is contained in:
Marshall Greenblatt 2011-02-23 14:56:10 +00:00
parent 60736ac4e0
commit 6ca5cff06c
2 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
libcef/dom_document_impl.cc
View File

@ -255,6 +255,14 @@ void CefDOMDocumentImpl::Detach()
if (!VerifyContext()) if (!VerifyContext())
return; return;
// If you hit this assert it means that you are keeping references to node
// objects beyond the valid scope.
DCHECK(node_map_.empty());
// If you hit this assert it means that you are keeping references to this
// document object beyond the valid scope.
DCHECK(GetRefCt() == 1);
if (!node_map_.empty()) { if (!node_map_.empty()) {
NodeMap::const_iterator it = node_map_.begin(); NodeMap::const_iterator it = node_map_.begin();
for (; it != node_map_.end(); ++it) for (; it != node_map_.end(); ++it)

4
libcef/dom_event_impl.cc
View File

@ -147,6 +147,10 @@ CefRefPtr<CefDOMNode> CefDOMEventImpl::GetCurrentTarget()
void CefDOMEventImpl::Detach() void CefDOMEventImpl::Detach()
{ {
// If you hit this assert it means that you are keeping references to this
// event object beyond the valid scope.
DCHECK(GetRefCt() == 1);
document_ = NULL; document_ = NULL;
event_.assign(WebDOMEvent()); event_.assign(WebDOMEvent());
} }