676924 - uint64/JSUint64 conflict leading to build failure in dom/base/nsGlobalWindow.cpp and dom/indexedDB/IDBObjectStore.cpp

Status: RESOLVED FIXED

(Core :: Storage: IndexedDB, defect)

Description

[Landry Breuil (:gaston)]

Probably since landing of http://hg.mozilla.org/mozilla-central/rev/6647ba316a10 adding the use of uint64 type, builds fails for me with the following :

/var/buildslave-mozilla/comm-central-amd64/build/mozilla/dom/base/nsGlobalWindow.cpp: In member function 'void PostMessageEvent::SetJSData(JSAutoStructuredCloneBuffer&)':
/var/buildslave-mozilla/comm-central-amd64/build/mozilla/dom/base/nsGlobalWindow.cpp:5944: error: invalid conversion from 'uint64**' to 'JSUint64**'
/var/buildslave-mozilla/comm-central-amd64/build/mozilla/dom/base/nsGlobalWindow.cpp:5944: error:   initializing argument 1 of 'void JSAutoStructuredCloneBuffer::steal(JSUint64**, size_t*, JSUint32*)'
/var/buildslave-mozilla/comm-central-amd64/build/mozilla/dom/base/nsGlobalWindow.cpp: In member function 'virtual nsresult PostMessageEvent::Run()':
/var/buildslave-mozilla/comm-central-amd64/build/mozilla/dom/base/nsGlobalWindow.cpp:5991: error: invalid conversion from 'uint64*' to 'JSUint64*'
/var/buildslave-mozilla/comm-central-amd64/build/mozilla/dom/base/nsGlobalWindow.cpp:5991: error:   initializing argument 1 of 'void JSAutoStructuredCloneBuffer::adopt(JSUint64*, size_t, JSUint32)'
gmake[6]: *** [nsGlobalWindow.o] Error 1

Full log at http://buildbot.rhaalovely.net/builders/comm-central-amd64/builds/96/steps/build/logs/stdio

That's probably related to an uint64/JSUint64 mismatch.. and somewhat related to https://bugzilla.mozilla.org/show_bug.cgi?id=662852

Comment 1

[Landry Breuil (:gaston)]

Attachment: Potential fix i'm testing

Comment 2

[Landry Breuil (:gaston)]

With that patch, the build goes further, but fails later with :

/var/buildslave-mozilla/mozilla-central-amd64/build/dom/indexedDB/IDBObjectStore.cpp: In static member function 'static nsresult mozilla::dom::indexedDB::IDBObjectStore::GetStructuredCloneDataFromStatement(mozIStorageStatement*, PRUint32, JSAutoStructuredCloneBuffer&)':
/var/buildslave-mozilla/mozilla-central-amd64/build/dom/indexedDB/IDBObjectStore.cpp:838: error: invalid conversion from 'const uint64*' to 'const JSUint64*'
/var/buildslave-mozilla/mozilla-central-amd64/build/dom/indexedDB/IDBObjectStore.cpp:838: error:   initializing argument 1 of 'bool JSAutoStructuredCloneBuffer::copy(const JSUint64*, size_t, JSUint32)'
gmake[6]: *** [IDBObjectStore.o] Error 1

Adding another fix on top of the previous...

Comment 3

[Landry Breuil (:gaston)]

Attachment: second fix, this time replacing uint64 by uint64_t in IDBObjectStore.cpp

Build passes fine in dom/base and dom/indexedDB with this patch, will confirm if the full build is ok..

Comment 4

[Landry Breuil (:gaston)]

As can be seen on http://buildbot.rhaalovely.net/builders/mozilla-central-amd64/builds/119, the build is successful with the patch fixing those two issues.. after that, it might not be the perfect patch though.

Comment 5

[Landry Breuil (:gaston)]

And if that can give another data point, it started failing 7 days ago, http://buildbot.rhaalovely.net/builders/mozilla-central-amd64/builds/110 was successful, http://buildbot.rhaalovely.net/builders/mozilla-central-amd64/builds/111 was the first time this particular failure showed.

Comment 6

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 551158 [details] [diff] [review]
second fix, this time replacing uint64 by uint64_t in IDBObjectStore.cpp

r=me

Awesome that you have a buildbot setup for OpenBSD.

Comment 7

[Landry Breuil (:gaston)]

Not sure it's the best patch (i'm still lost in all those typedefs) or if it's worth a try run, but setting checkin-needed.

And yes, that buildbot is quite handy, but needs time to be babysitted.. it's very rare to get it all green :)

Comment 8

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/0d36e53e837d