Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(81)

Unified Diff: extensions/renderer/script_injection.cc

Issue 2556153002: [Extensions] Convert more callers of CallModuleMethod (Closed)
Patch Set: revertdoc Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/renderer/script_injection.cc
diff --git a/extensions/renderer/script_injection.cc b/extensions/renderer/script_injection.cc
index 2a668126d0645903bfd1ddc3a7984b080b161284..e7d0b90e77ac1fd58bcbcc3fa315057be24aa1c7 100644
--- a/extensions/renderer/script_injection.cc
+++ b/extensions/renderer/script_injection.cc
@@ -288,12 +288,12 @@ void ScriptInjection::InjectJs(std::set<std::string>* executing_scripts,
}
void ScriptInjection::OnJsInjectionCompleted(
- const blink::WebVector<v8::Local<v8::Value> >& results) {
+ const std::vector<v8::Local<v8::Value>>& results) {
DCHECK(!did_inject_js_);
bool expects_results = injector_->ExpectsResults();
if (expects_results) {
- if (!results.isEmpty() && !results[0].IsEmpty()) {
+ if (!results.empty() && !results[0].IsEmpty()) {
// Right now, we only support returning single results (per frame).
std::unique_ptr<content::V8ValueConverter> v8_converter(
content::V8ValueConverter::create());

Powered by Google App Engine
This is Rietveld 408576698