Remove CHECK and NOTREACHED for mouse lock, fix full screen transition.

Remove CHECK and NOTREACHED for mouse lock, fix full screen transition.
BUG=http://code.google.com/p/nativeclient/issues/detail?id=2937
TEST=From Dust, ppapi tests

[jschuh, 2012-08-03 00:19:17]

lgtm

[viettrungluu, 2012-08-03 00:50:04]

https://chromiumcodereview.appspot.com/10824156/diff/1/webkit/plugins/ppapi/p...
File webkit/plugins/ppapi/ppapi_plugin_instance.cc (right):

https://chromiumcodereview.appspot.com/10824156/diff/1/webkit/plugins/ppapi/p...
webkit/plugins/ppapi/ppapi_plugin_instance.cc:1636: if
(!TrackedCallback::IsPending(lock_mouse_callback_)) {
Get rid of the braces (for consistency with the other one-line |if|s in this
file).

Or, better yet, change this to:

if (TrackedCallback::IsPending(...)) {
  TrackedCallback::ClearAndRun(...
                               ...);
}

[viettrungluu, 2012-08-03 00:50:38]

(LGTM with style nit fixed, though.)

On 2012/08/03 00:50:04, viettrungluu wrote:
>
https://chromiumcodereview.appspot.com/10824156/diff/1/webkit/plugins/ppapi/p...
> File webkit/plugins/ppapi/ppapi_plugin_instance.cc (right):
> 
>
https://chromiumcodereview.appspot.com/10824156/diff/1/webkit/plugins/ppapi/p...
> webkit/plugins/ppapi/ppapi_plugin_instance.cc:1636: if
> (!TrackedCallback::IsPending(lock_mouse_callback_)) {
> Get rid of the braces (for consistency with the other one-line |if|s in this
> file).
> 
> Or, better yet, change this to:
> 
> if (TrackedCallback::IsPending(...)) {
>   TrackedCallback::ClearAndRun(...
>                                ...);
> }

[commit-bot: I haz the power, 2012-08-03 01:11:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nfullagar@google.com/10824156/6

[commit-bot: I haz the power, 2012-08-03 02:48:22]

Change committed as 149789

[Brad Chen, 2012-08-03 16:57:50]

On 2012/08/03 02:48:22, I haz the power (commit-bot) wrote:
> Change committed as 149789

This change has been approved for merge into the M21 release branch. Nicholas
can you please merge into 1180 and 1180_87?