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

Issue 825753002: Revert "Fix Website Settings crash with Plugins ASK setting." (Closed)

Created:
5 years, 12 months ago by eroman
Modified:
5 years, 12 months ago
Reviewers:
tommycli
CC:
chromium-reviews, markusheintz_
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert "Fix Website Settings crash with Plugins ASK setting." This reverts commit 5abc69b1a406999169052b7d457de86fd67c6f8c. Reason for revert: The Linux GN bot failed compile: http://build.chromium.org/p/chromium.linux/buildstatus?builder=Linux%20GN&number=19568 Due to error running grit: KeyError: 'Duplicate keys: [18500]' We have seen this error before, and it should have already been fixed in grit's resource_ids.... I expect the problem is with GN since this failure isn't happening on any other bot, however reverting in the meantime to make the tree green until this is sorted out. TBR=tommycli@chromium.org NOTRY=true Committed: https://crrev.com/09eafe7304589043a9b9aa34b9ef559ead1e8e59 Cr-Commit-Position: refs/heads/master@{#309567}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -17 lines) Patch
M chrome/app/generated_resources.grd View 3 chunks +0 lines, -9 lines 0 comments Download
M chrome/browser/ui/website_settings/permission_menu_model.cc View 1 chunk +0 lines, -6 lines 0 comments Download
M chrome/browser/ui/website_settings/website_settings_ui.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 11 (1 generated)
eroman
5 years, 12 months ago (2014-12-23 18:48:45 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/825753002/1
5 years, 12 months ago (2014-12-23 18:49:28 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 12 months ago (2014-12-23 18:50:13 UTC) #4
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/09eafe7304589043a9b9aa34b9ef559ead1e8e59 Cr-Commit-Position: refs/heads/master@{#309567}
5 years, 12 months ago (2014-12-23 18:51:05 UTC) #5
eroman
@tommycli: You are unfortunately caught in the middle of some infrastructure problems. Following up on ...
5 years, 12 months ago (2014-12-23 19:21:50 UTC) #6
tommycli
On 2014/12/23 19:21:50, eroman wrote: > @tommycli: You are unfortunately caught in the middle of ...
5 years, 12 months ago (2014-12-23 19:24:04 UTC) #7
eroman
I am about to re-land your change, I have an idea on how to make ...
5 years, 12 months ago (2014-12-23 19:28:46 UTC) #8
tommycli
On 2014/12/23 19:28:46, eroman wrote: > I am about to re-land your change, I have ...
5 years, 12 months ago (2014-12-23 19:31:17 UTC) #9
chromium-reviews
By the way, this https://codereview.chromium.org/819183002/ is a followup that's due to be sent in next. ...
5 years, 12 months ago (2014-12-23 19:32:04 UTC) #10
eroman
5 years, 12 months ago (2014-12-23 20:33:31 UTC) #11
Message was sent while issue was closed.
The bots are happy now (see
https://code.google.com/p/chromium/issues/detail?id=444902).

Feel free to commit any follow-ups at your leisure!

Powered by Google App Engine
This is Rietveld 408576698