MultibufferDataSource: Handle HasSingleOrigin() before init completes.

MultibufferDataSource: Handle HasSingleOrigin() before init completes.

This extends the time that WebMediaPlayerImpl::hasSingleSecurityOrigin()
returns true from "until the data source exists" to "until the data source is
initialized". I've verified (by source code inspection) that every existing
caller is safe with respect to races here. It seems to be an undocumented
rule followed by Blink developers.

BUG=638763
Committed: https://crrev.com/dfe8ede1f60884311bb9c0c36417e31c53babef4
Cr-Commit-Position: refs/heads/master@{#418472}

[sandersd (OOO until July 31), 2016-09-13 22:40:48]

sandersd@chromium.org changed reviewers:
+ hubbe@chromium.org

[hubbe, 2016-09-13 22:43:23]

lgtm

https://codereview.chromium.org/2341483003/diff/1/media/blink/multibuffer_dat...
File media/blink/multibuffer_data_source.cc (right):

https://codereview.chromium.org/2341483003/diff/1/media/blink/multibuffer_dat...
media/blink/multibuffer_data_source.cc:254: return true;
single_origin_ defaults to true, so this if statement is not required I think.

[sandersd (OOO until July 31), 2016-09-13 22:50:37]

https://codereview.chromium.org/2341483003/diff/1/media/blink/multibuffer_dat...
File media/blink/multibuffer_data_source.cc (right):

https://codereview.chromium.org/2341483003/diff/1/media/blink/multibuffer_dat...
media/blink/multibuffer_data_source.cc:254: return true;
On 2016/09/13 22:43:23, hubbe wrote:
> single_origin_ defaults to true, so this if statement is not required I think.

But now it looks like it took me seconds to make this CL, instead of minutes!

[sandersd (OOO until July 31), 2016-09-13 22:50:41]

The CQ bit was checked by sandersd@chromium.org

[sandersd (OOO until July 31), 2016-09-13 22:50:42]

The patchset sent to the CQ was uploaded after l-g-t-m from hubbe@chromium.org
Link to the patchset: https://codereview.chromium.org/2341483003/#ps20001
(title: "Remove code.")

[commit-bot: I haz the power, 2016-09-13 22:51:34]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-14 03:11:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-14 03:13:50]

Description was changed from

==========
MultibufferDataSource: Handle HasSingleOrigin() before init completes.

This extends the time that WebMediaPlayerImpl::hasSingleSecurityOrigin()
returns true from "until the data source exists" to "until the data source is
initialized". I've verified (by source code inspection) that every existing
caller is safe with respect to races here. It seems to be an undocumented
rule followed by Blink developers.

BUG=638763
==========

to

==========
MultibufferDataSource: Handle HasSingleOrigin() before init completes.

This extends the time that WebMediaPlayerImpl::hasSingleSecurityOrigin()
returns true from "until the data source exists" to "until the data source is
initialized". I've verified (by source code inspection) that every existing
caller is safe with respect to races here. It seems to be an undocumented
rule followed by Blink developers.

BUG=638763

Committed: https://crrev.com/dfe8ede1f60884311bb9c0c36417e31c53babef4
Cr-Commit-Position: refs/heads/master@{#418472}
==========

[commit-bot: I haz the power, 2016-09-14 03:13:50]

Patchset 2 (id:??) landed as
https://crrev.com/dfe8ede1f60884311bb9c0c36417e31c53babef4
Cr-Commit-Position: refs/heads/master@{#418472}