fix(script): parse template as JSON if content-type attr matches 'json'

This change is needed so that $http can avoid parsing JSON if an inline
template is in use.

BREAKING CHANGE: inline templates which were previously parsed as JSON by
default, now require the attribute `content-type` to contain the text 'json'.

Additionally, the data pushed into $templateCache by the script directive is
now always an array, with the following items:

[ status (200), data (script text or parsed JSON), headers (the empty object) ]

BEFORE:

```
<script type="text/ng-template" id="famous-foods.json">
{
  "elvis presley": "peanut butter, banana & bacon sandwich",
  "shirley temple": "whipped potatoes & roast beef",
  "marilyn monroe": "grilled steak"
}
</script>
```

AFTER:

```
<script type="text/ng-template" id="famous-foods.json" content-type="application/json">
{
  "elvis presley": "peanut butter, banana & bacon sandwich",
  "shirley temple": "whipped potatoes & roast beef",
  "marilyn monroe": "grilled steak"
}
</script>

Author: caitp

docs/src/example.js

@@ -115,14 +115,14 @@ exports.Example.prototype.toHtmlTabs = function() {
   htmlTabs(this.html);
   htmlTabs(this.css);
   htmlTabs(this.js);
-  htmlTabs(this.json);
+  htmlTabs(this.json, 'application/json');
   htmlTabs(this.unit);
   htmlTabs(this.scenario);
   htmlTabs(this.protractorTest);
   out.push('</div>');
   return out.join('');
 
-  function htmlTabs(sources) {
+  function htmlTabs(sources, contentType) {
     sources.forEach(function(source) {
       var wrap = '',
           isCss = source.name.match(/\.css$/),
@@ -139,7 +139,9 @@ exports.Example.prototype.toHtmlTabs = function() {
           '<pre class="prettyprint linenums" ng-set-text="' + source.id + '"' + wrap + '></pre>\n' +
           (isCss
              ? ('<style type="text/css" id="' + source.id + '">' + source.content + '</style>\n')
-             : ('<script type="text/ng-template" id="' + source.id + '">' + source.content + '</script>\n') ) +
+             : ('<script type="text/ng-template"' + (' id="' + source.id + '"') +
+                (contentType ? ' content-type="' + contentType + '"' : '') + '>' +
+               source.content + '</script>\n') ) +
         '</div>\n');
     });
   }

src/ng/directive/script.js

@@ -34,6 +34,7 @@
   </doc:example>
  */
 var scriptDirective = ['$templateCache', function($templateCache) {
+  var JSON_CONTENT_TYPE = /json/;
   return {
     restrict: 'E',
     terminal: true,
@@ -43,7 +44,11 @@ var scriptDirective = ['$templateCache', function($templateCache) {
             // IE is not consistent, in scripts we have to read .text but in other nodes we have to read .textContent
             text = element[0].text;
 
-        $templateCache.put(templateUrl, text);
+        if (JSON_CONTENT_TYPE.test(attr.contentType || '')) {
+          text = JSON.parse(text);
+        }
+
+        $templateCache.put(templateUrl, [200, text, {}]);
       }
     }
   };

test/ng/directive/scriptSpec.js

@@ -15,7 +15,7 @@ describe('scriptDirective', function() {
                    '<script id="/ignore">ignore me</script>' +
                    '<script type="text/ng-template" id="/myTemplate.html"><x>{{y}}</x></script>' +
                  '</div>' );
-        expect($templateCache.get('/myTemplate.html')).toBe('<x>{{y}}</x>');
+        expect($templateCache.get('/myTemplate.html')[1]).toBe('<x>{{y}}</x>');
         expect($templateCache.get('/ignore')).toBeUndefined();
       }
   ));
@@ -36,4 +36,11 @@ describe('scriptDirective', function() {
     expect(scripts.eq(1)[0].text).toBe('other {{binding}}');
     dealoc(doc);
   }));
+
+
+  it('should parse JSON if attribute `content-type` matches \'json\'', inject(function($compile, $templateCache, $rootScope) {
+    var doc = '<div><script type="text/ng-template" id="file.json" content-type="application/json">{"foo":"bar"}</script></div>';
+    element = $compile(doc)($rootScope);
+    expect($templateCache.get('file.json')[1]).toEqual({"foo": "bar"});
+  }));
 });