1221067 - error: no matching function for call to 'NS_NewStreamLoader'

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[aleth [:aleth]]

Build bustage:

/builds/slave/tb-c-cen-lx-ntly-0000000000000/build/mailnews/addrbook/src/nsAbContentHandler.cpp:129:43: error: no matching function for call to 'NS_NewStreamLoader(nsGetterAddRefs<nsIStreamLoader>, nsCOMPtr<nsIURI>&, nsAbContentHandler* const, nsCOMPtr<nsIPrincipal>&, nsILoadInfo::<anonymous enum>, nsIContentPolicyBase::<anonymous enum>, nsIInterfaceRequestor*&)'

FYI there is an older (fixed) bug that may be similar, Bug 1071497.

Comment 1

[aleth [:aleth]]

also on OSX:
/builds/slave/tb-c-cen-m64-ntly-000000000000/build/mailnews/addrbook/src/nsAbContentHandler.cpp:123:10: error: no matching function for call to 'NS_NewStreamLoader'

on Windows:
c:/builds/moz2_slave/tb-c-cen-w32-ntly-000000000000/build/mailnews/addrbook/src/nsAbContentHandler.cpp(129) : error C2665: 'NS_NewStreamLoader' : none of the 3 overloads could convert all the argument types

Comment 2

[aleth [:aleth]]

Likely fallout from bug 1206964.

Comment 3

[Philip Chee]

Attachment: Patch v1.0 fix NS_NewStreamLoader

Comment 4

[Philip Chee]

Comment on attachment 8682537 [details] [diff] [review]
Patch v1.0 fix NS_NewStreamLoader

Bustage fix

Comment 5

[Magnus Melin [:mkmelin]]

Comment on attachment 8682537 [details] [diff] [review]
Patch v1.0 fix NS_NewStreamLoader

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

Looks good, thx! r=mkmelin

Comment 6

[aleth [:aleth]]

https://hg.mozilla.org/comm-central/rev/f3388da5d1a5da2e6972d0c76b31e668b0bd64ae
Bug 1221067 - error: no matching function for call to 'NS_NewStreamLoader'. r=mkmelin, a=bustage fix on CLOSED TREE

Comment 7

[aleth [:aleth]]

https://hg.mozilla.org/mozilla-central/rev/c6eb37d7c93e also seems to change the nsIContentPolicy and the nsILoadInfo argument in some cases, are you sure that's not required here?

Comment 9

[Philip Chee]

(In reply to aleth [:aleth] from comment #7)
> https://hg.mozilla.org/mozilla-central/rev/c6eb37d7c93e also seems to change
> the nsIContentPolicy and the nsILoadInfo argument in some cases, are you
> sure that's not required here?
I just did the minimum necessary. If it isn't broke don't fix it.
Also why would you want a content blocker (like ABP) block v-cards?
Also I didn't think it's worth the effort to create a new TYPE_VCARD just for one caller.

Comment 10

[aleth [:aleth]]

(In reply to Philip Chee from comment #9)
> (In reply to aleth [:aleth] from comment #7)
> > https://hg.mozilla.org/mozilla-central/rev/c6eb37d7c93e also seems to change
> > the nsIContentPolicy and the nsILoadInfo argument in some cases, are you
> > sure that's not required here?
> I just did the minimum necessary. If it isn't broke don't fix it.
> Also why would you want a content blocker (like ABP) block v-cards?
> Also I didn't think it's worth the effort to create a new TYPE_VCARD just
> for one caller.

No worries, I don't know this code, I just thought I'd ask ;)