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

Issue 10388120: Fixes the naming in the About page of CrOS. (Closed)

Created:
8 years, 7 months ago by Kyle Horimoto
Modified:
8 years, 7 months ago
CC:
chromium-reviews, stevenjb+watch_chromium.org, nkostylev+watch_chromium.org, davemoore+watch_chromium.org, kareng
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Fixes the naming in the About page of CrOS. Introduces a new constant IDS_SHORT_PRODUCT_OS_NAME (i.e., Chrome OS), and changes the old constant IDS_PRODUCT_OS_NAME to be the full name (i.e., Google Chrome OS) to remain in sync with the other string names. Changes previous references to IDS_PRODUCT_OS_NAME to IDS_SHORT_PRODUCT_OS_NAME. BUG=122645 TEST=look at help page on CrOS and see that text says Google Chrome OS Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=137195

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -7 lines) Patch
M chrome/app/chromium_strings.grd View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/app/google_chrome_strings.grd View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/browser/chromeos/login/network_screen.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/login/webui_login_display.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/views/update_recommended_message_box.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/help/help_handler.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/ui/webui/options2/chromeos/cros_language_options_handler2.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/uber/uber_ui.cc View 1 chunk +2 lines, -2 lines 2 comments Download
M chrome/common/chrome_version_info.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (0 generated)
Kyle Horimoto
jhawkins: Need review for chrome/common/chrome_version_info.cc estade: Need review for chrome/browser/ui/webui/* cpu: Need review for chrome/app/* ...
8 years, 7 months ago (2012-05-14 20:03:35 UTC) #1
James Hawkins
my file LGTM
8 years, 7 months ago (2012-05-14 20:10:45 UTC) #2
cpu_(ooo_6.6-7.5)
my file lgtm
8 years, 7 months ago (2012-05-14 21:05:25 UTC) #3
Evan Stade
webui lgtm https://chromiumcodereview.appspot.com/10388120/diff/1/chrome/browser/ui/webui/uber/uber_ui.cc File chrome/browser/ui/webui/uber/uber_ui.cc (right): https://chromiumcodereview.appspot.com/10388120/diff/1/chrome/browser/ui/webui/uber/uber_ui.cc#newcode94 chrome/browser/ui/webui/uber/uber_ui.cc:94: // TODO(jhawkins): Attempt to get rid of ...
8 years, 7 months ago (2012-05-14 22:53:48 UTC) #4
Kyle Horimoto
https://chromiumcodereview.appspot.com/10388120/diff/1/chrome/browser/ui/webui/uber/uber_ui.cc File chrome/browser/ui/webui/uber/uber_ui.cc (right): https://chromiumcodereview.appspot.com/10388120/diff/1/chrome/browser/ui/webui/uber/uber_ui.cc#newcode94 chrome/browser/ui/webui/uber/uber_ui.cc:94: // TODO(jhawkins): Attempt to get rid of IDS_SHORT_PRODUCT_OS_NAME. On ...
8 years, 7 months ago (2012-05-14 23:03:23 UTC) #5
Nikita (slow)
chromeos/login lgtm
8 years, 7 months ago (2012-05-15 10:23:17 UTC) #6
Ben Goodger (Google)
LGTM for ui/views. note that for rote changes like this you can just tbr owners ...
8 years, 7 months ago (2012-05-15 16:08:17 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/khorimoto@chromium.org/10388120/1
8 years, 7 months ago (2012-05-15 17:05:22 UTC) #8
commit-bot: I haz the power
8 years, 7 months ago (2012-05-15 19:01:53 UTC) #9
Change committed as 137195

Powered by Google App Engine
This is Rietveld 408576698