Closed
Bug 1332631
Opened 7 years ago
Closed 7 years ago
Rearrange/remove files from xpcom/glue
Categories
(Core :: XPCOM, defect)
Core
XPCOM
Tracking
()
RESOLVED
FIXED
mozilla54
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: benjamin, Unassigned)
References
Details
Attachments
(8 files)
59 bytes,
text/x-review-board-request
|
froydnj
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
froydnj
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
froydnj
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
froydnj
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
froydnj
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
froydnj
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
froydnj
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
froydnj
:
review+
|
Details |
Now that the XPCOM glue isn't a thing, it's confusing to have a bunch of files still in the xpcom/glue directory. This bug tracks moving these files to other locations. Most of this is really simple. nsStringAPI/nsStringGlue is hard. I'll probably split those out into a separate bug/patch because there are dependencies in webrtc.
Reporter | ||
Comment 1•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a7e8848760bee2e82b762efabb6f4739ec98fed2
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 10•7 years ago
|
||
mozreview-review |
Comment on attachment 8829420 [details] Bug 1332631 part A - remove two files which aren't referenced or used, https://reviewboard.mozilla.org/r/106526/#review107648
Attachment #8829420 -
Flags: review?(nfroyd) → review+
Comment 11•7 years ago
|
||
mozreview-review |
Comment on attachment 8829421 [details] Bug 1332631 part B - file moves from xpcom/glue to xpcom/base, https://reviewboard.mozilla.org/r/106528/#review107650 ::: xpcom/base/nsSystemInfo.cpp:80 (Diff revision 1) > // system call to discover the appropriate value is not thread-safe, > // so we must call it before going multithreaded, but nsSystemInfo::Init > // only happens well after that point. > uint32_t nsSystemInfo::gUserUmask = 0; > > +using namespace mozilla::dom; Why is this necessary?
Attachment #8829421 -
Flags: review?(nfroyd) → review+
Comment 12•7 years ago
|
||
mozreview-review |
Comment on attachment 8829422 [details] Bug 1332631 part C - file moves from xpcom/glue to xpcom/ds, https://reviewboard.mozilla.org/r/106530/#review107652
Attachment #8829422 -
Flags: review?(nfroyd) → review+
Comment 13•7 years ago
|
||
mozreview-review |
Comment on attachment 8829423 [details] Bug 1332631 part D - file moves from xpcom/glue to xpcom/threads, https://reviewboard.mozilla.org/r/106532/#review107654
Attachment #8829423 -
Flags: review?(nfroyd) → review+
Comment 14•7 years ago
|
||
mozreview-review |
Comment on attachment 8829424 [details] Bug 1332631 part E - file moves from xpcom/glue to xpcom/string, https://reviewboard.mozilla.org/r/106534/#review107656
Attachment #8829424 -
Flags: review?(nfroyd) → review+
Comment 15•7 years ago
|
||
mozreview-review |
Comment on attachment 8829425 [details] Bug 1332631 part F - move nsXPTCUtils.h to xpcom/reflect/xptcall, https://reviewboard.mozilla.org/r/106536/#review107662
Attachment #8829425 -
Flags: review?(nfroyd) → review+
Comment 16•7 years ago
|
||
mozreview-review |
Comment on attachment 8829426 [details] Bug 1332631 part G - file moves from xpcom/glue to xpcom/components, and remove GenericModule which is no longer useful, https://reviewboard.mozilla.org/r/106538/#review107664
Attachment #8829426 -
Flags: review?(nfroyd) → review+
Comment 17•7 years ago
|
||
mozreview-review |
Comment on attachment 8829427 [details] Bug 1332631 part H - move gtests from xpcom/glue to xpcom/tests, and unify the two TestThreadUtils.cpp into one file, https://reviewboard.mozilla.org/r/106540/#review107666 I did not look closely at the new content; I am assuming that your cut-and-paste buttons work OK. ::: xpcom/tests/gtest/TestThreadUtils.cpp:386 (Diff revision 1) > +TEST(ThreadUtils, TypeTraits) > +{ I am unsure that a gtest with no actual `EXPECT_*` or `ASSERT_*` calls is going to work very well.
Attachment #8829427 -
Flags: review?(nfroyd) → review+
Reporter | ||
Comment 18•7 years ago
|
||
> > +using namespace mozilla::dom; > > Why is this necessary? It fixes a unified build bustage because of a type from PContent which is used in this file. > I am unsure that a gtest with no actual `EXPECT_*` or `ASSERT_*` calls is > going to work very well. Indeed I'm not sure this needs to be a TEST. It's basically just a bunch of static_asserts. But it does pass and was strictly copied, so I think I'll keep it for now.
Comment 19•7 years ago
|
||
Pushed by bsmedberg@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/affff812d187 part A - remove two files which aren't referenced or used, r=froydnj https://hg.mozilla.org/integration/mozilla-inbound/rev/19391089d475 part B - file moves from xpcom/glue to xpcom/base, r=froydnj https://hg.mozilla.org/integration/mozilla-inbound/rev/a3af628d69b3 part C - file moves from xpcom/glue to xpcom/ds, r=froydnj https://hg.mozilla.org/integration/mozilla-inbound/rev/cade3cd4cc6f part D - file moves from xpcom/glue to xpcom/threads, r=froydnj https://hg.mozilla.org/integration/mozilla-inbound/rev/6434aad0c30c part E - file moves from xpcom/glue to xpcom/string, r=froydnj https://hg.mozilla.org/integration/mozilla-inbound/rev/c6dea69a5022 part F - move nsXPTCUtils.h to xpcom/reflect/xptcall, r=froydnj https://hg.mozilla.org/integration/mozilla-inbound/rev/f30012f9ed37 part G - file moves from xpcom/glue to xpcom/components, and remove GenericModule which is no longer useful, r=froydnj https://hg.mozilla.org/integration/mozilla-inbound/rev/43fabd330b94 part H - move gtests from xpcom/glue to xpcom/tests, and unify the two TestThreadUtils.cpp into one file, r=froydnj
Comment 20•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/affff812d187 https://hg.mozilla.org/mozilla-central/rev/19391089d475 https://hg.mozilla.org/mozilla-central/rev/a3af628d69b3 https://hg.mozilla.org/mozilla-central/rev/cade3cd4cc6f https://hg.mozilla.org/mozilla-central/rev/6434aad0c30c https://hg.mozilla.org/mozilla-central/rev/c6dea69a5022 https://hg.mozilla.org/mozilla-central/rev/f30012f9ed37 https://hg.mozilla.org/mozilla-central/rev/43fabd330b94
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in
before you can comment on or make changes to this bug.
Description
•