Left: | ||
Right: |
OLD | NEW |
---|---|
1 /* | 1 /* |
2 * This file is part of Adblock Plus <https://adblockplus.org/>, | 2 * This file is part of Adblock Plus <https://adblockplus.org/>, |
3 * Copyright (C) 2006-present eyeo GmbH | 3 * Copyright (C) 2006-present eyeo GmbH |
4 * | 4 * |
5 * Adblock Plus is free software: you can redistribute it and/or modify | 5 * Adblock Plus is free software: you can redistribute it and/or modify |
6 * it under the terms of the GNU General Public License version 3 as | 6 * it under the terms of the GNU General Public License version 3 as |
7 * published by the Free Software Foundation. | 7 * published by the Free Software Foundation. |
8 * | 8 * |
9 * Adblock Plus is distributed in the hope that it will be useful, | 9 * Adblock Plus is distributed in the hope that it will be useful, |
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of | 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of |
(...skipping 27 matching lines...) Expand all Loading... | |
38 { | 38 { |
39 this(ctor(logSystem, webRequest, basePath)); | 39 this(ctor(logSystem, webRequest, basePath)); |
40 } | 40 } |
41 | 41 |
42 protected Platform(final long ptr) | 42 protected Platform(final long ptr) |
43 { | 43 { |
44 this.ptr = ptr; | 44 this.ptr = ptr; |
45 this.disposer = new Disposer(this, new DisposeWrapper(ptr)); | 45 this.disposer = new Disposer(this, new DisposeWrapper(ptr)); |
46 } | 46 } |
47 | 47 |
48 public void setUpJsEngine(final AppInfo appInfo, final Long v8IsolatePtr) | |
sergei
2017/09/26 09:44:13
Why not to use primitive value here and in Adblock
anton
2017/09/26 10:55:06
There is almost no difference.
I will use primitiv
| |
49 { | |
50 setUpJsEngine(this.ptr, appInfo, (v8IsolatePtr != null ? v8IsolatePtr.longVa lue() : 0L)); | |
51 } | |
52 | |
48 public void setUpJsEngine(final AppInfo appInfo) | 53 public void setUpJsEngine(final AppInfo appInfo) |
49 { | 54 { |
50 setUpJsEngine(this.ptr, appInfo); | 55 setUpJsEngine(appInfo, null); |
51 } | 56 } |
52 | 57 |
53 public JsEngine getJsEngine() | 58 public JsEngine getJsEngine() |
54 { | 59 { |
55 return new JsEngine(getJsEnginePtr(this.ptr)); | 60 return new JsEngine(getJsEnginePtr(this.ptr)); |
56 } | 61 } |
57 | 62 |
58 public void setUpFilterEngine(final IsAllowedConnectionCallback isSubscription DownloadAllowedCallback) | 63 public void setUpFilterEngine(final IsAllowedConnectionCallback isSubscription DownloadAllowedCallback) |
59 { | 64 { |
60 setUpFilterEngine(this.ptr, isSubscriptionDownloadAllowedCallback); | 65 setUpFilterEngine(this.ptr, isSubscriptionDownloadAllowedCallback); |
(...skipping 28 matching lines...) Expand all Loading... | |
89 public void dispose() | 94 public void dispose() |
90 { | 95 { |
91 dtor(this.ptr); | 96 dtor(this.ptr); |
92 } | 97 } |
93 } | 98 } |
94 | 99 |
95 private final static native void registerNatives(); | 100 private final static native void registerNatives(); |
96 | 101 |
97 private final static native long ctor(LogSystem logSystem, WebRequest webReque st, String basePath); | 102 private final static native long ctor(LogSystem logSystem, WebRequest webReque st, String basePath); |
98 | 103 |
99 private final static native void setUpJsEngine(long ptr, AppInfo appInfo); | 104 private final static native void setUpJsEngine(long ptr, AppInfo appInfo, long v8IsolatePtr); |
100 | 105 |
101 private final static native long getJsEnginePtr(long ptr); | 106 private final static native long getJsEnginePtr(long ptr); |
102 | 107 |
103 private final static native void setUpFilterEngine(long ptr, IsAllowedConnecti onCallback isSubscriptionDownloadAllowedCallback); | 108 private final static native void setUpFilterEngine(long ptr, IsAllowedConnecti onCallback isSubscriptionDownloadAllowedCallback); |
104 | 109 |
105 private final static native void ensureFilterEngine(long ptr); | 110 private final static native void ensureFilterEngine(long ptr); |
106 | 111 |
107 private final static native void dtor(long ptr); | 112 private final static native void dtor(long ptr); |
108 } | 113 } |
OLD | NEW |