Setting HTMLMediaElement.controller does not properly remove the 'mediagroup' content attribute

LayoutTests/media/media-controller.html

Index: LayoutTests/media/media-controller.html
diff --git a/LayoutTests/media/media-controller.html b/LayoutTests/media/media-controller.html
index 119506399a4f0860c09b354f6f877172e11c6997..3cbaf83390215f8eb638ff5da48aa08e53e3a5c3 100644
--- a/LayoutTests/media/media-controller.html
+++ b/LayoutTests/media/media-controller.html
@@ -31,7 +31,7 @@
             testExpected("video.controller", controller);
             consoleWrite("");
             consoleWrite("Test assigning the wrong type to the controller attribute of a media element:")
-            testException("video.controller = 42", '"TypeError: Value is not of type MediaController"');
+            testException("video.controller = 42", '"TypeError: Failed to set the \'controller\' property on \'HTMLMediaElement\': The provided value is not of type \'MediaController\'."');
             testExpected("video.controller", controller);
             consoleWrite("");
             consoleWrite("Test assigning a null controller to a media element:")
@@ -42,6 +42,22 @@
             run("video.mediaGroup = 'group'");
             run("video2.mediaGroup = 'group'");
             testExpected("video.controller === video2.controller", true);
+            consoleWrite("");
+            consoleWrite("Test that setting the controller clears the mediaGroup:")
+            testExpected("video.mediaGroup", "group");
+            run("video.controller = null");
+            testExpected("video.controller", null);
+            testExpected("video.mediaGroup", "");

[philipj_slow, 2013/11/08 19:58:04]

Also checking !video.hasAttribute('mediagroup') here and below would be good,
since I think this would also pass if the attribute was present but empty.

[Inactive, 2013/11/08 20:12:06]

Done.

+            run("video.mediaGroup = 'group'");
+            testExpected("video.mediaGroup", "group");
+            run("video.controller = controller");
+            testExpected("video.controller", controller);
+            testExpected("video.mediaGroup", "");
+            consoleWrite("");
+            consoleWrite("Test that setting the controller to undefined is treated as null:")
+            testExpected("video.controller", null, "!=");
+            run("video.controller = undefined");
+            testExpected("video.controller", null);
             endTest();
         }
         </script>