1376647 - use new MayHaveAnonymousChildren flag to skip work in AllChildrenIterator and some util functions

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement)

Comment 1

[Cameron McCormack (:heycam)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6966c4cb2d518767d792a665c095ee1321eb9f2c

Comment 2

[Cameron McCormack (:heycam)]

Attachment: Bug 1376647 - Use new MayHaveAnonymousChildren flag to skip work in AllChildrenIterator and some util functions.

Review commit: https://reviewboard.mozilla.org/r/152792/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/152792/

Comment 3

[Bobby Holley (:bholley)]

Comment on attachment 8881627 [details]
Bug 1376647 - Use new MayHaveAnonymousChildren flag to skip work in AllChildrenIterator and some util functions.

https://reviewboard.mozilla.org/r/152792/#review157920

Comment 4

[Cameron McCormack (:heycam)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=837435cf63760128c0d3ae493f2c956271362bb8

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ee62473b968c
Use new MayHaveAnonymousChildren flag to skip work in AllChildrenIterator and some util functions. r=bholley

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Backed out for build bustage: no member named 'MayHaveAnonymousChildren' in 'nsIContent' at dom/base/nsContentUtils.cpp:10258:

https://hg.mozilla.org/integration/autoland/rev/de69c85231614841598c2b8a6a6f651646b60592

Push with bustage: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=ee62473b968c825e7a5d2d9aac19906def2be3f0&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable
Build log: https://treeherder.mozilla.org/logviewer.html#?job_id=110493402&repo=autoland

> /home/worker/workspace/build/src/dom/base/nsContentUtils.cpp:10258:17: error: no member named 'MayHaveAnonymousChildren' in 'nsIContent'

Comment 7

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

So bug 1376647 landed before bug 1372061 and that caused the bustage - at least it looks like gone after that push at the moment.

Comment 8

[Cameron McCormack (:heycam)]

Thanks, sorry I must have clicked the autoland buttons in the wrong order.  (Or because autoland was closed at the time it landed them in a different order from what I clicked?)

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/11c7c5432a1a
Use new MayHaveAnonymousChildren flag to skip work in AllChildrenIterator and some util functions. r=bholley

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/11c7c5432a1a