The default bug view has changed. See this FAQ.

Build bustage 2017-03-09: RefPtr.h:40:5: error: invalid use of incomplete type 'class mozilla::DOMMediaStream'

RESOLVED FIXED in mozilla55

Status

()

Core
Web Speech
RESOLVED FIXED
15 days ago
15 days ago

People

(Reporter: Jorg K (GMT+1), Assigned: Jorg K (GMT+1))

Tracking

Trunk
mozilla55
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

15 days ago
Compilation seems to fail on an M-C file with some strange errors varying from platform to platform:

Linux:
mozilla/RefPtr.h:40:5: error: invalid use of incomplete type 'class mozilla::DOMMediaStream'
mozilla/dom/AudioContext.h:38:7: error: forward declaration of 'class mozilla::DOMMediaStream'

Mac:
mozilla/RefPtr.h:40:9: error: member access into incomplete type 'mozilla::DOMMediaStream'

Windows:
mozilla/RefPtr.h(40): error C2027: use of undefined type 'mozilla::DOMMediaStream'
mozilla/RefPtr.h(40): error C2227: left of '->Release' must point to class/struct/union/generic type

I did a clobber, but that didn't fix anything.
(Assignee)

Comment 1

15 days ago
Richard reports to be able to compile with M-C at 120c713a857f but not further. Here from the M-C pushlog:

e2a70a374a1c	Hiroyuki Ikezoe — Bug 1344619 - Part 1: Introduce WillCompose() to update various state that we need to update as the result of ComposeStyle(). r=birtles

120c713a857f	Zibi Braniecki — Bug 1337694 - Add language negotiation heuristics to LocaleService. r=jfkthame
(Assignee)

Comment 2

15 days ago
Hmm, from the compiler log (Mac):

-fno-omit-frame-pointer  -Werror   /builds/slave/tb-c-cen-m64-00000000000000000/build/mailnews/base/util/nsMsgIncomingServer.cpp
In file included from /builds/slave/tb-c-cen-m64-00000000000000000/build/objdir-tb/dom/bindings/UnifiedBindings18.cpp:2:
In file included from /builds/slave/tb-c-cen-m64-00000000000000000/build/objdir-tb/dom/bindings/SVGUnitTypeValuesBinding.cpp:3:
In file included from /builds/slave/tb-c-cen-m64-00000000000000000/build/objdir-tb/dist/include/mozilla/dom/NonRefcountedDOMObject.h:10:
In file included from /builds/slave/tb-c-cen-m64-00000000000000000/build/objdir-tb/dist/include/nsISupportsImpl.h:14:
In file included from /builds/slave/tb-c-cen-m64-00000000000000000/build/objdir-tb/dist/include/nsISupportsUtils.h:15:
/builds/slave/tb-c-cen-m64-00000000000000000/build/objdir-tb/dist/include/mozilla/RefPtr.h:40:9: error: member access into incomplete type 'mozilla::DOMMediaStream'
    aPtr->Release();
(Assignee)

Comment 3

15 days ago
However, on Windows I see:

c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/objdir-tb/_virtualenv/Scripts/python.exe c:/builds/moz2_slave/tb-c-cen-w32-00000000000000000/build/mozilla/config/expandlibs_gen.py -o dom_quota.lib.desc Unified_cpp_dom_quota0.obj  
UnifiedBindings18.cpp
c:\builds\moz2_slave\tb-c-cen-w32-00000000000000000\build\objdir-tb\dist\include\mozilla/RefPtr.h(40): error C2027: use of undefined type 'mozilla::DOMMediaStream'
c:\builds\moz2_slave\tb-c-cen-w32-00000000000000000\build\objdir-tb\dist\include\mozilla\dom\SpeechRecognition.h(35): note: see declaration of 'mozilla::DOMMediaStream'
c:\builds\moz2_slave\tb-c-cen-w32-00000000000000000\build\objdir-tb\dist\include\mozilla/RefPtr.h(39): note: while compiling class template member function 'void mozilla::RefPtrTraits<U>::Release(U *)'
        with
        [
            U=mozilla::DOMMediaStream
        ]

And on Linux:
[snip] -Werror   /builds/slave/tb-c-cen-l64-00000000000000000/build/mailnews/jsaccount/src/JaAbDirectory.cpp
/builds/slave/tb-c-cen-l64-00000000000000000/build/gcc/bin/../lib/gcc/x86_64-unknown-linux-gnu/4.9.4/../../../../x86_64-unknown-linux-gnu/bin/as: /lib64/libz.so.1: no version information available (required by /builds/slave/tb-c-cen-l64-00000000000000000/build/gcc/bin/../lib/gcc/x86_64-unknown-linux-gnu/4.9.4/../../../../x86_64-unknown-linux-gnu/bin/as)
In file included from /builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dist/include/nsISupportsUtils.h:15:0,
                 from /builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dist/include/nsISupportsImpl.h:14,
                 from /builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dist/include/mozilla/dom/NonRefcountedDOMObject.h:10,
                 from /builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dom/bindings/SVGUnitTypeValuesBinding.cpp:3,
                 from /builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dom/bindings/UnifiedBindings18.cpp:2:
/builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dist/include/mozilla/RefPtr.h: In instantiation of 'static void mozilla::RefPtrTraits<U>::Release(U*) [with U = mozilla::DOMMediaStream]':
/builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dist/include/mozilla/RefPtr.h:395:45:   required from 'static void RefPtr<T>::ConstRemovingRefPtrTraits<U>::Release(U*) [with U = mozilla::DOMMediaStream; T = mozilla::DOMMediaStream]'
/builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dist/include/mozilla/RefPtr.h:78:52:   required from 'RefPtr<T>::~RefPtr() [with T = mozilla::DOMMediaStream]'
/builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dist/include/mozilla/dom/SpeechRecognition.h:133:32:   required from here
/builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dist/include/mozilla/RefPtr.h:40:5: error: invalid use of incomplete type 'class mozilla::DOMMediaStream'
     aPtr->Release();

Hiro and Brian, can you please tell me what's going on here? Maybe also Cameron.
Flags: needinfo?(hikezoe)
Flags: needinfo?(cam)
Flags: needinfo?(bbirtles)
(In reply to Jorg K (GMT+1) from comment #3)
> /builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dist/include/
> mozilla/dom/SpeechRecognition.h:133:32:   required from here
> /builds/slave/tb-c-cen-l64-00000000000000000/build/objdir-tb/dist/include/
> mozilla/RefPtr.h:40:5: error: invalid use of incomplete type 'class
> mozilla::DOMMediaStream'
>      aPtr->Release();

I am guessing SpeechRecognition.h should include DOMMediaStream.h.
Flags: needinfo?(hikezoe)
(Assignee)

Comment 5

15 days ago
(In reply to Hiroyuki Ikezoe (:hiro) from comment #4)
> I am guessing SpeechRecognition.h should include DOMMediaStream.h.
That's all M-C, right, so who's going to fix that?
(Assignee)

Comment 6

15 days ago
Created attachment 8845310 [details] [diff] [review]
1345771.patch

Trying this now, will take an hour to complete if successful.
(Assignee)

Comment 7

15 days ago
Or maybe that should be: #include "mozilla/DOMMediaStream.h"
DOMMediaStream.h is listed in the EXPORTS section of moz.build rather than EXPORTS.mozilla, for some reason, so it's just "DOMMediaStream.h".
Flags: needinfo?(cam)
(Assignee)

Comment 9

15 days ago
Comment on attachment 8845310 [details] [diff] [review]
1345771.patch

OK, so since according to the previous comment this should be OK, maybe just r+ this and I can get it landed if it works. Or would you like me to move the line up or down? I heard it's meant to be alphabetical, but that's just a myth, right?
Attachment #8845310 - Flags: review?(hikezoe)
Attachment #8845310 - Flags: review?(cam)
Comment on attachment 8845310 [details] [diff] [review]
1345771.patch

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

Looks good, thanks.  I wouldn't worry about the order of the includes since it's all over the place at the moment.
Attachment #8845310 - Flags: review?(hikezoe)
Attachment #8845310 - Flags: review?(cam)
Attachment #8845310 - Flags: review+
(Assignee)

Comment 11

15 days ago
Thanks Cameron (fellow Ozzie?). I'll get it landed *if* it works ;-)
Component: General → DOM: Animation
Flags: needinfo?(bbirtles)
Product: Thunderbird → Core
Version: unspecified → Trunk
(In reply to Jorg K (GMT+1) from comment #11)
> Thanks Cameron (fellow Ozzie?). I'll get it landed *if* it works ;-)

Here it works.
(Assignee)

Comment 13

15 days ago
Dear Sheriff, please land with:
Bug 1345771 - Include DOMMediaStream.h in SpeechRecognition.h. r=heycam
Keywords: checkin-needed

Comment 14

15 days ago
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/c40ca7a1bdd9
Include DOMMediaStream.h in SpeechRecognition.h r=heycam a=tomcat
Keywords: checkin-needed
Fwiw, bustage also seen on http://buildbot.rhaalovely.net/builders/comm-central-amd64/builds/1948
Component: DOM: Animation → Web Speech
(Assignee)

Comment 16

15 days ago
Thanks, Tomcat!
Status: NEW → RESOLVED
Last Resolved: 15 days ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
And the build is indeed fixed too on BSD with this m-c push: http://buildbot.rhaalovely.net/builders/comm-central-amd64/builds/1949
(In reply to Pulsebot from comment #14)
> Pushed by cbook@mozilla.com:
> https://hg.mozilla.org/mozilla-central/rev/c40ca7a1bdd9
> Include DOMMediaStream.h in SpeechRecognition.h r=heycam a=tomcat

This was never merged to inbound?  I've been fighting bustage because of this today.
Assignee: nobody → jorgk
Its now been merged back to m-i.  Seems to solve my problems.  Sorry for the noise here.
You need to log in before you can comment on or make changes to this bug.