Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: src/plugin/PluginFilter.cpp

Issue 11012013: Hanging resolved (critical sections unlocking) (Closed)
Patch Set: Created June 14, 2013, 12:17 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/plugin/AdblockPlusDomTraverser.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/plugin/PluginFilter.cpp
===================================================================
--- a/src/plugin/PluginFilter.cpp
+++ b/src/plugin/PluginFilter.cpp
@@ -286,7 +286,6 @@
}
}
-
// Check attributes
for (std::vector<CFilterElementHideAttrSelector>::const_iterator attrIt = m_attributeSelectors.begin();
attrIt != m_attributeSelectors.end(); ++ attrIt)
@@ -558,6 +557,7 @@
DEBUG_HIDE_EL(indent + "HideEl::Found (tag/id) filter:" + idIt->second.m_filterText)
CPluginDebug::DebugResultHiding(tag, "id:" + id, idIt->second.m_filterText);
#endif
+ s_criticalSectionFilterMap.Unlock();
return true;
}
}
@@ -572,6 +572,7 @@
DEBUG_HIDE_EL(indent + "HideEl::Found (?/id) filter:" + idIt->second.m_filterText)
CPluginDebug::DebugResultHiding(tag, "id:" + id, idIt->second.m_filterText);
#endif
+ s_criticalSectionFilterMap.Unlock();
return true;
}
}
@@ -595,6 +596,7 @@
DEBUG_HIDE_EL(indent + "HideEl::Found (tag/class) filter:" + classIt->second.m_filterText)
CPluginDebug::DebugResultHiding(tag, "class:" + className, classIt->second.m_filterText);
#endif
+ s_criticalSectionFilterMap.Unlock();
return true;
}
}
@@ -609,6 +611,7 @@
DEBUG_HIDE_EL(indent + "HideEl::Found (?/class) filter:" + classIt->second.m_filterText)
CPluginDebug::DebugResultHiding(tag, "class:" + className, classIt->second.m_filterText);
#endif
+ s_criticalSectionFilterMap.Unlock();
return true;
}
}
@@ -629,6 +632,7 @@
DEBUG_HIDE_EL(indent + "HideEl::Found (tag) filter:" + tagIt->second.m_filterText)
CPluginDebug::DebugResultHiding(tag, "-", tagIt->second.m_filterText);
#endif
+ s_criticalSectionFilterMap.Unlock();
return true;
}
}
« no previous file with comments | « src/plugin/AdblockPlusDomTraverser.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld