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

Side by Side Diff: components/update_client/action_update.cc

Issue 2384873002: Use the new fingerprint during unpacking, not the old. (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "components/update_client/action_update.h" 5 #include "components/update_client/action_update.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 121 matching lines...) Expand 10 before | Expand all | Expand 10 after
132 FROM_HERE, 132 FROM_HERE,
133 base::Bind(&ActionUpdate::DoInstallOnBlockingTaskRunner, 133 base::Bind(&ActionUpdate::DoInstallOnBlockingTaskRunner,
134 base::Unretained(this), update_context_, item, crx_path)); 134 base::Unretained(this), update_context_, item, crx_path));
135 } 135 }
136 136
137 void ActionUpdate::DoInstallOnBlockingTaskRunner( 137 void ActionUpdate::DoInstallOnBlockingTaskRunner(
138 UpdateContext* update_context, 138 UpdateContext* update_context,
139 CrxUpdateItem* item, 139 CrxUpdateItem* item,
140 const base::FilePath& crx_path) { 140 const base::FilePath& crx_path) {
141 unpacker_ = new ComponentUnpacker( 141 unpacker_ = new ComponentUnpacker(
142 item->component.pk_hash, crx_path, item->component.fingerprint, 142 item->component.pk_hash, crx_path, item->next_fp,
143 item->component.installer, 143 item->component.installer,
144 update_context->config->CreateOutOfProcessPatcher(), 144 update_context->config->CreateOutOfProcessPatcher(),
145 update_context->blocking_task_runner); 145 update_context->blocking_task_runner);
146 unpacker_->Unpack(base::Bind(&ActionUpdate::EndUnpackingOnBlockingTaskRunner, 146 unpacker_->Unpack(base::Bind(&ActionUpdate::EndUnpackingOnBlockingTaskRunner,
147 base::Unretained(this), update_context, item, 147 base::Unretained(this), update_context, item,
148 crx_path)); 148 crx_path));
149 } 149 }
150 150
151 void ActionUpdate::EndUnpackingOnBlockingTaskRunner( 151 void ActionUpdate::EndUnpackingOnBlockingTaskRunner(
152 UpdateContext* update_context, 152 UpdateContext* update_context,
(...skipping 211 matching lines...) Expand 10 before | Expand all | Expand 10 after
364 364
365 item->error_category = static_cast<int>(UnpackerErrorToErrorCategory(error)); 365 item->error_category = static_cast<int>(UnpackerErrorToErrorCategory(error));
366 item->error_code = error; 366 item->error_code = error;
367 item->extra_code1 = extended_error; 367 item->extra_code1 = extended_error;
368 ChangeItemState(item, CrxUpdateItem::State::kNoUpdate); 368 ChangeItemState(item, CrxUpdateItem::State::kNoUpdate);
369 369
370 UpdateCrxComplete(item); 370 UpdateCrxComplete(item);
371 } 371 }
372 372
373 } // namespace update_client 373 } // namespace update_client
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698