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

Issue 2884423002: Increase theme version to fix M57->M58 upgrade not migrating themes. (Closed)

Created:
3 years, 7 months ago by Bret
Modified:
3 years, 7 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3071
Project:
chromium
Visibility:
Public.

Description

Increase theme version to fix M57->M58 upgrade not migrating themes. This is what happened: 1. http://crrev.com/2667753004 (Tom's patch) lands, which modifies ThemeProperties::OverwritableByUserThemeProperty and bumps the theme version to 46. 2. http://crrev.com/2663363002 (my patch) lands, which does some unrelated stuff and also bumps the theme version to 47. 3. My patch is merged back to M57. When people install M57 their theme version goes from 45 to 47, and so their themes are regenerated. 4. M58 comes out, which includes Tom's patch, but the version is still 47 so no one's patches are regenerated. So I essentially "skipped" Tom's theme version bump. This patch bumps the version again, in order to regenerate everyone's themes and get them in sync with Tom's patch. BUG=652948 Review-Url: https://codereview.chromium.org/2880003003 Cr-Original-Commit-Position: refs/heads/master@{#471606} Review-Url: https://codereview.chromium.org/2884423002 . Cr-Commit-Position: refs/branch-heads/3071@{#588} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} Committed: https://chromium.googlesource.com/chromium/src/+/fb1a94d396a22cfae2bf8876f7639ac57f31e886

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M chrome/browser/themes/browser_theme_pack.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (1 generated)
Bret
3 years, 7 months ago (2017-05-16 19:59:08 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
fb1a94d396a22cfae2bf8876f7639ac57f31e886.

Powered by Google App Engine
This is Rietveld 408576698