Re: Need review of PHP file in Progress Event test suite [Was: Re: RfR: Progress Events Test Cases; deadline January 28]

On Mon, 28 Jan 2013, Jungkee Song wrote:

> Hi Yves, Mike,
>
> It seems that <http://w3c-test.org/webapps/ProgressEvents/tests/submissions/Samsung/firing-events-http-no-content-length.html>
> gets a timed out error due to the size of php data transfer. I suppose
> testharness raises the error if the data transer is not complete
> within 2sec or so.
>
> Hence, I reduced the size of the data transfer in the php file as below:
> https://dvcs.w3.org/hg/webapps/rev/0a4f622868eb
>
> Could you please review it?
Sure,


>
> Regards,
> Jungkee
>
> On Wed, Jan 23, 2013 at 1:21 AM, Yves Lafon <ylafon@w3.org> wrote:
>> On Tue, 22 Jan 2013, Jungkee Song wrote:
>>
>>> Hi Yves,
>>>
>>> I've found that the php file has been mirrored at w3c-test.org, but it
>>> seems the version is the one before I made the changes. Could you
>>> check it?
>>
>>
>> Thanks to Mike, I was able to have it mirrored a few minutes ago.
>>
>>
>>>
>>> Thanks,
>>> Jungkee
>>>
>>>
>>> On Tue, Jan 22, 2013 at 5:56 AM, Jungkee Song <jungkees@gmail.com> wrote:
>>>>
>>>> On Tue, Jan 22, 2013 at 12:35 AM, Yves Lafon <ylafon@w3.org> wrote:
>>>>>
>>>>> On Mon, 21 Jan 2013, Arthur Barstow wrote:
>>>>>
>>>>>> On 1/20/13 7:29 AM, ext Arthur Barstow wrote:
>>>>>> Jungkee - Robin reported the PHP file will not get mirrored to
>>>>>> w3c-test.org until it is OK'ed by someone in the W3C Team.
>>>>
>>>>
>>>> Hi Art,
>>>>
>>>> thanks for the help.
>>>>
>>>>>> Yves, Mike - on January 18, Jungkee added the following PHP file to the
>>>>>> Samsung submission directory of the Progress Events test suite:
>>>>>>
>>>>>>
>>>>>>
>>>>>> <https://dvcs.w3.org/hg/webapps/diff/ec95ece1bbf8/ProgressEvents/tests/submissions/Samsung/resources/no-content-length.php>
>>>>>>
>>>>>> Would you please review it so it will be mirrored to:
>>>>>>
>>>>>>
>>>>>>
>>>>>> <http://w3c-test.org/webapps/ProgressEvents/tests/submissions/Samsung/resources/>
>>>>>
>>>>>
>>>>>
>>>>> I'm wondering what is the intent of this, sending content with no
>>>>> content-length (and then chunked transfer-coding per HTTP/1.1 spec), or
>>>>> do
>>>>> double encoding.
>>>>> Also the chunk size is wrong, as \r\n is added twice, and the final
>>>>> chunk is
>>>>> missing.
>>>>> Cheers,
>>>>
>>>>
>>>> Hi Yves,
>>>>
>>>> Yes, the intention is sending content with no content-length using
>>>> HTTP/1.1 chunked transfer. I've tried to address your comments in the
>>>> following changeset:
>>>> https://dvcs.w3.org/hg/webapps/rev/b47697234d1a
>>>>
>>>> Could you please review it?
>>>>
>>>> Regards,
>>>> Jungkee
>>>>
>>>>> --
>>>>> Baroula que barouleras, au tiéu toujou t'entourneras.
>>>>>
>>>>>         ~~Yves
>>>>>
>>>>
>>>>
>>>
>>
>> --
>> Baroula que barouleras, au tiéu toujou t'entourneras.
>>
>>         ~~Yves
>>
>
>
>
>

-- 
Baroula que barouleras, au tiéu toujou t'entourneras.

         ~~Yves

Received on Monday, 28 January 2013 13:18:40 UTC