Clean up threading model of external extension providers

Move blocking IO of loading the lists of external extensions from the UI thread to the FILE thread. Remove the ExternalPolicyExtensionProvider-specific parts from ExtensionsService. Side-effects: reduce the number of PostTask operations from 9 to 4. Trigger uninstall of external extensions immediately after they are removed from a policy.

BUG=65107,63667
TEST=ExtensionManagementTest.ExternalPolicyRefresh for the new functionality, lot of other extension tests are updated

Review URL: http://codereview.chromium.org/5742008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71427 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 0 insertions, 46 deletions

diff --git a/base/base.gypi b/base/base.gypi
index 27ec764..e2c659a 100644
--- a/base/base.gypi
+++ b/base/base.gypi
@@ -287,8 +287,6 @@
           'utf_string_conversions.h',
           'values.cc',
           'values.h',
-          'values_util.cc',
-          'values_util.h',
           'version.cc',
           'version.h',
           'vlog.cc',
diff --git a/base/values_util.cc b/base/values_util.cc
deleted file mode 100644
index fbc616b..0000000
--- a/base/values_util.cc
+++ /dev/null
@@ -1,18 +0,0 @@
-// Copyright (c) 2010 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 "base/values_util.h"
-
-RefCountedList::RefCountedList(ListValue* list) {
-  list_ = list;
-}
-
-RefCountedList::~RefCountedList() {
-  if (list_)
-    delete list_;
-}
-
-ListValue* RefCountedList::Get() {
-  return list_;
-}
diff --git a/base/values_util.h b/base/values_util.h
deleted file mode 100644
index 1626bb5..0000000
--- a/base/values_util.h
+++ /dev/null
@@ -1,26 +0,0 @@
-// Copyright (c) 2010 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.
-
-#ifndef BASE_VALUES_UTIL_H_
-#define BASE_VALUES_UTIL_H_
-#pragma once
-
-#include "base/values.h"
-#include "base/ref_counted.h"
-
-class RefCountedList : public base::RefCountedThreadSafe<RefCountedList> {
- public:
-  // Takes ownership of |list|.
-  explicit RefCountedList(ListValue* list);
-  virtual ~RefCountedList();
-
-  virtual ListValue* Get();
-
- private:
-  ListValue* list_;
-
-  DISALLOW_COPY_AND_ASSIGN(RefCountedList);
-};
-
-#endif  // BASE_VALUES_UTIL_H_