Fix tests failures caused by issue 906493004 (https://codereview.chromium.org/906493004/).

The failed tests were caused by missing the initialization of consumer_instance_type_ in UserScript.

BUG=463906
TBR=rdevlin

Review URL: https://codereview.chromium.org/976173002

Cr-Commit-Position: refs/heads/master@{#319207}

extensions/browser/user_script_loader.cc

@@ -444,7 +444,7 @@ void UserScriptLoader::OnScriptsLoaded(
   // We've got scripts ready to go.
   shared_memory_.reset(shared_memory.release());
 
-  // If user scripts are comming from a <webview>, will only notify the
+  // If user scripts are coming from a <webview>, will only notify the
   // RenderProcessHost of that <webview>; otherwise will notify all of the
   // RenderProcessHosts.
   if (user_scripts && !user_scripts->empty() &&

extensions/common/user_script.cc

@@ -81,6 +81,7 @@ UserScript::File::~File() {}
 
 UserScript::UserScript()
     : run_location_(DOCUMENT_IDLE),
+      consumer_instance_type_(TAB),
       user_script_id_(-1),
       emulate_greasemonkey_(false),
       match_all_frames_(false),

extensions/common/user_script.h

@@ -129,8 +129,8 @@ class UserScript {
   // Render's routing info of a <webview> that the user script will be injected
   // on. Only user scripts from <webview>s have a custom routing info.
   struct RoutingInfo {
-    RoutingInfo() : render_process_id(-1), render_view_id(-1) {};
-    ~RoutingInfo() {};
+    RoutingInfo() : render_process_id(-1), render_view_id(-1) {}
+    ~RoutingInfo() {}
 
     int render_process_id;
     int render_view_id;