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

Issue 43077: Add a GDK_COLOR_RGB macro for constructing GdkColors. (Closed)

Created:
11 years, 9 months ago by Dean McNamee
Modified:
9 years, 7 months ago
Reviewers:
Evan Martin, Evan Stade
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Add a GDK_COLOR_RGB macro for constructing GdkColors. This does the simple math of * 257 to map from a 1 byte component to the gint16 GdkColor values, and initializes the pixel field to 0.

Patch Set 1 #

Patch Set 2 : Merge to trunk #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -7 lines) Patch
M base/gfx/gtk_util.h View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/autocomplete/autocomplete_popup_view_gtk.cc View 2 chunks +4 lines, -3 lines 0 comments Download
M chrome/browser/gtk/download_shelf_gtk.cc View 1 2 chunks +3 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Dean McNamee
11 years, 9 months ago (2009-03-11 14:19:23 UTC) #1
Evan Stade
this lgtm
11 years, 9 months ago (2009-03-11 17:32:18 UTC) #2
Dean McNamee
Evan was saying he thinks this is wrong, so I gotta wait to see what ...
11 years, 9 months ago (2009-03-11 17:33:25 UTC) #3
Evan Martin
11 years, 9 months ago (2009-03-11 18:15:28 UTC) #4
LG

Powered by Google App Engine
This is Rietveld 408576698