710125 - Reduce LoadLibray/GetProcAddress for user32.dll and oleacc.dll on a11y

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Makoto Kato [:m_kato]]

Current Gecko supports under Windows 2000 and later  (After switching to VS2010, it is Windows XP or later).  So we don't need to use LoadLibrary for oleacc and user32 on accessibility code.

ex. LresultFromCode is from Win2000 by http://msdn.microsoft.com/en-us/library/windows/desktop/dd318557%28v=vs.85%29.aspx.

Comment 1

[Makoto Kato [:m_kato]]

Attachment: fix

Comment 2

[Trevor Saunders (:tbsaunde)]

Comment on attachment 581187 [details] [diff] [review]
fix

This means that we  will unconditionally link against oleacc.dll right?  Dows that have performance or other effects we should consider?

Comment 3

[Trevor Saunders (:tbsaunde)]

(asking roc for review since this touches widget/)

Comment 4

[Makoto Kato [:m_kato]]

(In reply to Trevor Saunders (:tbsaunde) from comment #2)
> Comment on attachment 581187 [details] [diff] [review]
> fix
> 
> This means that we  will unconditionally link against oleacc.dll right? 
> Dows that have performance or other effects we should consider?

oleacc.dll is already loaded on nspr4 and xul.dll's startup due to dependencies of shell32, comctl32 and others.

Comment 5

[alexander :surkov (:asurkov)]

Comment on attachment 581187 [details] [diff] [review]
fix

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

sorry for being slow, r=me

::: accessible/src/msaa/nsAccessibleWrap.h
@@ -333,5 @@
>  
> -  // NT4 does not have the oleacc that defines these methods. So we define copies here that automatically
> -  // load the library only if needed.
> -  static STDMETHODIMP AccessibleObjectFromWindow(HWND hwnd,DWORD dwObjectID,REFIID riid,void **ppvObject);
> -  static STDMETHODIMP NotifyWinEvent(DWORD event,HWND hwnd,LONG idObjectType,LONG idObject);

Shouldn't you use '::' prefix for methods like these? That's what we usually do for windows API calls.

Comment 6

[alexander :surkov (:asurkov)]

try server build - http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-2d6066684fcf

Marco, can you make sure everything is ok?

Comment 7

[Marco Zehe (:MarcoZ)]

All good, virtual buffers are created just as normal with this try-server build.

Comment 8

[alexander :surkov (:asurkov)]

inbound land https://hg.mozilla.org/integration/mozilla-inbound/rev/fb8fb19f6161

Comment 9

[Masatoshi Kimura [:emk]]

(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).

Comment 10

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/fb8fb19f6161