Declaring the weak_ptr_factory in proper order.

chrome/browser/chromeos/policy/app_pack_updater.cc

 // Copyright (c) 2012 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 "chrome/browser/chromeos/policy/app_pack_updater.h"
 
 #include "base/bind.h"
 #include "base/memory/ref_counted.h"
 #include "base/sequenced_task_runner.h"
 #include "base/threading/sequenced_worker_pool.h"
@@ skipping 44 matching lines @@
   virtual ~AppPackExternalLoader() {}
 
  private:
   base::DictionaryValue app_pack_prefs_;
 
   DISALLOW_COPY_AND_ASSIGN(AppPackExternalLoader);
 };
 
 AppPackUpdater::AppPackUpdater(net::URLRequestContextGetter* request_context,
                                EnterpriseInstallAttributes* install_attributes)
-    : weak_ptr_factory_(this),
-      created_extension_loader_(false),
+    : created_extension_loader_(false),
       install_attributes_(install_attributes),
       external_cache_(base::FilePath(kAppPackCacheDir),
                       request_context,
                       content::BrowserThread::GetBlockingPool()->
                           GetSequencedTaskRunnerWithShutdownBehavior(
                               content::BrowserThread::GetBlockingPool()->
                                   GetSequenceToken(),
                               base::SequencedWorkerPool::SKIP_ON_SHUTDOWN),
                       this,
                       false,
-                      false) {
+                      false),
+      weak_ptr_factory_(this) {
   app_pack_subscription_ = chromeos::CrosSettings::Get()->AddSettingsObserver(
       chromeos::kAppPack,
       base::Bind(&AppPackUpdater::AppPackChanged, base::Unretained(this)));
 
   if (install_attributes_->GetMode() == DEVICE_MODE_RETAIL_KIOSK) {
     // Already in Kiosk mode, start loading.
     BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
                             base::Bind(&AppPackUpdater::LoadPolicy,
                                        weak_ptr_factory_.GetWeakPtr()));
   } else {
@@ skipping 121 matching lines @@
 void AppPackUpdater::SetScreenSaverPath(const base::FilePath& path) {
   // Don't invoke the callback if the path isn't changing.
   if (path != screen_saver_path_) {
     screen_saver_path_ = path;
     if (!screen_saver_update_callback_.is_null())
       screen_saver_update_callback_.Run(screen_saver_path_);
   }
 }
 
 }  // namespace policy