build(deps): bump bitflags from 1.3.2 to 2.3.1

Bumps [bitflags](https://github.com/bitflags/bitflags) from 1.3.2 to 2.3.1.
 - [Release notes](https://github.com/bitflags/bitflags/releases)
 - [Changelog](https://github.com/bitflags/bitflags/blob/main/CHANGELOG.md)
 - [Commits](bitflags/bitflags@1.3.2...2.3.1)

Cargo.toml

@@ -17,7 +17,7 @@ atomic_refcell = "0.1.11"
 arrayvec = "0.7"
 backtrace = "0.3"
 base64 = "0.21.3"
-bitflags = "1.0"
+bitflags = "2.3"
 byteorder = "1.0"
 encoding_rs = "0.8"
 euclid = "0.22"

components/gfx/font.rs

@@ -177,16 +177,17 @@ impl Font {
 }
 
 bitflags! {
+    #[derive(Clone, Copy, Debug, Eq, Hash, PartialEq)]
     pub struct ShapingFlags: u8 {
-        #[doc = "Set if the text is entirely whitespace."]
+        /// Set if the text is entirely whitespace.
         const IS_WHITESPACE_SHAPING_FLAG = 0x01;
-        #[doc = "Set if we are to ignore ligatures."]
+        /// Set if we are to ignore ligatures.
         const IGNORE_LIGATURES_SHAPING_FLAG = 0x02;
-        #[doc = "Set if we are to disable kerning."]
+        /// Set if we are to disable kerning.
         const DISABLE_KERNING_SHAPING_FLAG = 0x04;
-        #[doc = "Text direction is right-to-left."]
+        /// Text direction is right-to-left.
         const RTL_FLAG = 0x08;
-        #[doc = "Set if word-break is set to keep-all."]
+        /// Set if word-break is set to keep-all.
         const KEEP_ALL_FLAG = 0x10;
     }
 }

components/gfx_traits/lib.rs

@@ -57,10 +57,9 @@ impl StackingContextId {
 }
 
 int_range_index! {
-    #[derive(Deserialize, Serialize)]
-    #[doc = "An index that refers to a byte offset in a text run. This could \
-             point to the middle of a glyph."]
-    #[derive(MallocSizeOf)]
+    #[derive(Deserialize, MallocSizeOf, Serialize)]
+    /// An index that refers to a byte offset in a text run. This could
+    /// the middle of a glyph.
     struct ByteIndex(isize)
 }
 

components/layout/block.rs

@@ -603,10 +603,11 @@ pub struct BlockFlow {
 }
 
 bitflags! {
+    #[derive(Clone, Copy)]
     struct BlockFlowFlags: u8 {
-        #[doc = "If this is set, then this block flow is the root flow."]
+        /// If this is set, then this block flow is the root flow.
         const IS_ROOT = 0b0000_0001;
-        #[doc = "If this is set, then this block flow has overflow and it will scroll."]
+        /// If this is set, then this block flow has overflow and it will scroll.
         const HAS_SCROLLING_OVERFLOW = 0b0000_0010;
     }
 }

components/layout/data.rs

@@ -51,10 +51,11 @@ impl Default for LayoutData {
 }
 
 bitflags! {
+    #[derive(Clone, Copy)]
     pub struct LayoutDataFlags: u8 {
-        #[doc = "Whether a flow has been newly constructed."]
+        /// Whether a flow has been newly constructed.
         const HAS_NEWLY_CONSTRUCTED_FLOW = 0x01;
-        #[doc = "Whether this node has been traversed by layout."]
+        /// Whether this node has been traversed by layout.
         const HAS_BEEN_TRAVERSED = 0x02;
     }
 }

components/layout/display_list/builder.rs

@@ -2223,6 +2223,7 @@ impl Fragment {
 }
 
 bitflags! {
+    #[derive(Clone, Copy)]
     pub struct StackingContextCollectionFlags: u8 {
         /// This flow never establishes a containing block.
         const POSITION_NEVER_CREATES_CONTAINING_BLOCK = 0b001;

components/layout/flow.rs

@@ -613,43 +613,43 @@ impl FlowClass {
 }
 
 bitflags! {
-    #[doc = "Flags used in flows."]
+    /// Flags used in flows.
+    #[derive(Clone, Copy, Debug)]
     pub struct FlowFlags: u32 {
-        // text align flags
-        #[doc = "Whether this flow is absolutely positioned. This is checked all over layout, so a"]
-        #[doc = "virtual call is too expensive."]
+        /// Whether this flow is absolutely positioned. This is checked all over layout, so a
+        /// virtual call is too expensive.
         const IS_ABSOLUTELY_POSITIONED = 0b0000_0000_0000_0000_0100_0000;
-        #[doc = "Whether this flow clears to the left. This is checked all over layout, so a"]
-        #[doc = "virtual call is too expensive."]
+        /// Whether this flow clears to the left. This is checked all over layout, so a
+        /// virtual call is too expensive.
         const CLEARS_LEFT = 0b0000_0000_0000_0000_1000_0000;
-        #[doc = "Whether this flow clears to the right. This is checked all over layout, so a"]
-        #[doc = "virtual call is too expensive."]
+        /// Whether this flow clears to the right. This is checked all over layout, so a
+        /// virtual call is too expensive.
         const CLEARS_RIGHT = 0b0000_0000_0000_0001_0000_0000;
-        #[doc = "Whether this flow is left-floated. This is checked all over layout, so a"]
-        #[doc = "virtual call is too expensive."]
+        /// Whether this flow is left-floated. This is checked all over layout, so a
+        /// virtual call is too expensive.
         const FLOATS_LEFT = 0b0000_0000_0000_0010_0000_0000;
-        #[doc = "Whether this flow is right-floated. This is checked all over layout, so a"]
-        #[doc = "virtual call is too expensive."]
+        /// Whether this flow is right-floated. This is checked all over layout, so a
+        /// virtual call is too expensive.
         const FLOATS_RIGHT = 0b0000_0000_0000_0100_0000_0000;
-        #[doc = "Text alignment. \
-                 \
-                 NB: If you update this, update `TEXT_ALIGN_SHIFT` below."]
+        /// Text alignment.
+        ///
+        /// NB: If you update this, update `TEXT_ALIGN_SHIFT` below.
         const TEXT_ALIGN = 0b0000_0000_0111_1000_0000_0000;
-        #[doc = "Whether this flow has a fragment with `counter-reset` or `counter-increment` \
-                 styles."]
+        /// Whether this flow has a fragment with `counter-reset` or `counter-increment`
+        /// styles.
         const AFFECTS_COUNTERS = 0b0000_0000_1000_0000_0000_0000;
-        #[doc = "Whether this flow's descendants have fragments that affect `counter-reset` or \
-                 `counter-increment` styles."]
+        /// Whether this flow's descendants have fragments that affect `counter-reset` or
+        //  `counter-increment` styles.
         const HAS_COUNTER_AFFECTING_CHILDREN = 0b0000_0001_0000_0000_0000_0000;
-        #[doc = "Whether this flow behaves as though it had `position: static` for the purposes \
-                 of positioning in the inline direction. This is set for flows with `position: \
-                 static` and `position: relative` as well as absolutely-positioned flows with \
-                 unconstrained positions in the inline direction."]
+        /// Whether this flow behaves as though it had `position: static` for the purposes
+        /// of positioning in the inline direction. This is set for flows with `position:
+        /// static` and `position: relative` as well as absolutely-positioned flows with
+        /// unconstrained positions in the inline direction."]
         const INLINE_POSITION_IS_STATIC = 0b0000_0010_0000_0000_0000_0000;
-        #[doc = "Whether this flow behaves as though it had `position: static` for the purposes \
-                 of positioning in the block direction. This is set for flows with `position: \
-                 static` and `position: relative` as well as absolutely-positioned flows with \
-                 unconstrained positions in the block direction."]
+        /// Whether this flow behaves as though it had `position: static` for the purposes
+        /// of positioning in the block direction. This is set for flows with `position:
+        /// static` and `position: relative` as well as absolutely-positioned flows with
+        /// unconstrained positions in the block direction.
         const BLOCK_POSITION_IS_STATIC = 0b0000_0100_0000_0000_0000_0000;
 
         /// Whether any ancestor is a fragmentation container

components/layout/fragment.rs

@@ -534,6 +534,7 @@ pub struct ScannedTextFragmentInfo {
 }
 
 bitflags! {
+    #[derive(Clone, Copy)]
     pub struct ScannedTextFlags: u8 {
         /// Whether a line break is required after this fragment if wrapping on newlines (e.g. if
         /// `white-space: pre` is in effect).
@@ -3328,10 +3329,10 @@ bitflags! {
     // Various flags we can use when splitting fragments. See
     // `calculate_split_position_using_breaking_strategy()`.
     struct SplitOptions: u8 {
-        #[doc = "True if this is the first fragment on the line."]
+        /// True if this is the first fragment on the line."]
         const STARTS_LINE = 0x01;
-        #[doc = "True if we should attempt to split at character boundaries if this split fails. \
-                 This is used to implement `overflow-wrap: break-word`."]
+        /// True if we should attempt to split at character boundaries if this split fails. \
+        /// This is used to implement `overflow-wrap: break-word`."]
         const RETRY_AT_CHARACTER_BOUNDARIES = 0x02;
     }
 }
@@ -3446,6 +3447,7 @@ impl Overflow {
 }
 
 bitflags! {
+    #[derive(Clone, Debug)]
     pub struct FragmentFlags: u8 {
         // TODO(stshine): find a better name since these flags can also be used for grid item.
         /// Whether this fragment represents a child in a row flex container.

components/layout/incremental.rs

@@ -16,8 +16,8 @@ pub enum RelayoutMode {
 
 bitflags! {
     pub struct SpecialRestyleDamage: u8 {
-        #[doc = "If this flag is set, we need to reflow the entire document. This is more or less a \
-                 temporary hack to deal with cases that we don't handle incrementally yet."]
+        /// If this flag is set, we need to reflow the entire document. This is more or less a
+        /// temporary hack to deal with cases that we don't handle incrementally yet.
         const REFLOW_ENTIRE_DOCUMENT = 0x01;
     }
 }

components/layout/inline.rs

@@ -221,7 +221,7 @@ impl Line {
 
 int_range_index! {
     #[derive(Serialize)]
-    #[doc = "The index of a fragment in a flattened vector of DOM elements."]
+    /// The index of a fragment in a flattened vector of DOM elements.
     struct FragmentIndex(isize)
 }
 
@@ -2053,6 +2053,7 @@ pub struct InlineFragmentNodeInfo {
 }
 
 bitflags! {
+    #[derive(Clone)]
     pub struct InlineFragmentNodeFlags: u8 {
         const FIRST_FRAGMENT_OF_ELEMENT = 0x01;
         const LAST_FRAGMENT_OF_ELEMENT = 0x02;

components/layout_2020/fragment_tree/base_fragment.rs

@@ -72,10 +72,10 @@ impl From<BaseFragmentInfo> for BaseFragment {
 }
 
 bitflags! {
-    #[doc = "Flags used to track various information about a DOM node during layout."]
-    #[derive(Serialize)]
+    /// Flags used to track various information about a DOM node during layout.
+    #[derive(Clone, Copy, Debug, Serialize)]
     pub(crate) struct FragmentFlags: u8 {
-        #[doc = "Whether or not this node is a body element on an HTML document."]
+        /// Whether or not this node is a body element on an HTML document.
         const IS_BODY_ELEMENT_OF_HTML_ELEMENT_ROOT = 0b00000001;
     }
 }

components/script/dom/bindings/codegen/CodegenRust.py

@@ -7964,7 +7964,7 @@ def InterfaceObjectMap(config):
         global_flags = CGWrapper(CGIndenter(CGList([
             CGGeneric("const %s = %#x;" % args)
             for args in flags
-        ], "\n")), pre="pub struct Globals: u8 {\n", post="\n}")
+        ], "\n")), pre="#[derive(Clone, Copy)]\npub struct Globals: u8 {\n", post="\n}")
         globals_ = CGWrapper(CGIndenter(global_flags), pre="bitflags! {\n", post="\n}")
 
         phf = CGGeneric("include!(concat!(env!(\"OUT_DIR\"), \"/InterfaceObjectMapPhf.rs\"));")

components/script/dom/htmliframeelement.rs

@@ -43,9 +43,11 @@ use servo_url::ServoUrl;
 use std::cell::Cell;
 use style::attr::{AttrValue, LengthOrPercentageOrAuto};
 
+#[derive(Clone, Copy, JSTraceable, MallocSizeOf)]
+struct SandboxAllowance(u8);
+
 bitflags! {
-    #[derive(JSTraceable, MallocSizeOf)]
-    struct SandboxAllowance: u8 {
+    impl SandboxAllowance: u8 {
         const ALLOW_NOTHING = 0x00;
         const ALLOW_SAME_ORIGIN = 0x01;
         const ALLOW_TOP_NAVIGATION = 0x02;

components/script/dom/node.rs

@@ -157,28 +157,30 @@ pub struct Node {
     style_and_layout_data: DomRefCell<Option<Box<StyleAndOpaqueLayoutData>>>,
 }
 
+/// Flags for node items
+#[derive(Clone, Copy, JSTraceable, MallocSizeOf)]
+pub struct NodeFlags(u16);
+
 bitflags! {
-    #[doc = "Flags for node items."]
-    #[derive(JSTraceable, MallocSizeOf)]
-    pub struct NodeFlags: u16 {
-        #[doc = "Specifies whether this node is in a document."]
+    impl NodeFlags: u16 {
+        /// Specifies whether this node is in a document.
         const IS_IN_DOC = 1 << 0;
 
-        #[doc = "Specifies whether this node needs style recalc on next reflow."]
+        /// Specifies whether this node needs style recalc on next reflow.
         const HAS_DIRTY_DESCENDANTS = 1 << 1;
 
-        #[doc = "Specifies whether or not there is an authentic click in progress on \
-                 this element."]
+        /// Specifies whether or not there is an authentic click in progress on
+        /// this element.
         const CLICK_IN_PROGRESS = 1 << 2;
 
-        #[doc = "Specifies whether this node is focusable and whether it is supposed \
-                 to be reachable with using sequential focus navigation."]
+        /// Specifies whether this node is focusable and whether it is supposed
+        /// to be reachable with using sequential focus navigation."]
         const SEQUENTIALLY_FOCUSABLE = 1 << 3;
 
         // There are two free bits here.
 
-        #[doc = "Specifies whether the parser has set an associated form owner for \
-                 this element. Only applicable for form-associatable elements."]
+        /// Specifies whether the parser has set an associated form owner for
+        /// this element. Only applicable for form-associatable elements.
         const PARSER_ASSOCIATED_FORM_OWNER = 1 << 6;
 
         /// Whether this element has a snapshot stored due to a style or

components/script/dom/validitystate.rs

@@ -20,9 +20,11 @@ use std::fmt;
 use style::element_state::ElementState;
 
 // https://html.spec.whatwg.org/multipage/#validity-states
+#[derive(Clone, Copy, JSTraceable, MallocSizeOf)]
+pub struct ValidationFlags(u32);
+
 bitflags! {
-    #[derive(JSTraceable, MallocSizeOf)]
-    pub struct ValidationFlags: u32 {
+    impl ValidationFlags: u32 {
         const VALUE_MISSING    = 0b0000000001;
         const TYPE_MISMATCH    = 0b0000000010;
         const PATTERN_MISMATCH = 0b0000000100;