Removing 'primary' from W3C-compliant pointer actions

Author: jimevans

dotnet/src/webdriver/Interactions/Actions.cs

@@ -29,7 +29,7 @@ public class Actions : IAction
     {
         private IWebDriver driver;
         private ActionBuilder actionBuilder = new ActionBuilder();
-        private PointerInputDevice defaultMouse = new PointerInputDevice(PointerKind.Mouse, true, "default mouse");
+        private PointerInputDevice defaultMouse = new PointerInputDevice(PointerKind.Mouse, "default mouse");
         private KeyInputDevice defaultKeyboard = new KeyInputDevice("default keyboard");
 
         private IKeyboard keyboard;

dotnet/src/webdriver/Interactions/PointerInputDevice.cs

@@ -95,37 +95,33 @@ internal enum MouseButton
     internal class PointerInputDevice : InputDevice
     {
         private PointerKind pointerKind;
-        private bool isPrimary;
 
         /// <summary>
         /// Initializes a new instance of the <see cref="PointerInputDevice"/> class.
         /// </summary>
         public PointerInputDevice()
-            : this(PointerKind.Mouse, true)
+            : this(PointerKind.Mouse)
         {
         }
 
         /// <summary>
         /// Initializes a new instance of the <see cref="PointerInputDevice"/> class.
         /// </summary>
         /// <param name="pointerKind">The kind of pointer represented by this input device.</param>
-        /// <param name="isPrimary">A value indicating whether the pointer device is the primary pointer device.</param>
-        public PointerInputDevice(PointerKind pointerKind, bool isPrimary)
-            : this(pointerKind, isPrimary, Guid.NewGuid().ToString())
+        public PointerInputDevice(PointerKind pointerKind)
+            : this(pointerKind, Guid.NewGuid().ToString())
         {
         }
 
         /// <summary>
         /// Initializes a new instance of the <see cref="PointerInputDevice"/> class.
         /// </summary>
         /// <param name="pointerKind">The kind of pointer represented by this input device.</param>
-        /// <param name="isPrimary">A value indicating whether the pointer device is the primary pointer device.</param>
         /// <param name="deviceName">The unique name for this input device.</param>
-        public PointerInputDevice(PointerKind pointerKind, bool isPrimary, string deviceName)
+        public PointerInputDevice(PointerKind pointerKind, string deviceName)
             : base(deviceName)
         {
             this.pointerKind = pointerKind;
-            this.isPrimary = isPrimary;
         }
 
         /// <summary>
@@ -148,7 +144,6 @@ public override Dictionary<string, object> ToDictionary()
             toReturn["id"] = this.DeviceName;
 
             Dictionary<string, object> parameters = new Dictionary<string, object>();
-            parameters["primary"] = this.isPrimary;
             parameters["pointerType"] = this.pointerKind.ToString().ToLowerInvariant();
             toReturn["parameters"] = parameters;