diff options
| author | xiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-23 01:17:35 +0000 |
|---|---|---|
| committer | xiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-23 01:17:35 +0000 |
| commit | 4349dcf41a371acd0ca85fa48fa46995186d2059 (patch) | |
| tree | beef3b992021678adf0a7304f7c0bc931735e7cd /chrome/browser/gtk/browser_titlebar.cc | |
| parent | 34e13c29bd983b7f6694c2d44162260a6de32b62 (diff) | |
| download | chromium_src-4349dcf41a371acd0ca85fa48fa46995186d2059.zip chromium_src-4349dcf41a371acd0ca85fa48fa46995186d2059.tar.gz chromium_src-4349dcf41a371acd0ca85fa48fa46995186d2059.tar.bz2 | |
Fix app shortcut mess up per issue 31789
Issue 31789 is caused by a longer than MAX_PATH chars web app
description. Fix it by limiting the description string size to MAX_PATH.
Note:
1. MSDN mentions that the size limitation for Win2000 or later is
INFOTIPSIZE but the limit on my vista dev machine is MAX_PATH.
<http://msdn.microsoft.com/en-us/library/bb774955(VS.85).aspx>
2. The size limit includes the terminating NULL.
BUG=31789
TEST=Verify fix for issue 31789. facebook.com has a 261-char description and chromium should be able to create a working shortcut for it.
Review URL: http://codereview.chromium.org/650186
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39686 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/gtk/browser_titlebar.cc')
0 files changed, 0 insertions, 0 deletions