Default to binary encoding when loading MHTML files.

Default to binary encoding when loading MHTML files.

Assume binary encoding on MHTML files that doesn't have an encoding
specified.

BUG=

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=158022

[christiank, 2013-08-28 12:49:57]

Adam, would you please have a look at this review? This is another patch related
to MHTML, following the others made by my colleague.

Since there appears to be some issue uploading binary files I have created an
archive containing a layout test and uploaded it here:

http://people.opera.com/christiank/codereview/23464015/

Both archives contain the same files:
LayoutTests/mhtml/content_transfer_encoding_none-expected.txt
LayoutTests/mhtml/content_transfer_encoding_none.html_original
LayoutTests/mhtml/content_transfer_encoding_none.mht

I was hoping that maybe you could assist me with integrating the test if/when
you accept this patch.

[abarth-chromium, 2013-08-28 17:40:58]

How does IE behave in this case?

The code change itself looks fine assuming this behavior is consistent with
other browsers.

[christiank, 2013-08-29 09:23:55]

On 2013/08/28 17:40:58, abarth wrote:
> How does IE behave in this case?
> 
> The code change itself looks fine assuming this behavior is consistent with
> other browsers.

I have tested IE 8 through 10 and all of them loads the test file with no
Content-Transfer-Encoding specified. Firefox also loads it (using the UnMHT
extension).

[abarth-chromium, 2013-08-30 06:42:49]

Great.  LGTM.  Let's land this together with the tests.  I can try to do that
for you tomorrow.

[christiank, 2013-08-30 07:02:07]

On 2013/08/30 06:42:49, abarth wrote:
> Great.  LGTM.  Let's land this together with the tests.  I can try to do that
> for you tomorrow.

Would be great, thanks!

[christiank, 2013-09-05 13:54:15]

On 2013/08/30 06:42:49, abarth wrote:
> Great.  LGTM.  Let's land this together with the tests.  I can try to do that
> for you tomorrow.

Any news? Did you have time to look at this?

[abarth-chromium, 2013-09-16 14:17:46]

On 2013/09/05 13:54:15, christiank wrote:
> Any news? Did you have time to look at this?

Sorry for the delay.  The good news is that the commit-queue should support
binary files now.  That means you can upload the tests along with this change
and land them yourself via the commit queue.  Please let me know if you have any
trouble.

[commit-bot: I haz the power, 2013-09-19 08:32:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/christiank@opera.com/23464015/10001

[commit-bot: I haz the power, 2013-09-19 09:33:13]

Change committed as 158022