rustdoc: correctly propagate cfgs for glob reexports

* rustdoc: correctly propagate cfgs for glob reexports
* address review feedback, add a comment explaining the test
* empty commit to trigger CI
* empty commit to trigger CI
* fix sorting of features in tests
* assert contents of index.html

Author: lcian

src/librustdoc/clean/mod.rs

@@ -243,8 +243,8 @@ fn generate_item_with_correct_attrs(
             ) || (is_glob_import(tcx, import_id)
                 && (cx.document_hidden() || !tcx.is_doc_hidden(def_id)))
                 || macro_reexport_is_inline(tcx, import_id, def_id);
-            attrs.extend(get_all_import_attributes(cx, import_id, def_id, is_inline));
             is_inline = is_inline || import_is_inline;
+            attrs.extend(get_all_import_attributes(cx, import_id, def_id, is_inline));
         }
         let keep_target_cfg = is_inline || matches!(kind, ItemKind::TypeAliasItem(..));
         add_without_unwanted_attributes(&mut attrs, target_attrs, keep_target_cfg, None);

tests/rustdoc-html/reexport/glob-reexport-feature-combination.rs

@@ -0,0 +1,67 @@
+// This test ensures that `cfg`s are correctly propagated for re-export chains
+// where the outermost is a glob re-export.
+
+#![crate_name = "foo"]
+#![feature(doc_cfg)]
+
+//@ has 'foo/index.html'
+//@ count - '//*[@class="item-table"]/dt' 3
+
+//@ has 'foo/index.html'
+//@ has - '//dt/a[@title="struct foo::A"]/../*[@class="stab portability"]' 'Non-bar and non-foo'
+
+//@ has 'foo/struct.A.html'
+//@ has - '//*[@id="main-content"]/*[@class="item-info"]/*[@class="stab portability"]' \
+//    'Available on non-crate feature bar and non-crate feature foo only.'
+
+mod a {
+    mod inner {
+        pub struct A {}
+    }
+    #[cfg(not(feature = "bar"))]
+    pub use self::inner::A;
+}
+#[cfg(not(feature = "foo"))]
+pub use a::*;
+
+//@ has 'foo/index.html'
+//@ has - '//dt/a[@title="struct foo::B"]/../*[@class="stab portability"]' 'Non-bar and non-baz and non-foo'
+
+//@ has 'foo/struct.B.html'
+//@ has - '//*[@id="main-content"]/*[@class="item-info"]/*[@class="stab portability"]' \
+//    'Available on non-crate feature bar and non-crate feature baz and non-crate feature foo only.'
+
+mod b {
+    mod inner {
+        mod innermost {
+            pub struct B {}
+        }
+        #[cfg(not(feature = "baz"))]
+        pub use self::innermost::B;
+    }
+    #[cfg(not(feature = "bar"))]
+    pub use self::inner::*;
+}
+#[cfg(not(feature = "foo"))]
+pub use b::*;
+
+//@ has 'foo/index.html'
+//@ has - '//dt/a[@title="struct foo::C"]/../*[@class="stab portability"]' 'Non-bar and non-baz and non-foo'
+
+//@ has 'foo/struct.C.html'
+//@ has - '//*[@id="main-content"]/*[@class="item-info"]/*[@class="stab portability"]' \
+//    'Available on non-crate feature bar and non-crate feature baz and non-crate feature foo only.'
+
+mod c {
+    mod inner {
+        mod innermost {
+            #[cfg(not(feature = "baz"))]
+            pub struct C {}
+        }
+        pub use self::innermost::*;
+    }
+    #[cfg(not(feature = "bar"))]
+    pub use self::inner::*;
+}
+#[cfg(not(feature = "foo"))]
+pub use c::*;