Changed VideoCaptureController to use correct session id when stopping the capture device.

Changed VideoCaptureController to use correct session id when stopping the capture device.



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

[mflodman_chromium_OOO, 2012-02-13 13:18:30]

Wei,
This solves the shut-down problem I saw when closing multiple tabs, using the
same capture device, in the same order as the tabs were created.

Please have a look and let me know what you think. If you'd like me to proceed
with this, I'll make some other changes in VideoCaptureManager destructor
together with http://codereview.chromium.org/9320070/

[wjia(left Chromium), 2012-02-13 22:08:43]

LGTM.

Thanks for taking care of this! I am thinking of also adding some code in VCM so
that when a device is closed, the corresponding controller will be deleted (ref
count--). refer to http://codereview.chromium.org/9386020/.

[commit-bot: I haz the power, 2012-02-14 12:08:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mflodman@chromium.org/9389006/3001

[commit-bot: I haz the power, 2012-02-14 13:37:47]

Commit queue rejected this change because the description was changed
between the time the change entered the commit queue and the time it
was ready to commit. You can safely check the commit box again.

[commit-bot: I haz the power, 2012-02-14 13:39:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mflodman@chromium.org/9389006/3001

[commit-bot: I haz the power, 2012-02-14 15:07:23]

Change committed as 121886