Comment #8 on issue 17566 by bugdro...@chromium.org: Browser crash @
TemplateURLTableModel::GetText()
http://code.google.com/p/chromium/issues/detail?id=17566
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=22823
------------------------------------------------------------------------
r22823 | lafo...@chromium.org | 2009-08-07 16:10:40 -0700 (Fri, 07 Aug
2009) | 12 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/branches/195/src/chrome/browser/search_engines/template_url_table_model.cc?r1=22823&r2=22822
M
http://src.chromium.org/viewvc/chrome/branches/195/src/chrome/browser/search_engines/template_url_table_model.h?r1=22823&r2=22822
Merge 22088 - Fix search engine manager allowing you to remove the current
default search engine.
If a search engine entry is already in the main group,
MakeDefaultTemplateURL should return the current index(so that button
states will be updated) rather than 1(which should mean nothing was done.)
BUG=17566
TEST=Select search engine that is already in the main group but not the
default, and click "make default". The "make default" and "remove" buttons
should no longer be clickable.
Review URL: http://codereview.chromium.org/159557
tbr=ma...@chromium.org
Review URL: http://codereview.chromium.org/165181
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
