Unified Diff: src/Notification.cpp
Issue 29417605:
Issue 5034 - Part 3: Create plain JsValue instead of JsValuePtr (Closed)
Base URL: https://hg.adblockplus.org/libadblockplus/
Patch Set: FIxed JsContext bug and a few others.
Created April 19, 2017, 9:55 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« src/JsValue.cpp ('K')
|
« src/JsValue.cpp ('k')
|
src/WebRequestJsObject.cpp » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: src/Notification.cpp |
| =================================================================== |
| --- a/src/Notification.cpp |
| +++ b/src/Notification.cpp |
| @@ -90,17 +90,17 @@ |
| JsValue jsLinks = GetProperty("links"); |
| if (!jsLinks.IsArray()) |
| { |
| return retValue; |
| } |
| JsValueList urlLinksList = jsLinks.AsList(); |
| for (const auto& link : urlLinksList) |
| { |
| - retValue.push_back(link->AsString()); |
| + retValue.push_back(link.AsString()); |
| } |
| return retValue; |
| } |
| void Notification::MarkAsShown() |
| { |
| jsEngine->Evaluate("API.markNotificationAsShown")->Call(GetProperty("id")); |
| } |
« src/JsValue.cpp ('K')
|
« src/JsValue.cpp ('k')
|
src/WebRequestJsObject.cpp » ('j')
|
no next file with comments »
