Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

type 'AudioBufferImplementation' is not a subtype of type 'AudioBuffer' #3101

Closed
DartBot opened this issue May 17, 2012 · 6 comments
Closed

Comments

@DartBot
Copy link

DartBot commented May 17, 2012

This issue was originally filed by @bp74


What steps will reproduce the problem?

  1. load some audio with XMLHttpRequest
  2. decode the audio
  3. the success callback fails in "Checked Mode"

like this Dart Audio API sample does...
http://news.dartlang.org/2012/02/web-audio-api-and-dart.html

What is the expected output? What do you see instead?

http://api.dartlang.org/html/AudioContext.html

void decodeAudioData(ArrayBuffer audioData,
    bool successCallback(AudioBuffer audioBuffer),
    [bool errorCallback(AudioBuffer audioBuffer)])

The "successCallback" should be an (AudioBuffer) => bool.
But when you run "Checked Mode" it says:

Exception: type '(AudioBuffer) => bool' is not a subtype of type 'AudioBufferCallback' of 'successCallback'.
or
Exception: type 'AudioBufferImplementation' is not a subtype of type 'AudioBuffer' of 'value'.

What version of the product are you using? On what operating system?
Dart Editor Build 7696 32 bit, Windows 7 64 bit

@DartBot
Copy link
Author

DartBot commented May 17, 2012

This comment was originally written by [email protected]


Thanks a lot for report, Bernhard.

That's a case with which we don't deal correctly yet and it's related to the fact that we wrap dart:dom_depercated objects in dart:html.

We expect to get rid of this wrapping soon (in a couple of weeks).

How important this problem is for you?


Added Area-Dartium, Triaged labels.

@DartBot
Copy link
Author

DartBot commented May 17, 2012

This comment was originally written by [email protected]


Added Accepted label.

@DartBot
Copy link
Author

DartBot commented May 17, 2012

This comment was originally written by @bp74


Thank you for the response. It's not very important because i have two workarounds for it: Do not use "Checked Mode" or use another code path which uses the AudioElement for audio playback. I'm sure you have a long list of ToDos, so fix it when you think it's the right time :)

@DartBot
Copy link
Author

DartBot commented May 17, 2012

This comment was originally written by [email protected]


Thank you, Bernhard.

@DartBot
Copy link
Author

DartBot commented May 30, 2012

This comment was originally written by @bp74


Tested with Dart Editor 8124 and it works now! Thanks.

@DartBot
Copy link
Author

DartBot commented May 31, 2012

This comment was originally written by [email protected]


Thanks for checking, Bernhard!


Set owner to [email protected].
Added Fixed label.

dart-bot pushed a commit that referenced this issue Sep 13, 2021
New commits in this version:

git -C third_party/pkg/pub log --oneline cd7a43f2109f7e5eb22e73c7f4e15d25fd57598e..d95c5713dda518ed53ada70e00789e6aadbfbe48
d95c5713 (HEAD, origin/master, origin/HEAD) Remove duplicate global invocation on Windows (#3055)
4c5198df master->main (#3101)
e793fd52 More tool/test.dart tweaks (#3097)
1b228edd Report retracted packages (#3093)
4fedb6c5 Tweak strict dependencies error message for `_validateBenchmarkTestTool` (#3087)
e608ab6e Improve test script (#3091)
abf702c4 Detect potential leaks in `dart pub publish`. (#3049)
9941c1f9 Fix broken simplification of prerelease constraints (#3078)
f0cdfa27 format (#3086)
5a1242c6 Fix unicode terminal detection windows (#2933)
58e2296d Dart format (#3084)
1426601c use incremental compilation in the tool/test.dart script (#3075)
9954f851 Fix a typo (#3062)
44489b31 Use relative import for path.dart (#2959)
77702ab1 Always precompile scripts before running them (#3074)

Change-Id: I913ab9e3b5bc7181d483a2de96ec4ad917028b75
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/213262
Reviewed-by: Jonas Jensen <[email protected]>
Commit-Queue: Sigurd Meldgaard <[email protected]>
copybara-service bot pushed a commit that referenced this issue Aug 5, 2022
…2 revisions)

https://dart.googlesource.com/dartdoc/+log/bd57c0e7b756..f419695f57c5

2022-08-05 [email protected] Fork built-in features into _BuiltInFeature (#3105)
2022-08-04 [email protected] Adopt badge design for all features #3047 (#3101)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/dart-doc-dart-sdk
Please CC [email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Dart Documentation Generator: https://github.com/dart-lang/dartdoc/issues
To file a bug in Dart SDK: https://github.com/dart-lang/sdk/issues

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Tbr: [email protected]
Change-Id: I6b7680fa427fac056756e69700c005ae69bedc79
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/253721
Reviewed-by: Devon Carew <[email protected]>
Commit-Queue: Devon Carew <[email protected]>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant