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

Delta Between Two Patch Sets: src/engine/main.cpp

Issue 10800092: Use libadblockplus update checker (Closed)
Left Patch Set: Created June 6, 2013, 3:18 p.m.
Right Patch Set: Addressed review comments Created June 7, 2013, 5:27 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
LEFTRIGHT
1 #include <AdblockPlus.h> 1 #include <AdblockPlus.h>
2 #include <functional> 2 #include <functional>
3 #include <vector> 3 #include <vector>
4 #include <Windows.h> 4 #include <Windows.h>
5 5
6 #include "../shared/AutoHandle.h" 6 #include "../shared/AutoHandle.h"
7 #include "../shared/Communication.h" 7 #include "../shared/Communication.h"
8 #include "../shared/Utils.h" 8 #include "../shared/Utils.h"
9 #include "../shared/Version.h" 9 #include "../shared/Version.h"
10 #include "Debug.h" 10 #include "Debug.h"
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
153 153
154 void OnUpdateAvailable(AdblockPlus::JsEnginePtr jsEngine, AdblockPlus::JsValue List& params) 154 void OnUpdateAvailable(AdblockPlus::JsEnginePtr jsEngine, AdblockPlus::JsValue List& params)
155 { 155 {
156 if (params.size() < 1) 156 if (params.size() < 1)
157 { 157 {
158 Debug("updateAvailable event missing URL"); 158 Debug("updateAvailable event missing URL");
159 return; 159 return;
160 } 160 }
161 161
162 Updater updater(jsEngine, params[0]->AsString()); 162 Updater updater(jsEngine, params[0]->AsString());
163 updater.StartUpdate(); 163 updater.Update();
164 } 164 }
165 } 165 }
166 166
167 std::auto_ptr<AdblockPlus::FilterEngine> CreateFilterEngine(const std::wstring& locale) 167 std::auto_ptr<AdblockPlus::FilterEngine> CreateFilterEngine(const std::wstring& locale)
168 { 168 {
169 AdblockPlus::AppInfo appInfo; 169 AdblockPlus::AppInfo appInfo;
170 appInfo.version = ToUtf8String(IEPLUGIN_VERSION); 170 appInfo.version = ToUtf8String(IEPLUGIN_VERSION);
171 appInfo.name = "adblockplusie"; 171 appInfo.name = "adblockplusie";
172 appInfo.platform = "msie"; 172 appInfo.platform = "msie";
173 appInfo.locale = ToUtf8String(locale); 173 appInfo.locale = ToUtf8String(locale);
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 } 221 }
222 catch (std::runtime_error e) 222 catch (std::runtime_error e)
223 { 223 {
224 DebugException(e); 224 DebugException(e);
225 return 1; 225 return 1;
226 } 226 }
227 } 227 }
228 228
229 return 0; 229 return 0;
230 } 230 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld