Bug 978509 - Don't preload any of the ICU DLLs, not just the ICU data DLL. r=bsmedberg, a=sledru
authorEhsan Akhgari <ehsan@mozilla.com>
Mon, 03 Mar 2014 14:08:13 -0500
changeset 183135 04b0dd33d9e881c41bf12ae8dc1c497aec2e8a84
parent 183134 fa4ee9bcf941f448194857cddac853cab485ac88
child 183136 7072e385c5de8d0483a25b02e63350f1d86cc633
push id3343
push userffxbld
push dateMon, 17 Mar 2014 21:55:32 +0000
treeherdermozilla-beta@2f7d3415f79f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbsmedberg, sledru
bugs978509
milestone29.0a2
Bug 978509 - Don't preload any of the ICU DLLs, not just the ICU data DLL. r=bsmedberg, a=sledru
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")