Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(413)

Side by Side Diff: chrome/browser/chromeos/app_mode/kiosk_external_update_validator.cc

Issue 1080453002: Always destroy CrxInstaller on the UI Thread (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix CrOZ Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/app_mode/kiosk_external_update_validator.h" 5 #include "chrome/browser/chromeos/app_mode/kiosk_external_update_validator.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/location.h" 8 #include "base/location.h"
9 #include "content/public/browser/browser_thread.h" 9 #include "content/public/browser/browser_thread.h"
10 #include "extensions/common/extension.h" 10 #include "extensions/common/extension.h"
11 #include "extensions/common/manifest_constants.h" 11 #include "extensions/common/manifest_constants.h"
12 12
13 namespace chromeos { 13 namespace chromeos {
14 14
15 KioskExternalUpdateValidator::KioskExternalUpdateValidator( 15 KioskExternalUpdateValidator::KioskExternalUpdateValidator(
16 const scoped_refptr<base::SequencedTaskRunner>& backend_task_runner, 16 const scoped_refptr<base::SequencedTaskRunner>& backend_task_runner,
17 const extensions::CRXFileInfo& file, 17 const extensions::CRXFileInfo& file,
18 const base::FilePath& crx_unpack_dir, 18 const base::FilePath& crx_unpack_dir,
19 const base::WeakPtr<KioskExternalUpdateValidatorDelegate>& delegate) 19 const base::WeakPtr<KioskExternalUpdateValidatorDelegate>& delegate)
20 : backend_task_runner_(backend_task_runner), 20 : SandboxedUnpackerClient(
21 content::BrowserThread::GetMessageLoopProxyForThread(
22 content::BrowserThread::UI)),
23 backend_task_runner_(backend_task_runner),
21 crx_file_(file), 24 crx_file_(file),
22 crx_unpack_dir_(crx_unpack_dir), 25 crx_unpack_dir_(crx_unpack_dir),
23 delegate_(delegate) { 26 delegate_(delegate) {
24 } 27 }
25 28
26 KioskExternalUpdateValidator::~KioskExternalUpdateValidator() { 29 KioskExternalUpdateValidator::~KioskExternalUpdateValidator() {
27 } 30 }
28 31
29 void KioskExternalUpdateValidator::Start() { 32 void KioskExternalUpdateValidator::Start() {
30 scoped_refptr<extensions::SandboxedUnpacker> unpacker( 33 scoped_refptr<extensions::SandboxedUnpacker> unpacker(
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 72
70 content::BrowserThread::PostTask( 73 content::BrowserThread::PostTask(
71 content::BrowserThread::UI, FROM_HERE, 74 content::BrowserThread::UI, FROM_HERE,
72 base::Bind( 75 base::Bind(
73 &KioskExternalUpdateValidatorDelegate::OnExtenalUpdateUnpackSuccess, 76 &KioskExternalUpdateValidatorDelegate::OnExtenalUpdateUnpackSuccess,
74 delegate_, crx_file_.extension_id, extension->VersionString(), 77 delegate_, crx_file_.extension_id, extension->VersionString(),
75 minimum_browser_version, temp_dir)); 78 minimum_browser_version, temp_dir));
76 } 79 }
77 80
78 } // namespace chromeos 81 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698