776024 - GCC 4.6 build failure after landings of bug 715814

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Compile error

I pulled from inbound and the landings from bug 715814 caused build failures on my machine.

I'm running Ubuntu 12.04 x64, and have gcc version 4.6.3 (Ubuntu/Linaro 4.6.3-1ubuntu5).

Error is:

nsLayoutModule.cpp
In file included from /home/cpearce/src/mozilla/inbound/layout/build/nsLayoutModule.cpp:84:0:
/home/cpearce/src/mozilla/inbound/netwerk/base/src/nsSimpleURI.h:37:58: warning: ‘virtual bool nsSimpleURI::Read(const IPC::Message*, void**)’ was hidden [-Woverloaded-virtual]
/home/cpearce/src/mozilla/inbound/dom/src/jsurl/nsJSProtocolHandler.h:82:62: warning:   by ‘virtual nsresult nsJSURI::Read(nsIObjectInputStream*)’ [-Woverloaded-virtual]
/home/cpearce/src/mozilla/inbound/netwerk/base/src/nsSimpleURI.h:37:155: warning: ‘virtual void nsSimpleURI::Write(IPC::Message*)’ was hidden [-Woverloaded-virtual]
/home/cpearce/src/mozilla/inbound/dom/src/jsurl/nsJSProtocolHandler.h:83:62: warning:   by ‘virtual nsresult nsJSURI::Write(nsIObjectOutputStream*)’ [-Woverloaded-virtual]
/home/cpearce/src/mozilla/inbound/layout/build/nsLayoutModule.cpp: In function ‘nsresult ActivityConstructor(nsISupports*, const nsIID&, void**)’:
/home/cpearce/src/mozilla/inbound/layout/build/nsLayoutModule.cpp:253:232: error: use of deleted function ‘mozilla::dom::Activity::Activity()’
In file included from /home/cpearce/src/mozilla/inbound/layout/build/nsLayoutModule.cpp:93:0:
../../dist/include/mozilla/dom/Activity.h:21:7: error: ‘mozilla::dom::Activity::Activity()’ is implicitly deleted because the default definition would be ill-formed:
../../dist/include/mozilla/dom/Activity.h:21:7: error: no matching function for call to ‘mozilla::dom::DOMRequest::DOMRequest()’
../../dist/include/mozilla/dom/Activity.h:21:7: note: candidates are:
../../dist/include/mozilla/dom/DOMRequest.h:44:3: note: mozilla::dom::DOMRequest::DOMRequest(nsIDOMWindow*)
../../dist/include/mozilla/dom/DOMRequest.h:44:3: note:   candidate expects 1 argument, 0 provided
../../dist/include/mozilla/dom/DOMRequest.h:20:7: note: mozilla::dom::DOMRequest::DOMRequest(const mozilla::dom::DOMRequest&)
../../dist/include/mozilla/dom/DOMRequest.h:20:7: note:   candidate expects 1 argument, 0 provided


Build output is attached.

This looks similar to bug 689301.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch

This fixes the compile error in GCC 4.6.

Comment 2

[Mounir Lamouri (:mounir)]

Comment on attachment 644447 [details] [diff] [review]
Patch

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

r=me with the nits addressed

::: dom/activities/src/Activity.cpp
@@ +76,5 @@
>      mProxy->Cleanup();
>    }
>  }
> +
> +Activity::Activity() : DOMRequest() {

nit: coding style wants:
Activity::Activity()
  : DOMRequest()
{
}

::: dom/activities/src/Activity.h
@@ +32,5 @@
>    // nsIJSNativeInitializer
>    NS_IMETHOD Initialize(nsISupports* aOwner, JSContext* aContext,
>                          JSObject* aObject, PRUint32 aArgc, jsval* aArgv);
>  
> +  Activity();

nit: can you leave a blank line between "Activity()" and "protected:"?

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c6e2e6537d70

Comment 4

[Ryan VanderMeulen [:RyanVM]]

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