Last Comment Bug 709562 - nsAutoDialWin does not need to load rasapi32/rasdlg dynamically
: nsAutoDialWin does not need to load rasapi32/rasdlg dynamically
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Networking (show other bugs)
: Trunk
: All Windows XP
: -- normal (vote)
: mozilla12
Assigned To: Makoto Kato [:m_kato]
:
Mentors:
Depends on:
Blocks: deadcode
  Show dependency treegraph
 
Reported: 2011-12-11 02:18 PST by Makoto Kato [:m_kato]
Modified: 2012-01-04 04:41 PST (History)
7 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
fix (15.91 KB, patch)
2011-12-11 20:51 PST, Makoto Kato [:m_kato]
jmathies: review+
Details | Diff | Splinter Review
part 2. add delay load for ras apis (886 bytes, patch)
2011-12-20 00:44 PST, Makoto Kato [:m_kato]
khuey: review+
Details | Diff | Splinter Review

Description Makoto Kato [:m_kato] 2011-12-11 02:18:30 PST
Current Gecko doesn't run on Windows 9x and NT4.0.  So it is unnecessary to use LoadLibrary for rasapi32.dll and rasdlg.dll.
Comment 1 Makoto Kato [:m_kato] 2011-12-11 20:51:49 PST
Created attachment 580822 [details] [diff] [review]
fix
Comment 2 Jim Mathies [:jimm] 2011-12-12 10:48:59 PST
Comment on attachment 580822 [details] [diff] [review]
fix

Presumably this is to help bug 709193? This looks fine.
Comment 3 Marco Castelluccio [:marco] 2011-12-14 16:21:18 PST
Did the patch land?
Comment 4 Makoto Kato [:m_kato] 2011-12-20 00:44:22 PST
Created attachment 583091 [details] [diff] [review]
part 2. add delay load for ras apis

Note You need to log in before you can comment on or make changes to this bug.