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

Issue 212703006: Update KeystoneRegistration.framework to 1.2.0.7407 (Closed)

Created:
6 years, 8 months ago by Mark Mentovai
Modified:
6 years, 8 months ago
Reviewers:
Robert Sesek
CC:
chromium-reviews
Visibility:
Public.

Description

Update KeystoneRegistration.framework to 1.2.0.7407. The big change in this version is that an installation attempt initiated by the application can show up as ondemand on the server. This will allow us to provide throttled Mac downloads (such as a "5% push") with correct behavior. A previous registration framework, 1.2.0.6782, intended to fix this bug was backed out because it was not compatible with older installed Keystones. This version contains a fix for that problem. Previous commit: r245361, https://codereview.chromium.org/141193002 Backed out: r245603, https://codereview.chromium.org/141583005 BUG=35495 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=261532

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -4 lines) Patch
M chrome/browser/mac/keystone_glue.mm View 1 chunk +5 lines, -1 line 0 comments Download
M chrome/browser/mac/keystone_glue_unittest.mm View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/mac/keystone_registration.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Mark Mentovai
6 years, 8 months ago (2014-04-03 21:33:29 UTC) #1
Mark Mentovai
Committed patchset #1 manually as r261532.
6 years, 8 months ago (2014-04-03 21:36:34 UTC) #2
Mark Mentovai
TBR (this was already reviewed before, anyway)
6 years, 8 months ago (2014-04-03 21:42:03 UTC) #3
Robert Sesek
6 years, 8 months ago (2014-04-03 23:31:03 UTC) #4
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698