Update inappropriate ComVisible attributes

src/System.Windows.Forms.Design/src/Properties/InternalsVisibleTo.cs → src/System.Windows.Forms.Design/src/Properties/AssemblyInfo.cs

@@ -4,6 +4,8 @@
 
 using System.Runtime.CompilerServices;
 
+[assembly: System.Runtime.InteropServices.ComVisible(false)]
+
 // expose internal types to System.Design for type forwarding
 [assembly: InternalsVisibleTo("System.Design, PublicKey=002400000480000094000000060200000024000052534131000400000100010007d1fa57c4aed9f0a32e84aa0faefd0de9e8fd6aec8f87fb03766c834c99921eb23be79ad9d5dcc1dd9ad236132102900b723cf980957fc4e177108fc607774f29e8320e92ea05ece4e821c0a5efe8f1645c4c0c93c1ab99285d622caa652c1dfad63d745d6f2de5f17e5eaf0fc4963d261c8a12436518206dc093344d5ad293")]
 

src/System.Windows.Forms.Design/src/System/ComponentModel/Design/DesignerActionListCollection.cs

@@ -7,7 +7,6 @@
 
 namespace System.ComponentModel.Design
 {
-    [ComVisible(true)]
     public class DesignerActionListCollection : CollectionBase
     {
         public DesignerActionListCollection()

src/System.Windows.Forms.Design/src/System/ComponentModel/Design/DesignerActionListsChangedEventHandler.cs

@@ -9,6 +9,5 @@ namespace System.ComponentModel.Design
     /// <summary>
     ///  This event is raised by the <see cref="DesignerActionService"/> when a shortcut is either added or removed to/from the related object.
     /// </summary>
-    [ComVisible(true)]
     public delegate void DesignerActionListsChangedEventHandler(object sender, DesignerActionListsChangedEventArgs e);
 }

src/System.Windows.Forms.Design/src/System/ComponentModel/Design/DesignerActionListsChangedType.cs

@@ -9,7 +9,6 @@ namespace System.ComponentModel.Design
     /// <summary>
     ///  An enum that defines what type of action happend to the related object's <see cref="DesignerActionListCollection">designer action lists collection</see>.
     /// </summary>
-    [ComVisible(true)]
     public enum DesignerActionListsChangedType
     {
         /// <summary>

src/System.Windows.Forms.Design/src/System/ComponentModel/Design/MenuCommandsChangedEventArgs.cs

@@ -9,7 +9,6 @@ namespace System.ComponentModel.Design
     ///     that there has been a change in MenuCommands (added or removed)
     ///     on the related object.
     /// </summary>
-    [System.Runtime.InteropServices.ComVisible(true)]
     public class MenuCommandsChangedEventArgs : EventArgs
     {
         /// <summary>

src/System.Windows.Forms.Design/src/System/ComponentModel/Design/MenuCommandsChangedEventHandler.cs

@@ -8,7 +8,6 @@ namespace System.ComponentModel.Design
     ///     This event is thown by the MenuCommandService when a shortcut
     ///     is either added or removed to/from the related object.
     /// </summary>
-    [System.Runtime.InteropServices.ComVisible(true)]
     public delegate void MenuCommandsChangedEventHandler(object sender, MenuCommandsChangedEventArgs e);
 }
 

src/System.Windows.Forms.Design/src/System/ComponentModel/Design/MenuCommandsChangedType.cs

@@ -8,7 +8,6 @@ namespace System.ComponentModel.Design
     ///     An enum that defines what time of action happend to the
     ///     related object's MenuCommands collection.
     /// </summary>
-    [System.Runtime.InteropServices.ComVisible(true)]
     public enum MenuCommandsChangedType
     {
         /// <summary>

src/System.Windows.Forms.Design/src/System/Drawing/Design/ColorEditor.cs

@@ -552,7 +552,6 @@ private void SetFocus(Point newFocus)
                 }
             }
 
-            [ComVisible(true)]
             public class ColorPaletteAccessibleObject : ControlAccessibleObject
             {
                 private readonly ColorCellAccessibleObject[] cells;
@@ -598,7 +597,6 @@ public override AccessibleObject HitTest(int x, int y)
                     return base.HitTest(x, y);
                 }
 
-                [ComVisible(true)]
                 public class ColorCellAccessibleObject : AccessibleObject
                 {
                     private readonly Color color;

src/System.Windows.Forms.Design/src/System/Windows/Forms/Design/ControlDesigner.cs

@@ -2358,7 +2358,6 @@ private int GetParentPointFromLparam(IntPtr lParam)
             return PARAM.ToInt(pt.X, pt.Y);
         }
 
-        [ComVisible(true)]
         public class ControlDesignerAccessibleObject : AccessibleObject
         {
             private readonly ControlDesigner _designer = null;

src/System.Windows.Forms.Design/src/System/Windows/Forms/Design/DesignerOptions.cs

@@ -11,7 +11,6 @@ namespace System.Windows.Forms.Design
     /// <summary>
     ///  Provides access to get and set option values for a designer.
     /// </summary>
-    [ComVisible(true)]
     public class DesignerOptions
     {
         private const int MinGridSize = 2;

src/System.Windows.Forms.Design/src/System/Windows/Forms/Design/ISupportInSituService.cs

@@ -7,7 +7,6 @@ namespace System.Windows.Forms.Design
     /// <summary>
     ///  Provides an interface for a designer to support Insitu editing for selected components.
     /// </summary>
-    [Runtime.InteropServices.ComVisible(true)]
     internal interface ISupportInSituService
     {
         /// <summary>

src/System.Windows.Forms.Primitives/src/Properties/InternalsVisibleTo.cs → src/System.Windows.Forms.Primitives/src/Properties/AssemblyInfo.cs

@@ -4,6 +4,8 @@
 
 using System.Runtime.CompilerServices;
 
+[assembly: System.Runtime.InteropServices.ComVisible(false)]
+
 [assembly: InternalsVisibleTo("System.Windows.Forms, PublicKey=00000000000000000400000000000000")]
 [assembly: InternalsVisibleTo("System.Windows.Forms.Design, PublicKey=00000000000000000400000000000000")]
 [assembly: InternalsVisibleTo("System.Windows.Forms.Design.Editors, PublicKey=00000000000000000400000000000000")]

src/System.Windows.Forms.Primitives/tests/UnitTests/Interop/Mocks/MockAxHost.cs

@@ -10,8 +10,6 @@
 
 namespace System.Windows.Forms.Primitives.Tests.Interop.Mocks
 {
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     internal class MockAxHost
     {
         private static Guid ipictureDisp_Guid = typeof(IPictureDisp).GUID;

src/System.Windows.Forms/src/Properties/InternalsVisibleTo.cs → src/System.Windows.Forms/src/Properties/AssemblyInfo.cs

@@ -4,6 +4,8 @@
 
 using System.Runtime.CompilerServices;
 
+[assembly: System.Runtime.InteropServices.ComVisible(false)]
+
 [assembly: InternalsVisibleTo("System.Windows.Forms.Tests, PublicKey=00000000000000000400000000000000")]
 [assembly: InternalsVisibleTo("MauiMonthCalendarTests, PublicKey=00000000000000000400000000000000")]
 

src/System.Windows.Forms/src/System/Windows/Forms/AccessibleObject.cs

@@ -18,7 +18,6 @@ namespace System.Windows.Forms
     ///  Provides an implementation for an object that can be inspected by an
     ///  accessibility application.
     /// </summary>
-    [ComVisible(true)]
     public partial class AccessibleObject :
         StandardOleMarshalObject,
         IReflect,

src/System.Windows.Forms/src/System/Windows/Forms/Appearance.cs

@@ -9,7 +9,6 @@ namespace System.Windows.Forms
     /// <summary>
     ///  Specifies the appearance of a control.
     /// </summary>
-    [ComVisible(true)]
     public enum Appearance
     {
         /// <summary>

src/System.Windows.Forms/src/System/Windows/Forms/ArrangeDirection.cs

@@ -10,7 +10,6 @@ namespace System.Windows.Forms
     /// <summary>
     ///  Specifies the direction the system uses to arrange minimized windows.
     /// </summary>
-    [ComVisible(true)]
     [Flags]
     public enum ArrangeDirection
     {

src/System.Windows.Forms/src/System/Windows/Forms/AxHost.AxComponentEditor.cs

@@ -14,7 +14,6 @@ namespace System.Windows.Forms
 {
     public abstract partial class AxHost
     {
-        [ComVisible(false)]
         public class AxComponentEditor : WindowsFormsComponentEditor
         {
             public override bool EditComponent(ITypeDescriptorContext context, object obj, IWin32Window parent)

src/System.Windows.Forms/src/System/Windows/Forms/AxHost.cs

@@ -28,8 +28,6 @@ namespace System.Windows.Forms
     /// <summary>
     ///  Wraps ActiveX controls and exposes them as fully featured windows forms controls.
     /// </summary>
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     [ToolboxItem(false)]
     [DesignTimeVisible(false)]
     [DefaultEvent(nameof(Enter))]

src/System.Windows.Forms/src/System/Windows/Forms/BindingNavigator.cs

@@ -11,8 +11,6 @@
 
 namespace System.Windows.Forms
 {
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     [DefaultProperty(nameof(BindingSource))]
     [DefaultEvent(nameof(RefreshItems))]
     [Designer("System.Windows.Forms.Design.BindingNavigatorDesigner, " + AssemblyRef.SystemDesign)]

src/System.Windows.Forms/src/System/Windows/Forms/Border3DSide.cs

@@ -10,7 +10,6 @@ namespace System.Windows.Forms
     /// <summary>
     ///  Specifies the sides of a rectangle to apply a three-dimensional border to.
     /// </summary>
-    [ComVisible(true)]
     [Flags]
     public enum Border3DSide
     {

src/System.Windows.Forms/src/System/Windows/Forms/Border3DStyle.cs

@@ -10,7 +10,6 @@ namespace System.Windows.Forms
     /// <summary>
     ///  Specifies the style of a three-dimensional border.
     /// </summary>
-    [ComVisible(true)]
     public enum Border3DStyle
     {
         /// <summary>

src/System.Windows.Forms/src/System/Windows/Forms/BorderStyle.cs

@@ -9,7 +9,6 @@ namespace System.Windows.Forms
     /// <summary>
     ///  Specifies the border style for a control or form.
     /// </summary>
-    [ComVisible(true)]
     public enum BorderStyle
     {
         /// <summary>

src/System.Windows.Forms/src/System/Windows/Forms/Button.cs

@@ -18,8 +18,6 @@ namespace System.Windows.Forms
     ///  Represents a
     ///  Windows button.
     /// </summary>
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     [SRDescription(nameof(SR.DescriptionButton))]
     [Designer("System.Windows.Forms.Design.ButtonBaseDesigner, " + AssemblyRef.SystemDesign)]
     public class Button : ButtonBase, IButtonControl

src/System.Windows.Forms/src/System/Windows/Forms/ButtonBase.cs

@@ -18,8 +18,6 @@ namespace System.Windows.Forms
     /// <summary>
     ///  Implements the basic functionality required by a button control.
     /// </summary>
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     [Designer("System.Windows.Forms.Design.ButtonBaseDesigner, " + AssemblyRef.SystemDesign)]
     public abstract class ButtonBase : Control
     {
@@ -1352,7 +1350,6 @@ protected override void WndProc(ref Message m)
             }
         }
 
-        [ComVisible(true)]
         public class ButtonBaseAccessibleObject : ControlAccessibleObject
         {
             public ButtonBaseAccessibleObject(Control owner) : base(owner)

src/System.Windows.Forms/src/System/Windows/Forms/CheckBox.cs

@@ -18,8 +18,6 @@ namespace System.Windows.Forms
     ///  Represents a Windows
     ///  check box.
     /// </summary>
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     [DefaultProperty(nameof(Checked))]
     [DefaultEvent(nameof(CheckedChanged))]
     [DefaultBindingProperty(nameof(CheckState))]
@@ -648,7 +646,6 @@ public override string ToString()
             return s + ", CheckState: " + checkState.ToString(CultureInfo.InvariantCulture);
         }
 
-        [ComVisible(true)]
         public class CheckBoxAccessibleObject : ButtonBaseAccessibleObject
         {
             public CheckBoxAccessibleObject(Control owner) : base(owner)

src/System.Windows.Forms/src/System/Windows/Forms/CheckedListBox.cs

@@ -22,8 +22,6 @@ namespace System.Windows.Forms
     ///
     ///  of each item.
     /// </summary>
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     [LookupBindingProperties]
     [SRDescription(nameof(SR.DescriptionCheckedListBox))]
     public class CheckedListBox : ListBox
@@ -1466,7 +1464,6 @@ internal void SetCheckedState(int index, CheckState value)
 
         internal override bool SupportsUiaProviders => false;
 
-        [ComVisible(true)]
         internal class CheckedListBoxAccessibleObject : ControlAccessibleObject
         {
             /// <summary>
@@ -1567,7 +1564,6 @@ public override AccessibleObject Navigate(AccessibleNavigation direction)
             }
         }
 
-        [ComVisible(true)]
         internal class CheckedListBoxItemAccessibleObject : AccessibleObject
         {
             private string name;

src/System.Windows.Forms/src/System/Windows/Forms/ComboBox.cs

@@ -27,8 +27,6 @@ namespace System.Windows.Forms
     ///  list or to enter new text. Displays only the editing field until the user
     ///  explicitly displays the list.
     /// </summary>
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     [DefaultEvent(nameof(SelectedIndexChanged))]
     [DefaultProperty(nameof(Items))]
     [DefaultBindingProperty(nameof(Text))]
@@ -3943,7 +3941,6 @@ protected override void WndProc(ref Message m)
             }
         }
 
-        [ComVisible(true)]
         private class ComboBoxChildNativeWindow : NativeWindow
         {
             private readonly ComboBox _owner;
@@ -4535,7 +4532,6 @@ internal void SetItemInternal(int index, object value)
             }
         } // end ObjectCollection
 
-        [ComVisible(true)]
         public class ChildAccessibleObject : AccessibleObject
         {
             readonly ComboBox owner;
@@ -4560,7 +4556,6 @@ public override string Name
         /// <summary>
         ///  Represents the ComboBox item accessible object.
         /// </summary>
-        [ComVisible(true)]
         internal class ComboBoxItemAccessibleObject : AccessibleObject
         {
             private readonly ComboBox _owningComboBox;
@@ -4876,7 +4871,6 @@ public int GetId(object item)
         ///  This inherits from the base ComboBoxExAccessibleObject and ComboBoxAccessibleObject
         ///  to have all base functionality.
         /// </summary>
-        [ComVisible(true)]
         internal class ComboBoxAccessibleObject : ControlAccessibleObject
         {
             private const int COMBOBOX_ACC_ITEM_INDEX = 1;
@@ -5342,7 +5336,6 @@ internal override int[] RuntimeId
         /// <summary>
         ///  Represents the ComboBox's child (inner) list native window control accessible object with UI Automation provider functionality.
         /// </summary>
-        [ComVisible(true)]
         internal class ComboBoxChildListUiaProvider : ChildAccessibleObject
         {
             private const string COMBO_BOX_LIST_AUTOMATION_ID = "1000";
@@ -5584,7 +5577,6 @@ public override AccessibleStates State
         /// <summary>
         ///  Represents the ComboBox's child text (is used instead of inner Edit when style is DropDownList but not DropDown) accessible object.
         /// </summary>
-        [ComVisible(true)]
         internal class ComboBoxChildTextUiaProvider : AccessibleObject
         {
             private const int COMBOBOX_TEXT_ACC_ITEM_INDEX = 1;
@@ -5759,7 +5751,6 @@ public override AccessibleStates State
         /// <summary>
         ///  Represents the ComboBox child (inner) DropDown button accessible object with UI Automation functionality.
         /// </summary>
-        [ComVisible(true)]
         internal class ComboBoxChildDropDownButtonUiaProvider : AccessibleObject
         {
             private const int COMBOBOX_DROPDOWN_BUTTON_ACC_ITEM_INDEX = 2;

src/System.Windows.Forms/src/System/Windows/Forms/ContainerControl.cs

@@ -18,8 +18,6 @@ namespace System.Windows.Forms
     /// <summary>
     ///  Defines a base class for controls that can parent other controls.
     /// </summary>
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     public class ContainerControl : ScrollableControl, IContainerControl
     {
         private Control _activeControl;

src/System.Windows.Forms/src/System/Windows/Forms/ContextMenuStrip.cs

@@ -13,8 +13,6 @@ namespace System.Windows.Forms
     /// <summary>
     ///  This class is just a conceptual wrapper around ToolStripDropDownMenu.
     /// </summary>
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     [DefaultEvent(nameof(Opening))]
     [SRDescription(nameof(SR.DescriptionContextMenuStrip))]
     public class ContextMenuStrip : ToolStripDropDownMenu

src/System.Windows.Forms/src/System/Windows/Forms/Control.ControlAccessibleObject.cs

@@ -16,7 +16,6 @@ public partial class Control
         /// <summary>
         ///  An implementation of AccessibleChild for use with Controls
         /// </summary>
-        [ComVisible(true)]
         public class ControlAccessibleObject : AccessibleObject
         {
             private static IntPtr s_oleAccAvailable = NativeMethods.InvalidIntPtr;

src/System.Windows.Forms/src/System/Windows/Forms/Control.ControlCollection.cs

@@ -18,7 +18,7 @@ public partial class Control
         /// <summary>
         ///  Collection of controls...
         /// </summary>
-        [ListBindable(false), ComVisible(false)]
+        [ListBindable(false)]
         public class ControlCollection : ArrangedElementCollection, IList, ICloneable
         {
             ///  A caching mechanism for key accessor

src/System.Windows.Forms/src/System/Windows/Forms/Control.cs

@@ -35,8 +35,6 @@ namespace System.Windows.Forms
     /// Do not add instance variables to Control absolutely necessary. Every control on a form has the overhead of
     /// all of these variables.
     /// </remarks>
-    [ComVisible(true)]
-    [ClassInterface(ClassInterfaceType.AutoDispatch)]
     [DefaultProperty(nameof(Text))]
     [DefaultEvent(nameof(Click))]
     [Designer("System.Windows.Forms.Design.ControlDesigner, " + AssemblyRef.SystemDesign)]

src/System.Windows.Forms/src/System/Windows/Forms/DataGridView.AccessibleObject.cs

@@ -12,7 +12,6 @@ namespace System.Windows.Forms
 {
     public partial class DataGridView
     {
-        [ComVisible(true)]
         protected class DataGridViewAccessibleObject : ControlAccessibleObject
         {
             private int[] runtimeId = null; // Used by UIAutomation

src/System.Windows.Forms/src/System/Windows/Forms/DataGridView.ControlCollection.cs

@@ -11,7 +11,6 @@ namespace System.Windows.Forms
 {
     public partial class DataGridView
     {
-        [ComVisible(false)]
         public class DataGridViewControlCollection : ControlCollection
         {
             readonly DataGridView owner;

src/System.Windows.Forms/src/System/Windows/Forms/DataGridView.TopRowAccessibleObject.cs

@@ -13,7 +13,6 @@ namespace System.Windows.Forms
 {
     public partial class DataGridView
     {
-        [ComVisible(true)]
         protected class DataGridViewTopRowAccessibleObject : AccessibleObject
         {
             private int[] runtimeId;