Initialize POD members of ApplicationManager, and other drive-by's.

shell/application_manager/application_manager.cc

Index: shell/application_manager/application_manager.cc
diff --git a/shell/application_manager/application_manager.cc b/shell/application_manager/application_manager.cc
index abaa11a1d932c773be249e4d5468fc1ea29d5173..037f963cabd3c9cde52243a8b75dd14a47066e3a 100644
--- a/shell/application_manager/application_manager.cc
+++ b/shell/application_manager/application_manager.cc
@@ -19,7 +19,6 @@
 #include "shell/application_manager/network_fetcher.h"
 #include "shell/application_manager/query_util.h"
 #include "shell/application_manager/shell_impl.h"
-#include "shell/switches.h"
 
 using mojo::Application;
 using mojo::ApplicationPtr;
@@ -97,7 +96,10 @@ bool ApplicationManager::TestAPI::HasFactoryForURL(const GURL& url) const {
 }
 
 ApplicationManager::ApplicationManager(Delegate* delegate)
-    : delegate_(delegate), weak_ptr_factory_(this) {
+    : delegate_(delegate),
+      blocking_pool_(),

[jamesr, 2015/05/14 22:11:22]

could you say blocking_pool_(nullptr)?  I know this does the same but it's a
corner of C++ that a surprising number of people don't know and having the type
info repeated here is useful

+      disable_cache_(false),
+      weak_ptr_factory_(this) {
 }
 
 ApplicationManager::~ApplicationManager() {