diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-11 01:10:45 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-11 01:10:45 +0000 |
| commit | b4037d13ad107e6fc72e1149ed4f760e08aedd9a (patch) | |
| tree | 370e5a60e694bab5d7e6dc04010c5c2a966fd6d0 /net | |
| parent | e896a4d2af4d4c15aeeeb66fb2b18950f777f103 (diff) | |
| download | chromium_src-b4037d13ad107e6fc72e1149ed4f760e08aedd9a.zip chromium_src-b4037d13ad107e6fc72e1149ed4f760e08aedd9a.tar.gz chromium_src-b4037d13ad107e6fc72e1149ed4f760e08aedd9a.tar.bz2 | |
Lands http://codereview.chromium.org/379007 for Dave:
Fixes leak in BackingStore as shown by valgrind. XFree doesn't free
the data of the image, where as XDestroyImage does.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/391008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31633 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net')
| -rw-r--r-- | net/base/net_util_posix.cc | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/base/net_util_posix.cc b/net/base/net_util_posix.cc index e1fc700..651be1b 100644 --- a/net/base/net_util_posix.cc +++ b/net/base/net_util_posix.cc @@ -21,6 +21,8 @@ bool FileURLToFilePath(const GURL& url, FilePath* path) { // Firefox seems to ignore the "host" of a file url if there is one. That is, // file://foo/bar.txt maps to /bar.txt. + // TODO(dhg): This should probably take into account UNCs which could + // include a hostname other than localhost or blank std::string old_path = url.path(); if (old_path.empty()) |