Liberal acceptance of webfont mimetypes

Liberal acceptance of webfont mimetypes

https://www.google.com/search?q=chrome+font+mime+type

MIME types for fonts has been a source of pain for developers, as
incorrect types cause excess console spam on every page load. In order
to deliver a clean console, let's complain less. Chrome itself doesn't 
require any particular mimetype.
                                          
BUG=248712
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=153362

[Dmitry Zvorygin, 2013-07-01 10:14:49]

LGTM.

[aandrey, 2013-07-01 10:53:35]

https://codereview.chromium.org/18308003/diff/1/Source/devtools/front_end/Net...
File Source/devtools/front_end/NetworkManager.js (right):

https://codereview.chromium.org/18308003/diff/1/Source/devtools/front_end/Net...
Source/devtools/front_end/NetworkManager.js:78: "application/octet-stream":   
{"font": true},
This mime indicates an arbitrary binary data, that may be, for example, an
image, audio, video, etc.

[caseq, 2013-07-01 11:28:54]

On 2013/07/01 10:53:35, aandrey wrote:
>
https://codereview.chromium.org/18308003/diff/1/Source/devtools/front_end/Net...
> File Source/devtools/front_end/NetworkManager.js (right):
> 
>
https://codereview.chromium.org/18308003/diff/1/Source/devtools/front_end/Net...
> Source/devtools/front_end/NetworkManager.js:78: "application/octet-stream":   
> {"font": true},
> This mime indicates an arbitrary binary data, that may be, for example, an
> image, audio, video, etc.

This map is only used to detect inconsistencies between the MIME type and actual
resource use, so adding font here has no effect beyond disabling warning for
fonts that are served with the fall-back MIME type. This seems fair, as the MIME
type for certain fonts is not defined.

[Dmitry Zvorygin, 2013-07-01 11:34:59]

https://codereview.chromium.org/18308003/diff/1/Source/devtools/front_end/Net...
File Source/devtools/front_end/NetworkManager.js (right):

https://codereview.chromium.org/18308003/diff/1/Source/devtools/front_end/Net...
Source/devtools/front_end/NetworkManager.js:78: "application/octet-stream":   
{"font": true},
On 2013/07/01 10:53:35, aandrey wrote:
> This mime indicates an arbitrary binary data, that may be, for example, an
> image, audio, video, etc.

{"font":true} just means that it's ok for font being served as
application-octet-stream, so I don't see an error in this line.

See, for example 

<code>
"image/svg+xml": {"image": true, "font": true, "document": true},
</code>

at line 67.

[aandrey, 2013-07-01 11:43:36]

On 2013/07/01 11:34:59, Dmitry Zvorygin wrote:
>
https://codereview.chromium.org/18308003/diff/1/Source/devtools/front_end/Net...
> File Source/devtools/front_end/NetworkManager.js (right):
> 
>
https://codereview.chromium.org/18308003/diff/1/Source/devtools/front_end/Net...
> Source/devtools/front_end/NetworkManager.js:78: "application/octet-stream":   
> {"font": true},
> On 2013/07/01 10:53:35, aandrey wrote:
> > This mime indicates an arbitrary binary data, that may be, for example, an
> > image, audio, video, etc.
> 
> {"font":true} just means that it's ok for font being served as
> application-octet-stream, so I don't see an error in this line.
> 
> See, for example 
> 
> <code>
> "image/svg+xml": {"image": true, "font": true, "document": true},
> </code>
> 
> at line 67.

ah ok, I see

[commit-bot: I haz the power, 2013-07-01 17:01:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/paulirish@chromium.org/18308003/1

[commit-bot: I haz the power, 2013-07-01 17:16:42]

Retried try job too often on blink_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_pres...

[pfeldman, 2013-07-01 17:19:22]

lgtm

[commit-bot: I haz the power, 2013-07-01 17:19:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/paulirish@chromium.org/18308003/1

[commit-bot: I haz the power, 2013-07-01 18:52:48]

Change committed as 153362