Video Player: Supports bound arguments of wrap() util function

Video Player: Supports bound arguments of wrap() util function

And with this patch, the error detail is shown in case of error.

BUG=355545
TEST=manually tested
R=hirono@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275328

[yoshiki, 2014-06-05 03:22:55]

@hirono, PTAL. Thanks.

[hirono, 2014-06-05 08:47:42]

https://codereview.chromium.org/315103004/diff/40001/ui/file_manager/video_pl...
File ui/file_manager/video_player/js/error_util.js (right):

https://codereview.chromium.org/315103004/diff/40001/ui/file_manager/video_pl...
ui/file_manager/video_player/js/error_util.js:35: return function
wrappedCallback(var_args) {
The name of wrappedCallback is needed?

https://codereview.chromium.org/315103004/diff/40001/ui/file_manager/video_pl...
ui/file_manager/video_player/js/error_util.js:43: console.error(e.name + ': ' +
e.message, e.stack);
e.stack includes e.name and e.message.

[yoshiki, 2014-06-05 14:28:13]

@hirono, PTAL again? Thanks.

https://codereview.chromium.org/315103004/diff/40001/ui/file_manager/video_pl...
File ui/file_manager/video_player/js/error_util.js (right):

https://codereview.chromium.org/315103004/diff/40001/ui/file_manager/video_pl...
ui/file_manager/video_player/js/error_util.js:35: return function
wrappedCallback(var_args) {
On 2014/06/05 08:47:42, hirono wrote:
> The name of wrappedCallback is needed?

It's useful in backtrace.

https://codereview.chromium.org/315103004/diff/40001/ui/file_manager/video_pl...
ui/file_manager/video_player/js/error_util.js:43: console.error(e.name + ': ' +
e.message, e.stack);
On 2014/06/05 08:47:42, hirono wrote:
> e.stack includes e.name and e.message.

Done.

[hirono, 2014-06-06 04:13:16]

Thank you. lgtm!

[yoshiki, 2014-06-06 04:19:38]

The CQ bit was checked by yoshiki@chromium.org

[commit-bot: I haz the power, 2014-06-06 04:21:43]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yoshiki@chromium.org/315103004/50001

[yoshiki, 2014-06-06 04:56:41]

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