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

Issue 6272018: Fix speech input bubble in GTK to avoid setting images when not visible. (Closed)

Created:
9 years, 11 months ago by Leandro Graciá Gil
Modified:
9 years, 7 months ago
Reviewers:
Satish
CC:
chromium-reviews
Visibility:
Public.

Description

Fix speech input bubble in GTK to avoid setting images when not visible. The visibility of the speech input bubble was not being checked when a image was requested to be set. Consequently, this could lead to invalid image objects in some cases producing glib and GTK errors. BUG=70223 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72501

Patch Set 1 : applying changes suggested by Satish. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -2 lines) Patch
M chrome/browser/speech/speech_input_bubble_gtk.cc View 2 chunks +1 line, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Leandro Graciá Gil
9 years, 11 months ago (2011-01-25 12:53:43 UTC) #1
Leandro Graciá Gil
Fix adapted to Satish's suggestions.
9 years, 11 months ago (2011-01-25 15:43:45 UTC) #2
Satish
9 years, 11 months ago (2011-01-25 16:30:18 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698