726576 - Optimize calls to nsLocalFileWin's HasFileAttribute

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Brian R. Bondy [:bbondy]]

Attachment: Patch v1.

HasFileAttribute does 2 calls to get the file attributes, one from ResolveAndStat and the other from GetFileAttributesW right after it.
This can be optimized to do only one by using Resolve instead.

Also this function doesn't check for INVALID_FILE_ATTRIBUTES so if ResolveAndStat was not dirty, and the state changed so the file got deleted, this function would return incorrect results before. 

Also the code in IsFile and IsDirectory can be simplified as Neil suggested previously to use HasFileAttribute instead.

Comment 1

[[e-mail]]

Comment on attachment 596670 [details] [diff] [review]
Patch v1.

>-  if (INVALID_FILE_ATTRIBUTES == attributes) {
Bah, I hadn't noticed that you like your comparisons the wrong way around ;-)

>+    DWORD attributes = GetFileAttributesW(mResolvedPath.get());
::GetFileAttributesW ?

Comment 2

[Brian R. Bondy [:bbondy]]

> Bah, I hadn't noticed that you like your comparisons the wrong way around ;-)

I'm pretty indifferent about it but I've been asked to switch it after some review comments so I just start off that way usually now :)

> ::GetFileAttributesW ?

I think we're moving away from ::, but if you want I can do it.

Comment 3

[Brian R. Bondy [:bbondy]]

and by we I mean me :)

Comment 4

[Brian R. Bondy [:bbondy]]

Attachment: Patch v2.

Returned converted windows error code instead of always returning invalid path since it was causing try failures. Carrying forward r+.

Comment 5

[Brian R. Bondy [:bbondy]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/52917f8cfc41

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/52917f8cfc41