Fixing dynamic assembly generation for .NET PageFactory for .NET 3.5

Author: jimevans

dotnet/src/support/PageObjects/DefaultPageObjectMemberDecorator.cs

@@ -214,8 +214,8 @@ private static Type CreateTypeForASingleElement()
         {
             AssemblyName tempAssemblyName = new AssemblyName(Guid.NewGuid().ToString());
 
-            AssemblyBuilder dynamicAssembly = AppDomain.CurrentDomain.DefineDynamicAssembly(tempAssemblyName, AssemblyBuilderAccess.RunAndCollect);
-            ModuleBuilder moduleBuilder = dynamicAssembly.DefineDynamicModule(tempAssemblyName.Name, tempAssemblyName + ".dll");
+            AssemblyBuilder dynamicAssembly = AppDomain.CurrentDomain.DefineDynamicAssembly(tempAssemblyName, AssemblyBuilderAccess.Run);
+            ModuleBuilder moduleBuilder = dynamicAssembly.DefineDynamicModule(tempAssemblyName.Name);
             TypeBuilder typeBuilder = moduleBuilder.DefineType(typeof(IWebElement).FullName, TypeAttributes.Public | TypeAttributes.Interface | TypeAttributes.Abstract);
 
             foreach (Type type in InterfacesToBeProxied)

dotnet/src/support/UI/ExpectedConditions.cs

@@ -317,7 +317,7 @@ public static Func<IWebDriver, bool> TextToBePresentInElementValue(By locator, s
 
         /// <summary>
         /// An expectation for checking whether the given frame is available to switch
-        /// to. <p> If the frame is available it switches the given driver to the
+        /// to. If the frame is available it switches the given driver to the
         /// specified frame.
         /// </summary>
         /// <param name="frameLocator">Used to find the frame (id or name)</param>
@@ -339,7 +339,7 @@ public static Func<IWebDriver, IWebDriver> FrameToBeAvailableAndSwitchToIt(strin
 
         /// <summary>
         /// An expectation for checking whether the given frame is available to switch
-        /// to. <p> If the frame is available it switches the given driver to the
+        /// to. If the frame is available it switches the given driver to the
         /// specified frame.
         /// </summary>
         /// <param name="locator">Locator for the Frame</param>