The default bug view has changed. See this FAQ.

Delayload oleacc.dll

RESOLVED FIXED in mozilla13

Status

()

Core
Disability Access APIs
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: emk, Assigned: m_kato)

Tracking

Trunk
mozilla13
All
Windows Vista
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [inbound])

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
+++ 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).
(Reporter)

Updated

5 years ago
Blocks: 710125
No longer depends on: 710125
(Assignee)

Comment 1

5 years ago
(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.
(Assignee)

Comment 2

5 years ago
Created attachment 594052 [details] [diff] [review]
fix
Assignee: nobody → m_kato
(Assignee)

Updated

5 years ago
Attachment #594052 - Flags: review?(surkov.alexander)

Comment 3

5 years ago
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?
(Assignee)

Comment 4

5 years ago
(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.
(Assignee)

Updated

5 years ago
Attachment #594052 - Flags: review?(surkov.alexander) → review?(khuey)
Attachment #594052 - Flags: review?(khuey) → review+
(Assignee)

Comment 5

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/50727e830c7b
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/50727e830c7b
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
You need to log in before you can comment on or make changes to this bug.