diff options
author | hbono@chromium.org <hbono@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-10-17 03:24:23 +0000 |
---|---|---|
committer | hbono@chromium.org <hbono@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-10-17 03:24:23 +0000 |
commit | 2d1f9b78381767f11fb897b7c9d624eb81569040 (patch) | |
tree | b28e3f74e9202c39257dd48f160893731fba9c0c /chrome/browser/views | |
parent | 983f90b71c19ba6c3715fe084732e6b50de708c1 (diff) | |
download | chromium_src-2d1f9b78381767f11fb897b7c9d624eb81569040.zip chromium_src-2d1f9b78381767f11fb897b7c9d624eb81569040.tar.gz chromium_src-2d1f9b78381767f11fb897b7c9d624eb81569040.tar.bz2 |
A quick fix for a possible assertion caused by r3407.
When a user opens a "Fonts and Languages" dialog (through [Options] -> [Change Font and Language Settings]) and presses its "OK" button without changing anything, the debug build of Chromium raises an assert exception because the LanguagePageView::SaveChanges() function reads the value of an initialized member variable |LanguagePageView::spellcheck_language_index_selected_|.
To prevent this possible assertion, this change initializes the member variable |LanguagePageView::spellcheck_language_index_selected_| to -1 in the constructor of the LanguagePageView class.
Review URL: http://codereview.chromium.org/7387
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@3506 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views')
-rw-r--r-- | chrome/browser/views/options/languages_page_view.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/views/options/languages_page_view.cc b/chrome/browser/views/options/languages_page_view.cc index fc8398e..1330027 100644 --- a/chrome/browser/views/options/languages_page_view.cc +++ b/chrome/browser/views/options/languages_page_view.cc @@ -482,7 +482,8 @@ LanguagesPageView::LanguagesPageView(Profile* profile) change_dictionary_language_combobox_(NULL), dictionary_language_label_(NULL), OptionsPageView(profile), - language_table_edited_(false) { + language_table_edited_(false), + spellcheck_language_index_selected_(-1) { accept_languages_.Init(prefs::kAcceptLanguages, profile->GetPrefs(), NULL); } |