Side by Side Diff: src/WebRequestJsObject.h
Issue 10173031:
Don`t use references to JsEngine to avoid use-after-free errors,switch to shared_ptr instead (Closed)
Patch Set:
Created April 18, 2013, 4:15 p.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« src/JsEngine.cpp ('K')
|
« src/JsValue.cpp ('k')
|
src/WebRequestJsObject.cpp » ('j')
|
no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| OLD | NEW |
|---|---|
| 1 #ifndef ADBLOCK_PLUS_WEB_REQUEST_JS_OBJECT_H | 1 #ifndef ADBLOCK_PLUS_WEB_REQUEST_JS_OBJECT_H |
| 2 #define ADBLOCK_PLUS_WEB_REQUEST_JS_OBJECT_H | 2 #define ADBLOCK_PLUS_WEB_REQUEST_JS_OBJECT_H |
| 3 | 3 |
| 4 #include <v8.h> | 4 #include <v8.h> |
| 5 #include <AdblockPlus/JsEngine.h> | |
| 5 | 6 |
| 6 namespace AdblockPlus | 7 namespace AdblockPlus |
| 7 { | 8 { |
| 8 class JsEngine; | 9 class JsEngine; |
| 9 | 10 |
| 10 namespace WebRequestJsObject | 11 namespace WebRequestJsObject |
| 11 { | 12 { |
| 12 JsValuePtr Setup(JsEngine& jsEngine, JsValuePtr obj); | 13 JsValuePtr Setup(JsEnginePtr jsEngine, JsValuePtr obj); |
| 13 } | 14 } |
| 14 } | 15 } |
| 15 | 16 |
| 16 #endif | 17 #endif |
| OLD | NEW |
« src/JsEngine.cpp ('K')
|
« src/JsValue.cpp ('k')
|
src/WebRequestJsObject.cpp » ('j')
|
no next file with comments »
