Bug 978509 - Don't preload any of the ICU DLLs, not just the ICU data DLL; r=bsmedberg
authorEhsan Akhgari <ehsan@mozilla.com>
Mon, 03 Mar 2014 14:08:13 -0500
changeset 188825 76a1e5c4c53d9fd106af8a97cfef46fad5a31325
parent 188824 85a239cf7dac055e014db295df70c1078a4216d6
child 188826 cc88afe57b8e1cf8289dc42945a76bc5a1dd0d06
push id3503
push userraliiev@mozilla.com
push dateMon, 28 Apr 2014 18:51:11 +0000
treeherdermozilla-beta@c95ac01e332e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbsmedberg
bugs978509
milestone30.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 978509 - Don't preload any of the ICU DLLs, not just the ICU data DLL; r=bsmedberg
toolkit/library/dependentlibs.py
--- a/toolkit/library/dependentlibs.py
+++ b/toolkit/library/dependentlibs.py
@@ -101,17 +101,17 @@ def dependentlibs(lib, libpaths, func):
             continue
         for dir in libpaths:
             deppath = os.path.join(dir, dep)
             if os.path.exists(deppath):
                 deps.extend([d for d in dependentlibs(deppath, libpaths, func) if not d in deps])
                 # Black list the ICU data DLL because preloading it at startup
                 # leads to startup performance problems because of its excessive
                 # size (around 10MB).
-                if not dep.startswith("icudt"):
+                if not dep.startswith("icu"):
                     deps.append(dep)
                 break
 
     return deps
 
 def main():
     parser = OptionParser()
     parser.add_option("-L", dest="libpaths", action="append", metavar="PATH", help="Add the given path to the library search path")