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

Issue 9664060: Replace string 'Misbehaving extension' with 'Extension error' in context of the webRequest API (Closed)

Created:
8 years, 9 months ago by battre
Modified:
8 years, 9 months ago
Reviewers:
chrome-ui-leads
CC:
chromium-reviews, jeffreyc, Matt Perry
Visibility:
Public.

Description

Replace string 'Misbehaving extension' with 'Extension error' in context of the webRequest API The string 'Misbehaving extension' indicated that the extension author did something wrong, even though the problem might be related to a conflicting modification of two webRequest API extensions. It is neither extension's fault, they are just incompatible. BUG=111700 TEST=no Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=127663

Patch Set 1 #

Patch Set 2 : Replaced 'Extension problems' with 'Extension error' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M chrome/app/generated_resources.grd View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (0 generated)
battre
Hi UI Leads, could you please review this? Thanks, Dominic
8 years, 9 months ago (2012-03-12 12:32:23 UTC) #1
battre
On 2012/03/12 12:32:23, battre wrote: > Hi UI Leads, > > could you please review ...
8 years, 9 months ago (2012-03-19 12:43:55 UTC) #2
Glen Murphy
On 2012/03/19 12:43:55, battre wrote: > On 2012/03/12 12:32:23, battre wrote: > > Hi UI ...
8 years, 9 months ago (2012-03-19 19:45:20 UTC) #3
battre
On 2012/03/19 19:45:20, Glen Murphy wrote: > On 2012/03/19 12:43:55, battre wrote: > > On ...
8 years, 9 months ago (2012-03-19 19:57:22 UTC) #4
Glen Murphy
LGTM
8 years, 9 months ago (2012-03-19 19:58:48 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/battre@chromium.org/9664060/5001
8 years, 9 months ago (2012-03-19 20:00:59 UTC) #6
commit-bot: I haz the power
Try job failure for 9664060-5001 (retry) on mac_rel for step "browser_tests". It's a second try, ...
8 years, 9 months ago (2012-03-19 21:49:45 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/battre@chromium.org/9664060/5001
8 years, 9 months ago (2012-03-19 21:55:03 UTC) #8
commit-bot: I haz the power
8 years, 9 months ago (2012-03-20 00:09:27 UTC) #9
Try job failure for 9664060-5001 (retry) on mac_rel for step "browser_tests".
It's a second try, previously, steps "ui_tests, browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

Powered by Google App Engine
This is Rietveld 408576698