Coverity: Initialize member variables.

net/url_request/url_request_job_manager.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/url_request/url_request_job_manager.h"
 
 #include <algorithm>
 
 #include "base/memory/singleton.h"
 #include "build/build_config.h"
@@ skipping 244 matching lines @@
   DCHECK(IsAllowedThread());
 
   base::AutoLock locked(lock_);
 
   InterceptorList::iterator i =
       std::find(interceptors_.begin(), interceptors_.end(), interceptor);
   DCHECK(i != interceptors_.end());
   interceptors_.erase(i);
 }
 
-URLRequestJobManager::URLRequestJobManager() : enable_file_access_(false) {
-#ifndef NDEBUG
-  allowed_thread_ = 0;
-  allowed_thread_initialized_ = false;
-#endif
+URLRequestJobManager::URLRequestJobManager()
+    : allowed_thread_(0),

[groby-ooo-7-16, 2011/12/20 20:03:34]

Are those intentionally uninitialized in debug builds?

[James Hawkins, 2011/12/20 20:41:37]

No, it's a bug.

+      allowed_thread_initialized_(false),
+      enable_file_access_(false) {
 }
 
 URLRequestJobManager::~URLRequestJobManager() {}
 
 }  // namespace net