W3C home > Mailing lists > Public > public-webapi@w3.org > July 2007

Re: [XHR] Editorial and comments

From: Anne van Kesteren <annevk@opera.com>
Date: Wed, 25 Jul 2007 11:38:26 +0200
To: David Håsäther <hasather@gmail.com>, public-webapi@w3.org
Message-ID: <op.tv0k2cee64w2qv@annevk-t60.oslo.opera.com>

On Thu, 19 Jul 2007 22:24:14 +0200, David Håsäther <hasather@gmail.com>  
wrote:
> Some comments on the XMLHttpRequest draft.

Thanks!


> It should probably say
>
>    | If there is a Content-Type header which contains a text/html MIME  
> type [...]

Done.


> It can never match if the length of the entity body is less than the  
> number of bytes in the first column, so it can probably be reworded as:
>
>    | If charset is "null" then, for each of the rows in the following  
> table,
>    | starting with the first one and going down, if the first bytes of  
> /bytes/
>    | match the bytes given in the first column [...]

Done.


> In the same step it says:
>
>    | When an encoding is determined or when no encoding is determined go  
> to the
>    | next step
>
> This text looks redundant.

Reworded.


> One of the steps for the open() method[2] says:
>
>    | Let async be the value of the async argument or true if it was  
> omitted.
>
> What happens if async is set to something other than false or true? Or  
> is that covered somewhere else?

That would be covered at the binding level (IDL -> language mapping).


> [...]
>
> Should '1' or '2' be sent? I tested this in Opera 9.2 and Firefox 2.  
> Opera doesn't seem to post anything when the parameter is a number.  
> Firefox sends '1'.

Addressed by another reply.


> The second to last step for the send() method says:
>
>    | [...] Finally, if the complete resource has been downloaded go to  
> the
>    | next step.
>
> This text might be redundant.

Kept for clarity.


>    | Responses to such requests must have the content-codings  
> automatically
>    | decoded.
>
> s/content-codings/content-encodings/?

Fixed.


> The third and fourth step for the getResponseHeader():
>
> s/for the last request sent return the/for the last request sent, return  
> the/ (added a comma for clarity).

Done.


> [2] <http://www.w3.org/TR/2007/WD-XMLHttpRequest-20070618/#dfn-open>


-- 
Anne van Kesteren
<http://annevankesteren.nl/>
<http://www.opera.com/>
Received on Wednesday, 25 July 2007 09:38:34 GMT

This archive was generated by hypermail 2.2.0+W3C-0.50 : Tuesday, 8 January 2008 14:18:58 GMT