Re: Issue 360251 in chromium: blink::WebMediaPlayer::didLoadingProgress() should not be const.

[chro...@googlecode.com]

Comment #3 on issue 360251 by bugdro...@chromium.org:
blink::WebMediaPlayer::didLoadingProgress() should not be const.
http://code.google.com/p/chromium/issues/detail?id=360251#c3

The following revision refers to this bug:
http://src.chromium.org/viewvc/blink?view=rev&rev=173980

------------------------------------------------------------------
r173980 | sand...@chromium.org | 2014-05-13T19:10:07.556818Z

Changed paths:
M
http://src.chromium.org/viewvc/blink/trunk/public/platform/WebMediaPlayer.h?r1=173980&r2=173979&pathrev=173980

Make WebMediaPlayer::didLoadingProgress() non-const.

The Chromium-side change is https://codereview.chromium.org/275303002,
which must be committed first.

BUG=360251

Review URL: https://codereview.chromium.org/272223004
-----------------------------------------------------------------

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

[chro...@googlecode.com]

Comment #4 on issue 360251 by bugdro...@chromium.org:

blink::WebMediaPlayer::didLoadingProgress() should not be const.

http://code.google.com/p/chromium/issues/detail?id=360251#c4

The following revision refers to this bug:

https://chromium.googlesource.com/chromium/src.git/+/4c4583b3554d660539ff7c7feff866f17400f9b5

commit 4c4583b3554d660539ff7c7feff866f17400f9b5
Author: sand...@chromium.org
<sand...@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Date: Thu May 15 00:02:32 2014

Remove const WebMediaPlayer::didLoadingProgress() implementations.

This is the third of three changes; the others are:
https://codereview.chromium.org/275303002 (chromium prep)
https://codereview.chromium.org/272223004 (blink side)

BUG=360251

Review URL: https://codereview.chromium.org/284723003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270520
0039d316-1c4b-4281-b951-d872f2087c98

[chro...@googlecode.com]

Comment #5 on issue 360251 by bugdro...@chromium.org:

blink::WebMediaPlayer::didLoadingProgress() should not be const.

http://code.google.com/p/chromium/issues/detail?id=360251#c5

------------------------------------------------------------------
r270520 | sand...@chromium.org | 2014-05-15T00:02:32.273393Z

Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/content/renderer/media/webmediaplayer_impl.h?r1=270520&r2=270519&pathrev=270520
M
http://src.chromium.org/viewvc/chrome/trunk/src/content/renderer/media/webmediaplayer_ms.cc?r1=270520&r2=270519&pathrev=270520
M
http://src.chromium.org/viewvc/chrome/trunk/src/content/renderer/media/android/webmediaplayer_android.cc?r1=270520&r2=270519&pathrev=270520
M
http://src.chromium.org/viewvc/chrome/trunk/src/content/renderer/media/webmediaplayer_impl.cc?r1=270520&r2=270519&pathrev=270520
M
http://src.chromium.org/viewvc/chrome/trunk/src/content/renderer/media/webmediaplayer_ms.h?r1=270520&r2=270519&pathrev=270520
M
http://src.chromium.org/viewvc/chrome/trunk/src/content/renderer/media/android/webmediaplayer_android.h?r1=270520&r2=270519&pathrev=270520

Remove const WebMediaPlayer::didLoadingProgress() implementations.

This is the third of three changes; the others are:
https://codereview.chromium.org/275303002 (chromium prep)
https://codereview.chromium.org/272223004 (blink side)

BUG=360251

Review URL: https://codereview.chromium.org/284723003

-----------------------------------------------------------------

[chro...@googlecode.com]

Updates:
Status: Fixed
Owner: sande...@chromium.org

Comment #6 on issue 360251 by sande...@chromium.org:

blink::WebMediaPlayer::didLoadingProgress() should not be const.
http://code.google.com/p/chromium/issues/detail?id=360251

(No comment was entered for this change.)