1538694 - Simplify reference counted FFI types.

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1538694 - Cleanup refcounted types. r=boris

And make the handling of ComputedStyle more similar to these.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f1eec8f7a3f8
Cleanup refcounted types. r=boris

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/563e8fb1be5d
Try to appease windows builds.

Comment 4

[Bogdan Tara[:bogdan_tara | bogdant]]

Backed out 4 changesets (Bug 1536584, Bug 1538661, Bug 1538694) for causing build bustages CLOSED TREE

Pushes with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&fromchange=73a48619739e34609d11f5ed15f3fc1436ee154f&tochange=105411e3cf06b2fca98d4e0507efb80875103228

Backout link: https://hg.mozilla.org/integration/autoland/rev/105411e3cf06b2fca98d4e0507efb80875103228

Failure logs: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=236123244&repo=autoland&lineNumber=12901

13:52:02 INFO - mozmake.EXE[5]: Entering directory 'z:/build/build/src/obj-firefox/xpcom/reflect/xptinfo'
13:52:02 INFO - z:/build/build/src/clang/bin/clang.exe --driver-mode=cl -Foxptdata.i_o -c -Iz:/build/build/src/obj-firefox/dist/stl_wrappers -DNDEBUG=1 -DTRIMMED=1 -DSTATIC_EXPORTABLE_JS_API -DMOZ_HAS_MOZGLUE -DMOZILLA_INTERNAL_API -DIMPL_LIBXUL -Iz:/build/build/src/xpcom/reflect/xptinfo -Iz:/build/build/src/obj-firefox/xpcom/reflect/xptinfo -Iz:/build/build/src/obj-firefox/dist/include -Iz:/build/build/src/obj-firefox/dist/include/nspr -Iz:/build/build/src/obj-firefox/dist/include/nss -MD -FI z:/build/build/src/obj-firefox/mozilla-config.h -DMOZILLA_CLIENT -Qunused-arguments -guard:cf -Qunused-arguments -fcrash-diagnostics-dir=z:/build/public/build -TP -nologo -Zc:sizedDealloc- -D_HAS_EXCEPTIONS=0 -guard:cf -W3 -Gy -Zc:inline -Gw -Wno-inline-new-delete -Wno-invalid-offsetof -Wno-microsoft-enum-value -Wno-microsoft-include -Wno-unknown-pragmas -Wno-ignored-pragmas -Wno-deprecated-declarations -Wno-invalid-noreturn -Wno-inconsistent-missing-override -Wno-implicit-exception-spec-mismatch -Wno-unused-local-typedef -Wno-ignored-attributes -Wno-used-but-marked-unused -D_SILENCE_TR1_NAMESPACE_DEPRECATION_WARNING -GR- -Z7 -fno-limit-debug-info -Xclang -load -Xclang z:/build/build/src/obj-firefox/build/clang-plugin/clang-plugin.dll -Xclang -add-plugin -Xclang moz-check -O2 -Oy- -WX -fprofile-instr-generate -Xclang -finstrument-functions-after-inlining -Xclang -MP -Xclang -dependency-file -Xclang .deps/xptdata.i_o.pp -Xclang -MT -Xclang xptdata.i_o z:/build/build/src/obj-firefox/xpcom/reflect/xptinfo/xptdata.cpp
13:52:02 INFO - In file included from z:/build/build/src/obj-firefox/xpcom/reflect/xptinfo/xptdata.cpp:6:
13:52:02 INFO - In file included from z:/build/build/src/obj-firefox/dist/include\mozilla/dom/CharacterData.h:23:
13:52:02 INFO - In file included from z:/build/build/src/obj-firefox/dist/include\mozilla/dom/ShadowRoot.h:15:
13:52:02 INFO - z:/build/build/src/obj-firefox/dist/include\mozilla/ServoBindings.h(184,49): error: 'Servo_SelectorList_Parse' has C-linkage specified, but returns incomplete type 'mozilla::StyleOwnedOrNull<RawServoSelectorList>' which could be incompatible with C [-Werror,-Wreturn-type-c-linkage]
13:52:02 INFO - mozilla::StyleOwnedOrNull<RawServoSelectorList> Servo_SelectorList_Parse(
13:52:02 INFO - ^
13:52:02 INFO - 1 error generated.
13:52:02 INFO - z:/build/build/src/config/rules.mk:805: recipe for target 'xptdata.i_o' failed
13:52:02 INFO - mozmake.EXE[5]: *** [xptdata.i_o] Error 1
13:52:02 INFO - mozmake.EXE[5]: Leaving directory 'z:/build/build/src/obj-firefox/xpcom/reflect/xptinfo'
13:52:02 INFO - z:/build/build/src/config/recurse.mk:74: recipe for target 'xpcom/reflect/xptinfo/target' failed
13:52:02 INFO - mozmake.EXE[4]: *** [xpcom/reflect/xptinfo/target] Error 2
13:52:02 INFO - mozmake.EXE[4]: *** Waiting for unfinished jobs....
13:52:02 INFO - mozmake.EXE[5]: Entering directory 'z:/build/build/src/obj-firefox/xpcom/build'
13:52:02 INFO - xpcom/build/perfprobe.i_o
13:52:02 INFO - mozmake.EXE[5]: Leaving directory 'z:/build/build/src/obj-firefox/xpcom/build'
13:52:02 INFO - mozmake.EXE[5]: Entering directory 'z:/build/build/src/obj-firefox/config/external/icu/i18n'
13:52:02 INFO - config/external/icu/i18n/tznames_impl.i_o

The fix introduced linux bustages: https://hg.mozilla.org/integration/autoland/rev/563e8fb1be5db3fa6f984c34270a35c7cf1dd4e6

https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=236127756&repo=autoland&lineNumber=12289

[task 2019-03-26T14:23:53.701Z] 14:23:53 INFO - In file included from /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/ServoStyleSet.h:15:0,
[task 2019-03-26T14:23:53.701Z] 14:23:53 INFO - from /builds/worker/workspace/build/src/obj-firefox/dist/include/nsIPresShell.h:16,
[task 2019-03-26T14:23:53.701Z] 14:23:53 INFO - from /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/dom/Document.h:26,
[task 2019-03-26T14:23:53.701Z] 14:23:53 INFO - from /builds/worker/workspace/build/src/obj-firefox/dist/include/nsContentUtils.h:47,
[task 2019-03-26T14:23:53.701Z] 14:23:53 INFO - from /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/dom/ClientIPCUtils.h:14,
[task 2019-03-26T14:23:53.701Z] 14:23:53 INFO - from /builds/worker/workspace/build/src/obj-firefox/ipc/ipdl/_ipdlheaders/mozilla/dom/ClientIPCTypes.h:22,
[task 2019-03-26T14:23:53.701Z] 14:23:53 INFO - from /builds/worker/workspace/build/src/obj-firefox/ipc/ipdl/_ipdlheaders/mozilla/dom/PContent.h:38,
[task 2019-03-26T14:23:53.701Z] 14:23:53 INFO - from /builds/worker/workspace/build/src/obj-firefox/ipc/ipdl/_ipdlheaders/mozilla/dom/PContentChild.h:9,
[task 2019-03-26T14:23:53.702Z] 14:23:53 INFO - from /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/dom/ContentChild.h:13,
[task 2019-03-26T14:23:53.702Z] 14:23:53 INFO - from /builds/worker/workspace/build/src/security/sandbox/linux/broker/SandboxBrokerPolicyFactory.cpp:19,
[task 2019-03-26T14:23:53.702Z] 14:23:53 INFO - from /builds/worker/workspace/build/src/obj-firefox/security/sandbox/linux/broker/Unified_cpp_linux_broker0.cpp:20:
[task 2019-03-26T14:23:53.703Z] 14:23:53 ERROR - /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/ServoBindingTypes.h:164:32: error: unknown option after '#pragma GCC diagnostic' kind [-Werror=pragmas]
[task 2019-03-26T14:23:53.703Z] 14:23:53 INFO - #pragma GCC diagnostic ignored "-Wreturn-type-c-linkage"
[task 2019-03-26T14:23:53.703Z] 14:23:53 INFO - ^~~~~~~~~~~~~~~~~~~~~~~~~
[task 2019-03-26T14:23:53.703Z] 14:23:53 INFO - cc1plus: all warnings being treated as errors
[task 2019-03-26T14:23:53.703Z] 14:23:53 INFO - /builds/worker/workspace/build/src/config/rules.mk:805: recipe for target 'Unified_cpp_linux_broker0.o' failed
[task 2019-03-26T14:23:53.703Z] 14:23:53 ERROR - make[4]: *** [Unified_cpp_linux_broker0.o] Error 1
[task 2019-03-26T14:23:53.703Z] 14:23:53 INFO - make[4]: Leaving directory '/builds/worker/workspace/build/src/obj-firefox/security/sandbox/linux/broker'
[task 2019-03-26T14:23:53.704Z] 14:23:53 INFO - /builds/worker/workspace/build/src/config/recurse.mk:74: recipe for target 'security/sandbox/linux/broker/target' failed
[task 2019-03-26T14:23:53.704Z] 14:23:53 ERROR - make[3]: *** [security/sandbox/linux/broker/target] Error 2
[task 2019-03-26T14:23:53.704Z] 14:23:53 INFO - make[3]: *** Waiting for unfinished jobs....
[task 2019-03-26T14:23:53.704Z] 14:23:53 INFO - make[4]: Entering directory '/builds/worker/workspace/build/src/obj-firefox/xpcom/threads'

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/76a3365dabc8
Cleanup refcounted types. r=boris

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/76a3365dabc8

Comment 7

[Peter Van der Beken [:peterv]]

https://hg.mozilla.org/projects/ash/rev/f1eec8f7a3f87f9d8edbe3a2c5d6fa9c629eb6f2
Bug 1538694 - Cleanup refcounted types. r=boris

https://hg.mozilla.org/projects/ash/rev/76a3365dabc828137b58059681654ec3d417abce
Bug 1538694 - Cleanup refcounted types. r=boris