Chromium Code Reviews
Help | Chromium Project | Sign in
(1)

Issue 12928004: Add nacl_arch to data we send in omaha update checks for extensions/components (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 3 months ago by Antony Sargent
Modified:
2 years, 3 months ago
Reviewers:
jvoung, Nico
CC:
chromium-reviews, cpu
Visibility:
Public.

Description

Add nacl_arch to data we send in omaha update checks for extensions/components This is to support the webstore multi-crx feature BUG=180402 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=190614

Patch Set 1 #

Total comments: 1

Patch Set 2 : add missing includes to fix windows compile error #

Messages

Total messages: 12 (0 generated)
Antony Sargent
Carlos - please take a look Jan - is it ok to consider the pnacl ...
2 years, 3 months ago (2013-03-23 00:05:33 UTC) #1
jvoung - send to chromium...
On 2013/03/23 00:05:33, Antony Sargent wrote: > Carlos - please take a look > > ...
2 years, 3 months ago (2013-03-23 00:32:59 UTC) #2
Antony Sargent
Sounds good. Jan, I guess you're actually in the OWNERS file for this directory - ...
2 years, 3 months ago (2013-03-25 03:31:06 UTC) #3
jvoung
lgtm
2 years, 3 months ago (2013-03-25 16:09:53 UTC) #4
Antony Sargent
thakis: please review changes to chrome/common/omaha_query_params.{h,cc} (As an aside, these files probably deserve more specific ...
2 years, 3 months ago (2013-03-25 17:14:29 UTC) #5
Nico
lgtm https://codereview.chromium.org/12928004/diff/1/chrome/common/omaha_query_params.h File chrome/common/omaha_query_params.h (right): https://codereview.chromium.org/12928004/diff/1/chrome/common/omaha_query_params.h#newcode38 chrome/common/omaha_query_params.h:38: // "arm", and "mips32". Is there a good ...
2 years, 3 months ago (2013-03-25 17:21:25 UTC) #6
Antony Sargent
On 2013/03/25 17:21:25, Nico wrote: > lgtm > > https://codereview.chromium.org/12928004/diff/1/chrome/common/omaha_query_params.h > File chrome/common/omaha_query_params.h (right): > ...
2 years, 3 months ago (2013-03-25 17:34:05 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/asargent@chromium.org/12928004/1
2 years, 3 months ago (2013-03-25 17:48:33 UTC) #8
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
2 years, 3 months ago (2013-03-25 18:51:48 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/asargent@chromium.org/12928004/31001
2 years, 3 months ago (2013-03-25 23:23:02 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/asargent@chromium.org/12928004/31001
2 years, 3 months ago (2013-03-26 07:06:32 UTC) #11
commit-bot: I haz the power
2 years, 3 months ago (2013-03-26 09:13:08 UTC) #12
Message was sent while issue was closed.
Change committed as 190614
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1f9106d