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

Issue 140693017: Clean-up: Replaces obsolete Font/FontList methods. (Closed)

Created:
6 years, 10 months ago by Yuki
Modified:
6 years, 10 months ago
Reviewers:
sadrul, Hiro Komatsu
CC:
chromium-reviews, kalyank, stevenjb+watch_chromium.org, sadrul, oshima+watch_chromium.org, ben+ash_chromium.org
Visibility:
Public.

Description

Clean-up: Replaces obsolete Font/FontList methods. Replaces obsolete methods of gfx::Font and gfx::FontList with new ones. BUG=265485 TEST=none Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=248926

Patch Set 1 #

Patch Set 2 : Synced. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -36 lines) Patch
M ash/ime/candidate_view.cc View 3 chunks +4 lines, -7 lines 0 comments Download
M ash/system/chromeos/audio/tray_audio.cc View 2 chunks +4 lines, -3 lines 0 comments Download
M ash/system/chromeos/network/tray_sms.cc View 3 chunks +7 lines, -7 lines 0 comments Download
M ash/system/date/date_view.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M ash/system/logout_button/logout_button_tray.cc View 2 chunks +2 lines, -5 lines 0 comments Download
M ash/system/monitor/tray_monitor.cc View 1 chunk +1 line, -1 line 0 comments Download
M ash/system/session_length_limit/tray_session_length_limit.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M ash/system/tray/hover_highlight_view.cc View 3 chunks +3 lines, -6 lines 0 comments Download
M ash/system/tray/tray_utils.cc View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Yuki
Could you guys review this CL? komatsu: ash/ime/ sadrul: ash/system/
6 years, 10 months ago (2014-02-03 12:58:31 UTC) #1
Hiro Komatsu
On 2014/02/03 12:58:31, Yuki wrote: > Could you guys review this CL? > > komatsu: ...
6 years, 10 months ago (2014-02-04 02:54:30 UTC) #2
sadrul
lgtm
6 years, 10 months ago (2014-02-04 16:19:34 UTC) #3
Yuki
The CQ bit was checked by yukishiino@chromium.org
6 years, 10 months ago (2014-02-05 04:42:58 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yukishiino@chromium.org/140693017/60001
6 years, 10 months ago (2014-02-05 04:54:38 UTC) #5
commit-bot: I haz the power
6 years, 10 months ago (2014-02-05 08:39:50 UTC) #6
Message was sent while issue was closed.
Change committed as 248926

Powered by Google App Engine
This is Rietveld 408576698