Remove unnecassary parameter "sampleRate" for loadAudioResource().

Remove unnecassary parameter "sampleRate" for loadAudioResource().

Parameter "sampleRate" is not required for loadAudioResource().
There is FIXME in AudioBus.cpp in blink
FIXME: the sampleRate parameter is ignored. It should be removed from the API.

It will be consist of 3 steps.
1. Removing "sampleRate" in chrome which is called by blink. (included deprecated method)
https://codereview.chromium.org/195043002/

2. Removing "sampleRate" in blink.
https://codereview.chromium.org/194753005/

3. Removing deprecated method from #1
https://codereview.chromium.org/199573004/

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

[KhNo, 2014-03-11 13:17:40]

Please review this patch.

[Raymond Toy, 2014-03-11 16:49:00]

You didn't say what other patch, but for the record it is
https://codereview.chromium.org/194753005/.

Why are these separate patches? Why not combine them into one?

[KhNo, 2014-03-11 16:54:10]

On 2014/03/11 16:49:00, Raymond Toy wrote:
> You didn't say what other patch, but for the record it is
> https://codereview.chromium.org/194753005/.
> 
> Why are these separate patches? Why not combine them into one?

Git path was different. Is there the way can upload togather?

[Raymond Toy, 2014-03-11 17:20:05]

On 2014/03/11 16:54:10, KhNo wrote:
> On 2014/03/11 16:49:00, Raymond Toy wrote:
> > You didn't say what other patch, but for the record it is
> > https://codereview.chromium.org/194753005/.
> > 
> > Why are these separate patches? Why not combine them into one?
> 
> Git path was different. Is there the way can upload togather?

Sorry, my fault. One is in blink and the other in chrome, so they have to be
different.

So this patch needs to be done first?

[Raymond Toy, 2014-03-11 18:03:06]

In the description, I think a link to the other patch is useful. A comment on
which needs to be applied first is useful.

https://codereview.chromium.org/195043002/diff/1/content/renderer/renderer_we...
File content/renderer/renderer_webkitplatformsupport_impl.cc (right):

https://codereview.chromium.org/195043002/diff/1/content/renderer/renderer_we...
content/renderer/renderer_webkitplatformsupport_impl.cc:797: size_t data_size) {
Can you make sample_rate an optional parameter instead of adding a new
deprecated method?

[KhNo, 2014-03-12 01:13:22]

On 2014/03/11 17:20:05, Raymond Toy wrote:
> On 2014/03/11 16:54:10, KhNo wrote:
> > On 2014/03/11 16:49:00, Raymond Toy wrote:
> > > You didn't say what other patch, but for the record it is
> > > https://codereview.chromium.org/194753005/.
> > > 
> > > Why are these separate patches? Why not combine them into one?
> > 
> > Git path was different. Is there the way can upload togather?
> 
> Sorry, my fault. One is in blink and the other in chrome, so they have to be
> different.
> 
> So this patch needs to be done first?

Yes, It is the first. Thanks for adding dependency codereview url.
Final goal is removing "sampleRate" parameter.
It will be consist of 3 steps.
1. Removing "sampleRate" in chrome which is called by blink. (included
deprecated method)
2. Removing "sampleRate" in blink.
3. Removing deprecated method from #1

[KhNo, 2014-03-12 01:15:54]

On 2014/03/12 01:13:22, KhNo wrote:
> On 2014/03/11 17:20:05, Raymond Toy wrote:
> > On 2014/03/11 16:54:10, KhNo wrote:
> > > On 2014/03/11 16:49:00, Raymond Toy wrote:
> > > > You didn't say what other patch, but for the record it is
> > > > https://codereview.chromium.org/194753005/.
> > > > 
> > > > Why are these separate patches? Why not combine them into one?
> > > 
> > > Git path was different. Is there the way can upload togather?
> > 
> > Sorry, my fault. One is in blink and the other in chrome, so they have to be
> > different.
> > 
> > So this patch needs to be done first?
> 
Yes, It is the first. Thanks for adding dependency codereview url.
Final goal is removing "sampleRate" parameter.
It will be consist of 3 steps.
1. Removing "sampleRate" in chrome which is called by blink. (included
deprecated method)
https://codereview.chromium.org/195043002/

2. Removing "sampleRate" in blink.
https://codereview.chromium.org/194753005/

3. Removing deprecated method from #1

[KhNo, 2014-03-12 01:51:31]

https://codereview.chromium.org/195043002/diff/1/content/renderer/renderer_we...
File content/renderer/renderer_webkitplatformsupport_impl.cc (right):

https://codereview.chromium.org/195043002/diff/1/content/renderer/renderer_we...
content/renderer/renderer_webkitplatformsupport_impl.cc:797: size_t data_size) {
On 2014/03/11 18:03:07, Raymond Toy wrote:
> Can you make sample_rate an optional parameter instead of adding a new
> deprecated method?

Actually I tried, but there are definations of loadAudioResource in both side.

public/platform/Platform.h ( blink)
virtual bool loadAudioResource(WebAudioBus* destinationBus, const char*
audioFileData, size_t dataSize) { return false; }

content/renderer/renderer_webkitplatformsupport_impl.h ( chrome )
   virtual bool loadAudioResource(
       blink::WebAudioBus* destination_bus, const char* audio_file_data, size_t
data_size, double sample_rate);

I can not do it. :(

[Raymond Toy, 2014-03-12 02:42:41]

lgtm

[KhNo, 2014-03-12 15:02:49]

On 2014/03/12 02:42:41, Raymond Toy wrote:
> lgtm

Ken Russell, James, Can I get OWNER's LGTM?

[Ken Russell (switch to Gerrit), 2014-03-12 16:33:25]

I'm not an OWNER in any of these directories. Please consult the OWNERS files in
them to try to find an appropriate reviewer.

[jamesr, 2014-03-12 21:26:52]

lgtm

[KhNo, 2014-03-12 22:15:56]

The CQ bit was checked by keonho07.kim@samsung.com

[commit-bot: I haz the power, 2014-03-12 22:16:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/keonho07.kim@samsung.com/195043002/1

[commit-bot: I haz the power, 2014-03-12 22:32:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-12 22:32:18]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[jamesr, 2014-03-12 22:34:20]

The PRESUBMIT says:

** Presubmit Warnings **
keonho07.kim@samsung.com is not in AUTHORS file. If you are a new contributor,
please visit
http://www.chromium.org/developers/contributing-code and read the "Legal"
section
If you are a chromite, verify the contributor signed the CLA.

I do see keonho07.kim@samsung.com in our CLA signers list, so you'll just have
to add yourself to AUTHORS

[KhNo, 2014-03-13 04:28:22]

The CQ bit was checked by keonho07.kim@samsung.com

[commit-bot: I haz the power, 2014-03-13 04:29:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/keonho07.kim@samsung.com/195043002/20001

[KhNo, 2014-03-13 04:37:40]

On 2014/03/12 22:34:20, jamesr wrote:
> The PRESUBMIT says:
> 
> ** Presubmit Warnings **
> mailto:keonho07.kim@samsung.com is not in AUTHORS file. If you are a new
contributor,
> please visit
> http://www.chromium.org/developers/contributing-code and read the "Legal"
> section
> If you are a chromite, verify the contributor signed the CLA.
> 
> I do see mailto:keonho07.kim@samsung.com in our CLA signers list, so you'll
just have
> to add yourself to AUTHORS

Thanks for kind your help. :)

[commit-bot: I haz the power, 2014-03-13 05:36:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-13 05:36:39]

Retried try job too often on android_clang_dbg for step(s) slave_steps
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[KhNo, 2014-03-13 05:44:36]

The CQ bit was checked by keonho07.kim@samsung.com

[KhNo, 2014-03-13 05:44:39]

The CQ bit was unchecked by keonho07.kim@samsung.com

[KhNo, 2014-03-13 05:51:05]

The CQ bit was checked by keonho07.kim@samsung.com

[commit-bot: I haz the power, 2014-03-13 05:51:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/keonho07.kim@samsung.com/195043002/40001

[commit-bot: I haz the power, 2014-03-13 05:57:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-13 05:57:28]

Try jobs failed on following builders:
  tryserver.chromium on win_chromium_compile_dbg

[Raymond Toy, 2014-03-13 05:59:35]

On 2014/03/13 05:51:30, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>
https://chromium-status.appspot.com/cq/keonho07.kim%40samsung.com/195043002/4...

Check the failure from the android_clang_dbg bot:
http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...

This looks like a real failure that you need to fix.

[KhNo, 2014-03-13 08:45:44]

The CQ bit was checked by keonho07.kim@samsung.com

[commit-bot: I haz the power, 2014-03-13 08:45:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/keonho07.kim@samsung.com/195043002/40001

[commit-bot: I haz the power, 2014-03-13 10:56:27]

Change committed as 256800

[KhNo, 2014-03-13 17:04:48]

On 2014/03/13 05:59:35, Raymond Toy wrote:
> On 2014/03/13 05:51:30, I haz the power (commit-bot) wrote:
> > CQ is trying da patch. Follow status at
> >
>
https://chromium-status.appspot.com/cq/keonho07.kim%2540samsung.com/195043002...
> 
> Check the failure from the android_clang_dbg bot:
>
http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...
> 
> This looks like a real failure that you need to fix.

Sorry, make you concern. I missed "git add audio_decoder_android.h" when I
uploaded the fist patch set. :(
That was fixed and merged. I will keep upload next steps.

[Raymond Toy, 2014-03-13 18:01:45]

On 2014/03/13 17:04:48, KhNo wrote:
> On 2014/03/13 05:59:35, Raymond Toy wrote:
> > On 2014/03/13 05:51:30, I haz the power (commit-bot) wrote:
> > > CQ is trying da patch. Follow status at
> > >
> >
>
https://chromium-status.appspot.com/cq/keonho07.kim%252540samsung.com/1950430...
> > 
> > Check the failure from the android_clang_dbg bot:
> >
>
http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...
> > 
> > This looks like a real failure that you need to fix.
> 
> Sorry, make you concern. I missed "git add audio_decoder_android.h" when I
> uploaded the fist patch set. :(
> That was fixed and merged. I will keep upload next steps.

In general that probably should have required a further review, but I see the
change was very simple.

You may want to use "git cl try" next time to make sure things work, especially
if you're making android changes and you don't build on android yourself.

[KhNo, 2014-03-14 02:26:28]

On 2014/03/13 18:01:45, Raymond Toy wrote:
> On 2014/03/13 17:04:48, KhNo wrote:
> > On 2014/03/13 05:59:35, Raymond Toy wrote:
> > > On 2014/03/13 05:51:30, I haz the power (commit-bot) wrote:
> > > > CQ is trying da patch. Follow status at
> > > >
> > >
> >
>
https://chromium-status.appspot.com/cq/keonho07.kim%25252540samsung.com/19504...
> > > 
> > > Check the failure from the android_clang_dbg bot:
> > >
> >
>
http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...
> > > 
> > > This looks like a real failure that you need to fix.
> > 
> > Sorry, make you concern. I missed "git add audio_decoder_android.h" when I
> > uploaded the fist patch set. :(
> > That was fixed and merged. I will keep upload next steps.
> 
> In general that probably should have required a further review, but I see the
> change was very simple.
> 
> You may want to use "git cl try" next time to make sure things work,
especially
> if you're making android changes and you don't build on android yourself.

Oh, It is very nice.
Thanks information, Now, I'm reading
http://www.chromium.org/developers/testing/try-server-usage