218820 - remove useless nsIExternalHelperAppService::canHandleContent

Status: RESOLVED FIXED

(Core Graveyard :: File Handling, defect, P3)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

it always returns PR_FALSE anyway

Comment 1

[Christian :Biesinger (don't email me, ping me on IRC)]

->me

Comment 2

[Christian :Biesinger (don't email me, ping me on IRC)]

Attachment: patch

Comment 3

[Boris Zbarsky [:bzbarsky]]

What about embedders who don't want to use our helper app service and want to
roll their own?

Then again, I guess this is never called...

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

hm... well, those have to deal with an api change now :)
mozilla never calls this function.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 131170 [details] [diff] [review]
patch

OK.  r=bzbarsky.

Comment 6

[Darin Fisher]

Comment on attachment 131170 [details] [diff] [review]
patch

sr=darin

Comment 7

[Christian :Biesinger (don't email me, ping me on IRC)]

Checking in nsExternalHelperAppService.cpp;
/cvsroot/mozilla/uriloader/exthandler/nsExternalHelperAppService.cpp,v  <-- 
nsExternalHelperAppService.cpp
new revision: 1.216; previous revision: 1.215
done
Checking in nsIExternalHelperAppService.idl;
/cvsroot/mozilla/uriloader/exthandler/nsIExternalHelperAppService.idl,v  <-- 
nsIExternalHelperAppService.idl
new revision: 1.13; previous revision: 1.12
done