MAGETWO-33407: Cover MAGETWO-31855 changes by static tests

- Merge remote-tracking branch 'origin/MAGETWO-33407' into UI

Conflicts:
	app/code/Magento/GoogleAdwords/view/frontend/templates/code.phtml

Author: Natalia Momotenko

app/code/Magento/GoogleAdwords/view/frontend/templates/code.phtml

@@ -18,7 +18,7 @@
     var google_conversion_value = <?php echo $block->getHelper()->getConversionValue() ?>;
     /* ]]> */
 </script>
-<script type="text/javascript" src="<?php echo $block->getHelper()->getConversionJsSrc()?>"></script>
+<script src="<?php echo $block->getHelper()->getConversionJsSrc()?>"></script>
 <noscript>
     <div style="display:inline;">
         <img height="1" width="1" style="border-style:none;" alt=""

dev/tests/static/testsuite/Magento/Test/Legacy/PhtmlTemplateTest.php

@@ -56,4 +56,25 @@ function ($file) {
             \Magento\Framework\Test\Utility\Files::init()->getPhtmlFiles()
         );
     }
+
+    public function testObsoleteJavascriptAttributeType()
+    {
+        $invoker = new \Magento\Framework\Test\Utility\AggregateInvoker($this);
+        $invoker(
+        /**
+         * "text/javascript" type attribute in not obligatory to use in templates due to HTML5 standards.
+         * For more details please go to "http://www.w3.org/TR/html5/scripting-1.html".
+         *
+         * @param string $file
+         */
+            function ($file) {
+                $this->assertNotRegexp(
+                    '/type="text\/javascript"/',
+                    file_get_contents($file),
+                    'Please do not use "text/javascript" type attribute.'
+                );
+            },
+            \Magento\Framework\Test\Utility\Files::init()->getPhtmlFiles()
+        );
+    }
 }