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

Unified Diff: include/AdblockPlus/Notification.h

Issue 5598762307158016: Issue 1550 - Get rid of V8ValueHolder.h (Closed)
Patch Set: update Created June 11, 2015, 1:02 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 | « include/AdblockPlus/JsValue.h ('k') | include/AdblockPlus/V8ValueHolder.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include/AdblockPlus/Notification.h
diff --git a/include/AdblockPlus/Notification.h b/include/AdblockPlus/Notification.h
index 57b1ff0e33e28fbf73b6d72b5e08c29a4ae19e18..f2779b29387ed34980f66e1e01d03e1befe7aeae 100644
--- a/include/AdblockPlus/Notification.h
+++ b/include/AdblockPlus/Notification.h
@@ -42,12 +42,12 @@ namespace AdblockPlus
{
friend class FilterEngine;
protected:
- static std::tr1::shared_ptr<Notification> JsValueToNotification(const JsValuePtr& jsValue);
+ static std::tr1::shared_ptr<Notification> JsValueToNotification(JsValue&& jsValue);
/**
* Constructor.
* @param jsValue `JsValuePtr` notification JavaScript object.
*/
- explicit Notification(const JsValuePtr& jsValue);
+ explicit Notification(JsValue&& jsValue);
public:
/**
* Retrieves the type of this notification.
« no previous file with comments | « include/AdblockPlus/JsValue.h ('k') | include/AdblockPlus/V8ValueHolder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld