Closed
Bug 1172405
Opened 10 years ago
Closed 10 years ago
Build error when adding files to dom/workers and gfx/layers
Categories
(Core :: General, defect)
Core
General
Tracking
()
RESOLVED
FIXED
mozilla41
| Tracking | Status | |
|---|---|---|
| firefox41 | --- | fixed |
People
(Reporter: mtseng, Assigned: mtseng)
References
Details
Attachments
(1 file)
|
1.76 KB,
patch
|
nical
:
review+
|
Details | Diff | Splinter Review |
Bug 709490 add some files to dom/workers/ and gfx/layers/. After adding files I encounter build failed. The error log as following.
0:05.37 /Users/Morris/mozilla/gecko-dev/gfx/layers/YCbCrImageDataSerializer.cpp:285:7: error: use of undeclared identifier 'NS_WARN_IF'
0:05.37 if (NS_WARN_IF(!result)) {
0:05.37 ^
0:05.37 1 error generated.
0:06.87 In file included from /Users/Morris/mozilla/gecko-dev/obj-x86_64-apple-darwin14.3.0/dom/workers/Unified_cpp_dom_workers1.cpp:2:
0:06.87 /Users/Morris/mozilla/gecko-dev/dom/workers/ServiceWorkerContainer.cpp:100:8: error: use of undeclared identifier 'NS_NewURI'
0:06.87 rv = NS_NewURI(getter_AddRefs(scriptURI), aScriptURL, nullptr,
0:06.87 ^
0:06.87 /Users/Morris/mozilla/gecko-dev/dom/workers/ServiceWorkerContainer.cpp:113:19: error: use of undeclared identifier 'NS_NewURI'
0:06.87 nsresult rv = NS_NewURI(getter_AddRefs(scopeURI), NS_LITERAL_CSTRING("./"),
0:06.87 ^
0:06.87 /Users/Morris/mozilla/gecko-dev/dom/workers/ServiceWorkerContainer.cpp:118:19: error: use of undeclared identifier 'NS_NewURI'
0:06.87 nsresult rv = NS_NewURI(getter_AddRefs(scopeURI), aOptions.mScope.Value(),
0:06.87 ^
0:06.87 3 errors generated.
This is because unified build group those files to another group so the error occurs. I'll upload a patch to fix this error.
| Assignee | ||
Comment 1•10 years ago
|
||
| Assignee | ||
Updated•10 years ago
|
Attachment #8616596 -
Flags: review?(nical.bugzilla)
Updated•10 years ago
|
Attachment #8616596 -
Flags: review?(nical.bugzilla) → review+
| Assignee | ||
Comment 2•10 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=661a86f9ed8d
Build only try should be ok.
| Assignee | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Keywords: checkin-needed
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox41:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in
before you can comment on or make changes to this bug.
Description
•