Documenting misleading DataTransfer API.

Documenting misleading DataTransfer API.

BUG=

Committed: https://code.google.com/p/dart/source/detail?r=21216

[Andrei Mouravski, 2013-02-01 18:23:21]

https://codereview.chromium.org/12092114/diff/1/sdk/lib/html/dart2js/html_dar...
File sdk/lib/html/dart2js/html_dart2js.dart (right):

https://codereview.chromium.org/12092114/diff/1/sdk/lib/html/dart2js/html_dar...
sdk/lib/html/dart2js/html_dart2js.dart:5832: +   * Gets the data for the
specified MIME type.
Do we want to add more data like here:
http://msdn.microsoft.com/en-us/library/ie/ms536436(v=vs.85).aspx

Specifically the bits in "Remarks."

Also, will this trigger the Chrome, "this app is trying to access your clipboard
message"?

https://codereview.chromium.org/12092114/diff/1/sdk/lib/html/dart2js/html_dar...
sdk/lib/html/dart2js/html_dart2js.dart:5835: +   * onDrop event) and will return
`null` before then.
Can you please add an example?

[blois, 2013-02-01 18:31:24]

On 2013/02/01 18:23:21, Andrei Mouravski wrote:
>
https://codereview.chromium.org/12092114/diff/1/sdk/lib/html/dart2js/html_dar...
> File sdk/lib/html/dart2js/html_dart2js.dart (right):
> 
>
https://codereview.chromium.org/12092114/diff/1/sdk/lib/html/dart2js/html_dar...
> sdk/lib/html/dart2js/html_dart2js.dart:5832: +   * Gets the data for the
> specified MIME type.
> Do we want to add more data like here:
> http://msdn.microsoft.com/en-us/library/ie/ms536436%28v=vs.85%29.aspx
> 
> Specifically the bits in "Remarks."
> 
> Also, will this trigger the Chrome, "this app is trying to access your
clipboard
> message"?
> 
>
https://codereview.chromium.org/12092114/diff/1/sdk/lib/html/dart2js/html_dar...
> sdk/lib/html/dart2js/html_dart2js.dart:5835: +   * onDrop event) and will
return
> `null` before then.
> Can you please add an example?

This will not trigger the 'access clipboard' prompt.

I added info on the cross-domain limitations.

This is definitely worthy of a sample, but the sample is actually quite large. I
think we should add a separate issue for that.

Right now I'm just looking to address the complaint in bug 8255 which I hit
myself.

[Andrei Mouravski, 2013-02-01 18:40:45]

https://code.google.com/p/dart/issues/detail?id=8260 filed.

https://codereview.chromium.org/12092114/diff/3001/sdk/lib/html/dart2js/html_...
File sdk/lib/html/dart2js/html_dart2js.dart (right):

https://codereview.chromium.org/12092114/diff/3001/sdk/lib/html/dart2js/html_...
sdk/lib/html/dart2js/html_dart2js.dart:5834: +   * The  data is only available
from within a drop operation (such as an
Nit: There's an extra space between "the" and "data".

https://codereview.chromium.org/12092114/diff/3001/sdk/lib/html/dart2js/html_...
sdk/lib/html/dart2js/html_dart2js.dart:5835: +   * onDrop event) and will return
`null` before then.
Nit: link to onDrop event if you can.

https://codereview.chromium.org/12092114/diff/3001/sdk/lib/html/dart2js/html_...
sdk/lib/html/dart2js/html_dart2js.dart:5835: +   * onDrop event) and will return
`null` before then.
"... and will return `null` anywhere else."
or maybe
"... and will return `null` before the event is triggered."

Whichever is more accurate.

https://codereview.chromium.org/12092114/diff/3001/sdk/lib/html/dart2js/html_...
sdk/lib/html/dart2js/html_dart2js.dart:5837: +   * Data transfer is also
prohibited across domains. If a drag originates
No need for "also." Just:
"Data transfer is prohibited across domains."

[Andrei Mouravski, 2013-03-07 00:59:51]

Will this be going in any time?

[blois, 2013-04-10 00:26:47]

https://codereview.chromium.org/12092114/diff/3001/sdk/lib/html/dart2js/html_...
File sdk/lib/html/dart2js/html_dart2js.dart (right):

https://codereview.chromium.org/12092114/diff/3001/sdk/lib/html/dart2js/html_...
sdk/lib/html/dart2js/html_dart2js.dart:5835: +   * onDrop event) and will return
`null` before then.
On 2013/02/01 18:40:45, Andrei Mouravski wrote:
> Nit: link to onDrop event if you can.

Done.

https://codereview.chromium.org/12092114/diff/3001/sdk/lib/html/dart2js/html_...
sdk/lib/html/dart2js/html_dart2js.dart:5835: +   * onDrop event) and will return
`null` before then.
On 2013/02/01 18:40:45, Andrei Mouravski wrote:
> "... and will return `null` anywhere else."
> or maybe
> "... and will return `null` before the event is triggered."
> 
> Whichever is more accurate.

Done.

[Andrei Mouravski, 2013-04-10 02:33:50]

lgtm

[blois, 2013-04-10 16:22:21]

Committed patchset #3 manually as r21216 (presubmit successful).