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

Unified Diff: src/plugin/PluginWbPassThrough.cpp

Issue 29330053: Issue 3303 - Unescape request URL for ShouldBlock in WBPassthruSink::BeginningTransaction (Closed)
Patch Set: Created Nov. 12, 2015, 10:51 a.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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/plugin/PluginWbPassThrough.cpp
diff --git a/src/plugin/PluginWbPassThrough.cpp b/src/plugin/PluginWbPassThrough.cpp
index b50c16a30ce86cf749d45debc8b175363c3869b3..45a3e8df8cbba35470ad1f04dbc426531f517f47 100644
--- a/src/plugin/PluginWbPassThrough.cpp
+++ b/src/plugin/PluginWbPassThrough.cpp
@@ -378,7 +378,7 @@ STDMETHODIMP WBPassthruSink::BeginningTransaction(LPCWSTR szURL, LPCWSTR szHeade
m_contentType = ContentType::CONTENT_TYPE_XMLHTTPREQUEST;
}
- if (client->ShouldBlock(szURL, m_contentType, m_boundDomain, /*debug flag but must be set*/true))
+ if (client->ShouldBlock(src.c_str(), m_contentType, m_boundDomain, /*debug flag but must be set*/true))
Eric 2015/11/14 18:11:25 The first argument for 'ShouldBlock' is "const std
sergei 2015/11/16 09:56:53 Good catch!
{
// NOTE: Feeding custom HTML to Flash, instead of original object subrequest
// doesn't have much sense. It also can manifest in unwanted result
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld