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

Unified Diff: src/WebRequestJsObject.cpp

Issue 29393589: Issue 5013 - Make more methods const.- introduced JsConstValuePtr and JsConstValueList- JsValue:… (Closed) Base URL: https://hg.adblockplus.org/libadblockplus/
Patch Set: Rebased on master Created March 24, 2017, 2:40 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/Notification.cpp ('k') | test/JsValue.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/WebRequestJsObject.cpp
===================================================================
--- a/src/WebRequestJsObject.cpp
+++ b/src/WebRequestJsObject.cpp
@@ -76,17 +76,17 @@ namespace
AdblockPlus::JsValuePtr headersObject = jsEngine->NewObject();
for (const auto& header : result.responseHeaders)
{
headersObject->SetProperty(header.first, header.second);
}
resultObject->SetProperty("responseHeaders", headersObject);
- AdblockPlus::JsValueList params;
+ AdblockPlus::JsConstValueList params;
params.push_back(resultObject);
callback->Call(params);
}
private:
AdblockPlus::JsEnginePtr jsEngine;
std::string url;
AdblockPlus::HeaderList headers;
« no previous file with comments | « src/Notification.cpp ('k') | test/JsValue.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld