From e0c31137434d7c7df2bc3475cb7682b1d7eecc8b Mon Sep 17 00:00:00 2001
From: Brandon Dail <aweary@users.noreply.github.com>
Date: Wed, 29 Nov 2017 09:59:56 -0800
Subject: [PATCH] Remove empty value for boolean attributes in SSR (#11708)

---
 .../src/__tests__/ReactDOMSelect-test.js       | 18 +++++++++---------
 .../src/server/DOMMarkupOperations.js          |  2 +-
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/packages/react-dom/src/__tests__/ReactDOMSelect-test.js b/packages/react-dom/src/__tests__/ReactDOMSelect-test.js
index 6e44858908..df206e7355 100644
--- a/packages/react-dom/src/__tests__/ReactDOMSelect-test.js
+++ b/packages/react-dom/src/__tests__/ReactDOMSelect-test.js
@@ -440,9 +440,9 @@ describe('ReactDOMSelect', () => {
       </select>
     );
     var markup = ReactDOMServer.renderToString(stub);
-    expect(markup).toContain('<option selected="" value="giraffe"');
-    expect(markup).not.toContain('<option selected="" value="monkey"');
-    expect(markup).not.toContain('<option selected="" value="gorilla"');
+    expect(markup).toContain('<option selected value="giraffe"');
+    expect(markup).not.toContain('<option selected value="monkey"');
+    expect(markup).not.toContain('<option selected value="gorilla"');
   });
 
   it('should support server-side rendering with defaultValue', () => {
@@ -454,9 +454,9 @@ describe('ReactDOMSelect', () => {
       </select>
     );
     var markup = ReactDOMServer.renderToString(stub);
-    expect(markup).toContain('<option selected="" value="giraffe"');
-    expect(markup).not.toContain('<option selected="" value="monkey"');
-    expect(markup).not.toContain('<option selected="" value="gorilla"');
+    expect(markup).toContain('<option selected value="giraffe"');
+    expect(markup).not.toContain('<option selected value="monkey"');
+    expect(markup).not.toContain('<option selected value="gorilla"');
   });
 
   it('should support server-side rendering with multiple', () => {
@@ -468,9 +468,9 @@ describe('ReactDOMSelect', () => {
       </select>
     );
     var markup = ReactDOMServer.renderToString(stub);
-    expect(markup).toContain('<option selected="" value="giraffe"');
-    expect(markup).toContain('<option selected="" value="gorilla"');
-    expect(markup).not.toContain('<option selected="" value="monkey"');
+    expect(markup).toContain('<option selected value="giraffe"');
+    expect(markup).toContain('<option selected value="gorilla"');
+    expect(markup).not.toContain('<option selected value="monkey"');
   });
 
   it('should not control defaultValue if readding options', () => {
diff --git a/packages/react-dom/src/server/DOMMarkupOperations.js b/packages/react-dom/src/server/DOMMarkupOperations.js
index b86f6fd601..0fdf64f706 100644
--- a/packages/react-dom/src/server/DOMMarkupOperations.js
+++ b/packages/react-dom/src/server/DOMMarkupOperations.js
@@ -90,7 +90,7 @@ export function createMarkupForProperty(name, value) {
       propertyInfo.hasBooleanValue ||
       (propertyInfo.hasOverloadedBooleanValue && value === true)
     ) {
-      return attributeName + '=""';
+      return attributeName;
     } else if (
       typeof value !== 'boolean' ||
       shouldAttributeAcceptBooleanValue(name)