See bug 666693 comment 22. *** https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=830df2eb8cec MSVC10 SeaMonkey succeeds. https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=2c02ab33151a MSVC8 SeaMonkey fails.

Fixes reported (= no more, no less) errors in Try logs. Succeeded as (SeaMonkey on MSVC8) https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=7d0401e10421

Comment on attachment 631152 [details] [diff] [review] (Av1) Add 'mozilla::dom::indexedDB::' namespace to unconfuse MSVC8 Review of attachment 631152 [details] [diff] [review]: ----------------------------------------------------------------- I really don't like all the extra goop this adds... Can we instead add a 'using namespace mozilla::dom::indexedDB::ipc;" to the top of these files, then remove the "ipc::" each place we use one of these types?

(In reply to ben turner [:bent] from comment #2) > Comment on attachment 631152 [details] [diff] [review] > (Av1) Add 'mozilla::dom::indexedDB::' namespace to unconfuse MSVC8 > > Review of attachment 631152 [details] [diff] [review]: > ----------------------------------------------------------------- > > I really don't like all the extra goop this adds... Can we instead add a > 'using namespace mozilla::dom::indexedDB::ipc;" to the top of these files, > then remove the "ipc::" each place we use one of these types? Ben, this is completely burning us, (and all local developers on MSVC8EE which is still supported!) any chance we can land the existing patch, and do the change you suggest in a followup? (fwiw I disagree with the disambiguity in the namespace in the first place ;-) )

Pick one, please

Comment on attachment 631737 [details] [diff] [review] Bigger patch [Checked in: Comment 6 & 9] Review of attachment 631737 [details] [diff] [review]: ----------------------------------------------------------------- If you've verified that this compiles, r=me.

Comment on attachment 631737 [details] [diff] [review] Bigger patch [Checked in: Comment 6 & 9] When this was likely a one or two line patch I had conditional-on-green a+ from alex for aurora (to get SeaMonkey/local-devs fixed) But since its bigger I figure an explicit approval is necessary. [Approval Request Comment] Bug caused by (feature/regressing bug #): exposed/caused by Bug 666693 User impact if declined: MSVC8 devs cannot builds, SeaMonkey Aurora cannot build on windows. Testing completed (on m-c, etc.): Checked into m-c today. (SeaMonkey machines have a longer end-to-end, so unsure there yet, but will hold off landing until I am sure that this fixes us) Risk to taking this patch (and alternatives if risky): None. String or UUID changes made by this patch: None

swapping assignee to author of accepted patch

Comment on attachment 631737 [details] [diff] [review] Bigger patch [Checked in: Comment 6 & 9] Bug 761635 - "[MSVC8] "dom/indexedDB/IDBCursor.cpp(42) : error C2872: 'ipc' : ambiguous symbol" after bug 666693" r=khuey a=khuey (Kyle claims this is simple enough to not need a 'real' approval since its a bustage fix for SeaMonkey. -- Taking him up on that statement) http://hg.mozilla.org/releases/mozilla-aurora/rev/bacdd7be41ff

http://tinderbox.mozilla.org/showbuilds.cgi?tree=SeaMonkey&maxdate=1339448638&hours=24&legend=0&norules=1 V.Fixed http://tinderbox.mozilla.org/showbuilds.cgi?tree=SeaMonkey-Aurora&maxdate=1339533421&hours=24&legend=0&norules=1 firefox15: verified.