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

Issue 7489036: Unify the version string to be displayed on "About Chromium" dialog. (3/6) (Closed)

Created:
9 years, 5 months ago by haraken1
Modified:
9 years, 5 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Unify the version string to be displayed on "About Chromium" dialog. (3/6) - Refactor the views about dialog to use shared code for generating the version string. - Remove the OS version string from the views about dialog because it is no longer needed to differentiate between Windows and ChromeOS. We decided to split the changes of issue 7104106 (http://codereview.chromium.org/7104106/) into the following steps. This is the third step of them: (1) Add CreateVersionString() to chrome_version_info.*, update the GTK+ code to use it. (reviewer: erg, tony) (2) Update the mac code to use CreateVersionString(). Update About.xib. (reviewer: mark) (3) Update the views code to use CreateVersionString(). This can happen at the same time as (2). (reviewer: davemoore, tony) (4) Update version.bat to use lastchange.py. (reviewer: evan, mark) (5) Update tweak_info_plist to use lastchange.py. This can happen at the same time as (4). (reviewer: mark) (6) Make changes to lastchange.py and webkit_version.py. (reviewer: evan, mark) BUG=37186 TEST=Observe that "About Chromium" dialog shows the version string like "14.0.787.0 (Developer Build 88242 Windows)" on Windows. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=94253

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -60 lines) Patch
M chrome/browser/ui/views/about_chrome_view.h View 2 chunks +0 lines, -9 lines 0 comments Download
M chrome/browser/ui/views/about_chrome_view.cc View 8 chunks +5 lines, -51 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
haraken1
davemoore, tony: Would you please take a look at it? mark: FYI
9 years, 5 months ago (2011-07-25 02:34:46 UTC) #1
DaveMoore
In chromeos the about chromium dialog is never displayed...we use the web ui.
9 years, 5 months ago (2011-07-25 17:04:19 UTC) #2
tony
The code change LGTM, but please fix the description to describe only what this change ...
9 years, 5 months ago (2011-07-25 17:04:50 UTC) #3
tony
On 2011/07/25 17:04:19, DaveMoore wrote: > In chromeos the about chromium dialog is never displayed...we ...
9 years, 5 months ago (2011-07-25 17:05:48 UTC) #4
haraken1
> In chromeos the about chromium dialog is never displayed...we use the web ui. Yes, ...
9 years, 5 months ago (2011-07-25 21:49:51 UTC) #5
DaveMoore
lgtm
9 years, 5 months ago (2011-07-26 23:32:09 UTC) #6
commit-bot: I haz the power
9 years, 5 months ago (2011-07-27 06:29:37 UTC) #7
Change committed as 94253

Powered by Google App Engine
This is Rietveld 408576698