diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-13 00:54:21 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-13 00:54:21 +0000 |
| commit | 046344c69b0e16f8fa2fdcaf7a8e486d26301486 (patch) | |
| tree | a2ff22d44b1595158eb6ffa7eafca06af34524f5 /chrome/browser | |
| parent | 7e8ede35f4a3e35e8d3948e7b0dec41d124a5244 (diff) | |
| download | chromium_src-046344c69b0e16f8fa2fdcaf7a8e486d26301486.zip chromium_src-046344c69b0e16f8fa2fdcaf7a8e486d26301486.tar.gz chromium_src-046344c69b0e16f8fa2fdcaf7a8e486d26301486.tar.bz2 | |
Fix layout test failures. Looks like keying the plugins based on OriginalFileName didn't work, since different filenames might still have the same value (i.e. QuickTime). Instead I did what Firefox does, which is collect the list of directories in one pass, then crawl them in another.
Review URL: http://codereview.chromium.org/17367
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@7907 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rw-r--r-- | chrome/browser/metrics_log.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/metrics_log.cc b/chrome/browser/metrics_log.cc index dc3ab86..c79ca04d 100644 --- a/chrome/browser/metrics_log.cc +++ b/chrome/browser/metrics_log.cc @@ -458,7 +458,8 @@ void MetricsLog::WritePluginList( // Plugin name and filename are hashed for the privacy of those // testing unreleased new extensions. WriteAttribute("name", CreateBase64Hash(WideToUTF8(iter->name))); - WriteAttribute("filename", CreateBase64Hash(iter->filename)); + WriteAttribute("filename", + CreateBase64Hash(WideToUTF8(iter->path.BaseName().ToWStringHack()))); WriteAttribute("version", WideToUTF8(iter->version)); } } |