Closed Bug 723797 Opened 9 years ago Closed 9 years ago

Delayload oleacc.dll

Categories

(Core :: Disability Access APIs, defect)

All
Windows Vista
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla13

People

(Reporter: emk, Assigned: m_kato)

References

Details

(Whiteboard: [inbound])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #710125 +++

Masatoshi Kimura [:emk] 2012-02-02 18:39:31 PST

(In reply to Makoto Kato from comment #4)
> oleacc.dll is already loaded on nspr4 and xul.dll's startup due to
> dependencies of shell32, comctl32 and others.
Incorrect. oleacc.dll is delay loaded from system DLLs. I didn't see oleacc.dll in the address space of firefox.exe (using Process Explorer).
Blocks: 710125
No longer depends on: 710125
(In reply to Masatoshi Kimura [:emk] from comment #0)
> +++ This bug was initially created as a clone of Bug #710125 +++
> 
> Masatoshi Kimura [:emk] 2012-02-02 18:39:31 PST
> 
> (In reply to Makoto Kato from comment #4)
> > oleacc.dll is already loaded on nspr4 and xul.dll's startup due to
> > dependencies of shell32, comctl32 and others.
> Incorrect. oleacc.dll is delay loaded from system DLLs. I didn't see
> oleacc.dll in the address space of firefox.exe (using Process Explorer).

Ah, windows vista is loaded.  but windows 7 isn't loaded.
Attached patch fixSplinter Review
Assignee: nobody → m_kato
Attachment #594052 - Flags: review?(surkov.alexander)
Comment on attachment 594052 [details] [diff] [review]
fix

Review of attachment 594052 [details] [diff] [review]:
-----------------------------------------------------------------

Maybe I'm not right person to review this. Does this change mean that oleacc library is loaded dynamically when the code calls any functions of this library? Btw, doesn't it overlap with OS_LIBS += $(call EXPAND_LIBNAME,oleacc) introduced in bug 710125?
(In reply to alexander :surkov from comment #3)
> Comment on attachment 594052 [details] [diff] [review]
> fix
> 
> Review of attachment 594052 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Maybe I'm not right person to review this. Does this change mean that oleacc
> library is loaded dynamically when the code calls any functions of this
> library? Btw, doesn't it overlap with OS_LIBS += $(call
> EXPAND_LIBNAME,oleacc) introduced in bug 710125?

-DELAYLOAD means that linker generates LoadLibrary when code uses the function of library.  Since previous code has some LoadLibrary call, it can reduce this call.

EXPAND_LIBNAME dones't generate DELAYLOAD flag since all libs doesn't need DELAYLOAD.  If adding DELAYLOAD for all libs, it will spend more startup time.
Attachment #594052 - Flags: review?(surkov.alexander) → review?(khuey)
https://hg.mozilla.org/mozilla-central/rev/50727e830c7b
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
You need to log in before you can comment on or make changes to this bug.