Closed
Bug 939569
Opened 11 years ago
Closed 11 years ago
Build layout/build in unified mode
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla28
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
(Whiteboard: [qa-])
Attachments
(2 files)
646 bytes,
patch
|
Details | Diff | Splinter Review | |
18.74 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Comment 3•11 years ago
|
||
Assignee | ||
Comment 4•11 years ago
|
||
Backed out because of Windows build bustage: https://hg.mozilla.org/integration/mozilla-inbound/rev/8b676ce79d10
https://tbpl.mozilla.org/php/getParsedLog.php?id=30660139&tree=Mozilla-Inbound
Assignee | ||
Comment 5•11 years ago
|
||
nsSynthVoiceRegistry.h uses #pragma once, which causes it to be included twice, once from the objdir and once from the source dir.
Assignee | ||
Comment 6•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Attachment #8333565 -
Flags: review?(bugs)
Comment 7•11 years ago
|
||
Comment on attachment 8333565 [details] [diff] [review]
Stop using #pragma once in webspeech code
Hmm, is any of the headers exported as mozilla/dom/Foo.h ?
If so, the define should have mozilla_dom_ prefix.
Check that please.
Attachment #8333565 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 8•11 years ago
|
||
OK, I'll do that before landing this.
Assignee | ||
Comment 9•11 years ago
|
||
Comment 10•11 years ago
|
||
(In reply to Ehsan Akhgari from comment #5)
> nsSynthVoiceRegistry.h uses #pragma once, which causes it to be included
> twice, once from the objdir and once from the source dir.
Someone's using #include "nsSynthVoiceRegistry.h" instead of #include "mozilla/dom/nsSynthVoiceRegistry.h"? (Why does it begin with ns if it lives in mozilla/dom anyway?)
Comment 11•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/896f22d74e73
https://hg.mozilla.org/mozilla-central/rev/b4bb0aebdaad
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Updated•11 years ago
|
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•