Move logic that sets variable m_isInitialized to false before stop destination.

Move logic that sets variable m_isInitialized to false before stop destination.

During uninitialize a destination node, audio thread can be entered
AudioIOCallback's render() or OfflineAudioContext's offlinerender.

Although the destination node already destroyed, audio thread can be run for while
since it is running on separated thread.

To block meeting this condition, the context()->isInitialized() must be quarantee,
if Audio destination is uninitializing, alwasy false.

BUG=420814, 417193, 428233, 431109
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185093

[KhNo, 2014-11-01 05:53:54]

keonho07.kim@samsung.com changed reviewers:
+ rtoy@chromium.org

[KhNo, 2014-11-01 05:53:54]

PTAL.

[Raymond Toy, 2014-11-07 18:43:41]

On 2014/11/01 05:53:54, KhNo wrote:
> PTAL.

Does derefUnfinishedNodes have any dependencies on m_isInitialized?

[KhNo, 2014-11-08 04:47:11]

On 2014/11/07 18:43:41, Raymond Toy wrote:
> On 2014/11/01 05:53:54, KhNo wrote:
> > PTAL.
> 
> Does derefUnfinishedNodes have any dependencies on m_isInitialized?

Yes, it doesn't have dependency with it.
I have checked with the log whether isInitialized() is called or not during
derefUnfinishedNodes().
Since m_isInitialized is private variable, out of AudioContext module has to
refer with isInitialized().

[Raymond Toy, 2014-11-10 21:48:11]

rtoy@chromium.org changed reviewers:
+ hongchan@chromium.org

[Raymond Toy, 2014-11-10 21:48:12]

lgtm

[KhNo, 2014-11-11 01:53:51]

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

[commit-bot: I haz the power, 2014-11-11 01:54:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/694993003/1

[commit-bot: I haz the power, 2014-11-11 02:55:07]

Committed patchset #1 (id:1) as 185093