Fix nit which is unefficiant sequence to deleteMarkedNodes.

Fix nit which is unefficiant sequence to deleteMarkedNodes.

deleteMarkedNodes need to move postion to prevent unefficiant do while loop.

BUG=none

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=172279

[KhNo, 2014-04-22 12:14:49]

Please review. It has been separated.

[Raymond Toy, 2014-04-22 17:25:25]

lgtm

BTW, just add my chromium address for review; I don't think my google address is
needed anymore.

https://codereview.chromium.org/246793004/diff/1/Source/modules/webaudio/Audi...
File Source/modules/webaudio/AudioContext.cpp (right):

https://codereview.chromium.org/246793004/diff/1/Source/modules/webaudio/Audi...
Source/modules/webaudio/AudioContext.cpp:211: } while (m_nodesToDelete.size());
With this change, is this loop ever executed more than once? If the audio thread
is dead, m_nodesMarkedForDeletion should never grow, right?

[KhNo, 2014-04-22 17:26:09]

On 2014/04/22 17:25:25, Raymond Toy wrote:
> lgtm
> 
> BTW, just add my chromium address for review; I don't think my google address
is
> needed anymore.
> 
>
https://codereview.chromium.org/246793004/diff/1/Source/modules/webaudio/Audi...
> File Source/modules/webaudio/AudioContext.cpp (right):
> 
>
https://codereview.chromium.org/246793004/diff/1/Source/modules/webaudio/Audi...
> Source/modules/webaudio/AudioContext.cpp:211: } while
(m_nodesToDelete.size());
> With this change, is this loop ever executed more than once? If the audio
thread
> is dead, m_nodesMarkedForDeletion should never grow, right?

Oh. I see. :)

[KhNo, 2014-04-22 17:26:14]

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

[commit-bot: I haz the power, 2014-04-22 17:26:26]

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

[commit-bot: I haz the power, 2014-04-22 18:12:44]

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

[commit-bot: I haz the power, 2014-04-22 18:12:44]

Try jobs failed on following builders:
  tryserver.blink on win_blink_rel

[KhNo, 2014-04-22 20:11:06]

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

[commit-bot: I haz the power, 2014-04-22 20:13:30]

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

[commit-bot: I haz the power, 2014-04-22 23:35:10]

Change committed as 172279