Re: [widgets] test bn in error

2010-12-03 Thread Marcos Caceres
On Wed, Dec 1, 2010 at 12:11 PM, Scott Wilson
scott.bradley.wil...@gmail.com wrote:
 # bn (download, files)

 Tests the UA's ability to deal with custom icon declarations in the config
 document and matching default icons. To pass, the icons list must contain a
 pointer to icons/pass.png, and locales/en/icon.jpg (ordering of the
 items in the list is irrelevant).

 The actual icons included are icons/pass.png and locales/en/icon.png
 (not .jpg).

Fixed. Committed.

-- 
Marcos Caceres
Opera Software ASA, http://www.opera.com/
http://datadriven.com.au



Re: [widgets] Error in test case bm

2010-12-03 Thread Marcos Caceres
On Wed, Dec 1, 2010 at 11:47 AM, Scott Wilson
scott.bradley.wil...@gmail.com wrote:
 # bm (download, files)

 Tests the UA's ability to deal with custom icon declaration in the config
 document and matching default icons. To pass, the icons list must contain a
 pointer to locales/en/icon.jpg, and icon.png, which is at the root of
 the widget. The icons list can be in any order, so long as it contains
 icon.png and locales/en/icon.jpg.

 The download contains two icons, but one of these is called 333icon.png
 and so would be ignored.

Recommitted the file, hopefully that fixes it.


-- 
Marcos Caceres
Opera Software ASA, http://www.opera.com/
http://datadriven.com.au



[widgets] Error in i18n tests for features

2010-12-03 Thread Scott Wilson
# i18nlro30 (download, files)
Tests that LRO direction does not apply to the feature element's required 
attribute. To pass, the value of the required attribute must be treated as 
false.


The actual feature is:

feature name=feature:x-bugus-feature required=false dir=rtl/

So as the feature is unsupported by the UA, the feature is ignored and the 
value of the required attribute is irrelevant. (All other xxx30 tests have the 
same issue).

smime.p7s
Description: S/MIME cryptographic signature


[widgets] Test widget has wrong content

2010-12-03 Thread Scott Wilson
See:

http://dev.w3.org/2006/waf/widgets/test-suite/test-cases/i18n-lro/001/i18nlro01.wgt

This is pretty obviously not what should be in here!

S

smime.p7s
Description: S/MIME cryptographic signature


Re: [widgets] Test widget has wrong content

2010-12-03 Thread Marcos Caceres
On Fri, Dec 3, 2010 at 12:19 PM, Scott Wilson
scott.bradley.wil...@gmail.com wrote:
 See:

 http://dev.w3.org/2006/waf/widgets/test-suite/test-cases/i18n-lro/001/i18nlro01.wgt

 This is pretty obviously not what should be in here!

 S

Fixed.

-- 
Marcos Caceres
Opera Software ASA, http://www.opera.com/
http://datadriven.com.au



Re: [File API] abort()

2010-12-03 Thread Arun Ranganathan
Anne,


- Original Message -
 Per http://dev.w3.org/2006/webapi/FileAPI/#abort invoking abort()
 always
 results in events getting dispatched. This is not what happens in e.g.
 Gecko at the moment. When the state is EMPTY the method simply
 returns.


To be clear, what is the behavior you are prescribing?  Chrome folks, what is 
the desired behavior for your implementation?


 
 The dispatching of events should probably also be defined in terms of
 the
 event queue so it becomes more clear which are to be dispatched
 synchronously, etc. Currently all those interactions are quite vague
 in
 the File API specification.

OK, I agree that this could be better, and should be improved in a subsequent 
version (especially making clear synchronous dispatch).  If you could be more 
specific about your nits, that would help as well.

-- A*



Re: [widgets] Error in i18n tests for features

2010-12-03 Thread Marcos Caceres
On Fri, Dec 3, 2010 at 12:05 PM, Scott Wilson
scott.bradley.wil...@gmail.com wrote:
 # i18nlro30 (download, files)

 Tests that LRO direction does not apply to the feature element's required
 attribute. To pass, the value of the required attribute must be treated as
 false.

 The actual feature is:
     feature name=feature:x-bugus-feature required=false dir=rtl/
 So as the feature is unsupported by the UA, the feature is ignored and the
 value of the required attribute is irrelevant. (All other xxx30 tests have
 the same issue).

Can you suggest an alternative way to test this?

-- 
Marcos Caceres
Opera Software ASA, http://www.opera.com/
http://datadriven.com.au



Re: [widgets] Error in i18n tests for features

2010-12-03 Thread Scott Wilson

On 3 Dec 2010, at 16:28, Marcos Caceres wrote:

 On Fri, Dec 3, 2010 at 12:05 PM, Scott Wilson
 scott.bradley.wil...@gmail.com wrote:
 # i18nlro30 (download, files)
 
 Tests that LRO direction does not apply to the feature element's required
 attribute. To pass, the value of the required attribute must be treated as
 false.
 
 The actual feature is:
 feature name=feature:x-bugus-feature required=false dir=rtl/
 So as the feature is unsupported by the UA, the feature is ignored and the
 value of the required attribute is irrelevant. (All other xxx30 tests have
 the same issue).
 
 Can you suggest an alternative way to test this?

How about:

feature name=feature:a9bb79c1 required=false dir=rtl/

?

 
 -- 
 Marcos Caceres
 Opera Software ASA, http://www.opera.com/
 http://datadriven.com.au



smime.p7s
Description: S/MIME cryptographic signature


Re: [widgets] Error in i18n tests for features

2010-12-03 Thread Marcos Caceres

On 12/3/10 5:38 PM, Scott Wilson wrote:


On 3 Dec 2010, at 16:28, Marcos Caceres wrote:


On Fri, Dec 3, 2010 at 12:05 PM, Scott Wilson
scott.bradley.wil...@gmail.com  wrote:

# i18nlro30 (download, files)

Tests that LRO direction does not apply to the feature element's required
attribute. To pass, the value of the required attribute must be treated as
false.

The actual feature is:
 feature name=feature:x-bugus-feature required=false dir=rtl/
So as the feature is unsupported by the UA, the feature is ignored and the
value of the required attribute is irrelevant. (All other xxx30 tests have
the same issue).


Can you suggest an alternative way to test this?


How about:

feature name=feature:a9bb79c1 required=false dir=rtl/

?


That might be ok, but the problem is still that if false is treated as 
eslaf it would still default to true (which is why I wrote the test 
with the bogus feature):


If a required attribute is used, then let required-feature be the 
result of applying the rule for getting a single attribute value to the 
required attribute. If the required attribute is not used or if 
required-feature is not a valid boolean value, then let the value of 
required-feature be the value 'true'.





Re: [widgets] Error in i18n tests for features

2010-12-03 Thread Scott Wilson
On 3 Dec 2010, at 16:54, Marcos Caceres wrote:

 On 12/3/10 5:38 PM, Scott Wilson wrote:
 
 On 3 Dec 2010, at 16:28, Marcos Caceres wrote:
 
 On Fri, Dec 3, 2010 at 12:05 PM, Scott Wilson
 scott.bradley.wil...@gmail.com  wrote:
 # i18nlro30 (download, files)
 
 Tests that LRO direction does not apply to the feature element's required
 attribute. To pass, the value of the required attribute must be treated as
 false.
 
 The actual feature is:
 feature name=feature:x-bugus-feature required=false dir=rtl/
 So as the feature is unsupported by the UA, the feature is ignored and the
 value of the required attribute is irrelevant. (All other xxx30 tests have
 the same issue).
 
 Can you suggest an alternative way to test this?
 
 How about:
 
 feature name=feature:a9bb79c1 required=false dir=rtl/
 
 ?
 
 That might be ok, but the problem is still that if false is treated as 
 eslaf it would still default to true (which is why I wrote the test with 
 the bogus feature):
 
 If a required attribute is used, then let required-feature be the result of 
 applying the rule for getting a single attribute value to the required 
 attribute. If the required attribute is not used or if required-feature is 
 not a valid boolean value, then let the value of required-feature be the 
 value 'true'.
 

Ah OK I think I understand now, so the actual test is OK if the UA doesn't 
throw an exception and has an empty feature list (its not supported, and 
required is false). You shouldn't see a false value on the Feature as if all 
goes well, it won't be used?



smime.p7s
Description: S/MIME cryptographic signature


Re: Structured clone in WebStorage

2010-12-03 Thread Darin Fisher
Have you guys considered what happens when a Blob is present?  I'm very
concerned about that case since WebStorage is a synchronous API.  If storing
a Blob in LocalStorage involves synchronous disk IO, then I'm pretty sure it
is something I would object to implementing.  I don't think there is a
hand-wavy answer about performing a file copy asynchronously in the
background.  What if the file copy fails?

-Darin



On Thu, Dec 2, 2010 at 1:45 PM, Anne van Kesteren ann...@opera.com wrote:

 On Thu, 02 Dec 2010 21:51:29 +0100, Maciej Stachowiak m...@apple.com
 wrote:

 We think this feature would be straightforward to implement in
 Safari/WebKit, and we think it is a useful feature. We would like to
 implement it at some point. I can't give a specific timeline.


 I am not sure if it is straightforward to implement in Opera, but this
 applies to us as well.


 --
 Anne van Kesteren
 http://annevankesteren.nl/




Re: Structured clone in WebStorage

2010-12-03 Thread Darin Fisher
I will also add that I think WebStorage (well LocalStorage) is terrible from
a performance point of view because it is synchronous, and I'd be very happy
if we could discourage its usage and give people more reasons to adopt a
better API like IndexedDB.

-Darin



On Fri, Dec 3, 2010 at 10:41 AM, Darin Fisher da...@chromium.org wrote:

 Have you guys considered what happens when a Blob is present?  I'm very
 concerned about that case since WebStorage is a synchronous API.  If storing
 a Blob in LocalStorage involves synchronous disk IO, then I'm pretty sure it
 is something I would object to implementing.  I don't think there is a
 hand-wavy answer about performing a file copy asynchronously in the
 background.  What if the file copy fails?

 -Darin



 On Thu, Dec 2, 2010 at 1:45 PM, Anne van Kesteren ann...@opera.comwrote:

 On Thu, 02 Dec 2010 21:51:29 +0100, Maciej Stachowiak m...@apple.com
 wrote:

 We think this feature would be straightforward to implement in
 Safari/WebKit, and we think it is a useful feature. We would like to
 implement it at some point. I can't give a specific timeline.


 I am not sure if it is straightforward to implement in Opera, but this
 applies to us as well.


 --
 Anne van Kesteren
 http://annevankesteren.nl/





Re: Structured clone in WebStorage

2010-12-03 Thread Oliver Hunt
I recall talking to hixie about this at some point, and I recall that at that 
point localstorage was explicitly prevented from Blobs, although I can't see 
any sign of that rule anymore :-/

--Oliver

On Dec 3, 2010, at 10:41 AM, Darin Fisher wrote:

 Have you guys considered what happens when a Blob is present?  I'm very 
 concerned about that case since WebStorage is a synchronous API.  If storing 
 a Blob in LocalStorage involves synchronous disk IO, then I'm pretty sure it 
 is something I would object to implementing.  I don't think there is a 
 hand-wavy answer about performing a file copy asynchronously in the 
 background.  What if the file copy fails?
 
 -Darin
 
 
 
 On Thu, Dec 2, 2010 at 1:45 PM, Anne van Kesteren ann...@opera.com wrote:
 On Thu, 02 Dec 2010 21:51:29 +0100, Maciej Stachowiak m...@apple.com wrote:
 We think this feature would be straightforward to implement in Safari/WebKit, 
 and we think it is a useful feature. We would like to implement it at some 
 point. I can't give a specific timeline.
 
 I am not sure if it is straightforward to implement in Opera, but this 
 applies to us as well.
 
 
 -- 
 Anne van Kesteren
 http://annevankesteren.nl/
 
 



Re: using BlobBuilder append method with generated binary data

2010-12-03 Thread Paul Kinlan
I might have missed something but for appending client-side generated
data to a blob can't you just append a UInt8Array as that is based off
ArrayBuffer?

I use it to generate zip files client side and attach them to an
iframe so they are downloaded.

P
On Thursday, December 2, 2010, Toni Ruottu toni.ruo...@iki.fi wrote:
 My code had a bug in it. The APIs seem to be working ok for my purposes.
 Using the APIs, I was able to write Firefox's sendAsBinary method for chrome.
 It is available from http://javascript0.org/wiki/Portable_sendAsBinary

   --Toni

 On Tue, Nov 16, 2010 at 5:25 PM, Toni Ruottu toni.ruo...@iki.fi wrote:
 Sorry for slow answer. Took me a while to get on top of this. I think
 they just might answer my problem. However, I can not really tell
 before I see a working implementation. I wrote a test. See
 http://www.cs.helsinki.fi/u/twruottu/testi/bpost3.html It does not
 currently work with the latest development version of Chrome. I
 wonder, if I am doing something wrong.

  --Toni

 On Wed, Oct 20, 2010 at 6:17 PM, Eric Uhrhane er...@google.com wrote:
 Toni:

    BlobBuilder now has an append() method that takes an ArrayBuffer,
 and FileReader has readAsArrayBuffer.  Do these together satisfy your
 needs?

        Eric

 On Wed, Oct 20, 2010 at 3:39 AM, Toni Ruottu toni.ruo...@iki.fi wrote:
 I have discussed the topic before on some Chromium bug threads. I
 searched the archives of this mailing list for blobs and generated
 content, but was not too successful. There may be multiple different
 reasons why one would need to build blobs from generated data. The one
 that concerns me, is being able to do binary POSTs with XmlHTTPRequest
 in a standard manner. To send binary data over XHR one is required to
 send out a blob, but first the blob needs to be constructed with
 BlobBuilder. Let me fill you in on how I see the current situation.
 Maybe someone can spot an error in my line of thought.

 A new (empty) BlobBuilder is created by stating...
 var bb = new BlobBuilder();

 After creating a BlobBuilder, the user appends some data into the blob
 being generated by using the append function. The append function is
 overloaded and can be used to append multiple different types of data
 into the blob. Currently only existing blobs, and utf-8 text are
 supported.

 One can add an existing blob into the new one by doing...
 bb.append(existingBlob);

 One can add a text string in utf-8 format by doing...
 bb.append(aTextString);

 Once the data is in there, it is possible to construct the new blob with...
 var blob = bb.getBlob()

 A blob has a slice method which makes it possible to turn a large blob
 into smaller ones. It is thus possible to cut out single bytes from a
 blob and use them construct new blobs by appending copies of these
 sample bytes together with the BlobBuilder.

 As sample bytes can only be generated by writing text into the blob as
 utf-8, it is not possible to generate all 8-bit patterns. More
 specifically, the bit patterns that are illegal in any given utf-8
 string, are not possible to generate using the current API. This leads
 developers into using various hacks to overcome the issue.

 For example one could ask the user to provide a file with sample
 bytes. Methods for reading a file into a blob are available, so there
 is no need to generate the sample bytes. There is however no way to
 make sure that the user provided a file with correct bit-patterns as
 the blob interface is to a large degree a read-only interface. A
 demonstration of using a file to overcome the issue is available at
 http://www.cs.helsinki.fi/u/twruottu/testi/bpost.html

 To solve the problem a third append function which accepts a list of
 integers, and adds the corresponding bytes into the blob is needed.
 Once such method has been introduced it should be possible to write
 the 7-bit ascii string foo into a BlobBuilder by doing...
 bb.append([102,111,111]);

 It should also be possible to add bytes 0x00 a

-- 
Paul Kinlan
Developer Advocate @ Google for Chrome and HTML5
t: +447730517944
tw: @Paul_Kinlan http://twitter.com/paul_kinlan
LinkedIn: http://uk.linkedin.com/in/paulkinlan
Blog: http://paul.kinlan.me
Skype: paul.kinlan



Re: Structured clone in WebStorage

2010-12-03 Thread Ian Hickson
On Fri, 3 Dec 2010, Oliver Hunt wrote:

 I recall talking to hixie about this at some point, and I recall that at 
 that point localstorage was explicitly prevented from Blobs, although I 
 can't see any sign of that rule anymore :-/

Blobs are fine, they're async anyway. To store one you'd just need to let 
the blob know that you need a checkpoint. The actual storage in 
localStorage can all be done in the background. The feature that was 
problematic with localStorage wasn't Blobs, it was ImageData, and that 
should still be blocked.

-- 
Ian Hickson   U+1047E)\._.,--,'``.fL
http://ln.hixie.ch/   U+263A/,   _.. \   _\  ;`._ ,.
Things that are impossible just take longer.   `._.-(,_..'--(,_..'`-.;.'



Re: Structured clone in WebStorage

2010-12-03 Thread João Eiras

On , Darin Fisher da...@chromium.org wrote:


I will also add that I think WebStorage (well LocalStorage) is terrible from
a performance point of view because it is synchronous, and I'd be very happy
if we could discourage its usage and give people more reasons to adopt a
better API like IndexedDB.

-Darin



I don't understand how storing values in a hash map is performant terrible.