More prep for CORS support in WebKit/chromium.

More prep for CORS support in WebKit/chromium.

BUG=123369
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=140949

[Ami GONE FROM CHROMIUM, 2012-06-07 00:07:29]

This is needed to support the latest patch in http://webk.it/88349

[Ami GONE FROM CHROMIUM, 2012-06-07 00:16:21]

If LGTY, please check the CQ box.

[scherkus (not reviewing), 2012-06-07 01:20:53]

LGTM w/ one q that might require updating docs on some methods / adding a dcheck

https://chromiumcodereview.appspot.com/10536044/diff/1/webkit/media/buffered_...
File webkit/media/buffered_resource_loader.h (right):

https://chromiumcodereview.appspot.com/10536044/diff/1/webkit/media/buffered_...
webkit/media/buffered_resource_loader.h:179: bool DidPassCORSAccessCheck()
const;
is it possible to have this method executed prior to Start() completing?

I'm curious because we have the constraint that Start()-ing must have completed
either via didReceiveResponse() or didFail() prior to HSO() can be called, but I
noticed that the canvas code also calls down into HSO()

[Ami GONE FROM CHROMIUM, 2012-06-07 01:36:49]

https://chromiumcodereview.appspot.com/10536044/diff/1/webkit/media/buffered_...
File webkit/media/buffered_resource_loader.h (right):

https://chromiumcodereview.appspot.com/10536044/diff/1/webkit/media/buffered_...
webkit/media/buffered_resource_loader.h:179: bool DidPassCORSAccessCheck()
const;
On 2012/06/07 01:20:53, scherkus wrote:
> is it possible to have this method executed prior to Start() completing?
> 
> I'm curious because we have the constraint that Start()-ing must have
completed
> either via didReceiveResponse() or didFail() prior to HSO() can be called, but
I
> noticed that the canvas code also calls down into HSO()

Done.

[commit-bot: I haz the power, 2012-06-07 01:37:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/10536044/6001

[commit-bot: I haz the power, 2012-06-07 04:00:02]

Change committed as 140949