Fix the bug that closing a tab which owns a fullscreen Flash widget causes crash.

Fix the bug that closing a tab which owns a fullscreen Flash widget causes crash.

BUG=NONE
TEST=1. goes to http://www.vimeo.com/hd
     2. play any video and enter fullscreen mode.
     3. right click to open Flash context menu.
     4. choose Global Settings..., which causes the browser window to be brought to the foreground.
     5. close the tab of http://www.vimeo.com/hd
     6. the settings page should not crash.


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

[yzshen1, 2011-10-07 22:11:02]

Hi, Antoine.
Please review this CL. Thanks!

[piman, 2011-10-07 22:36:22]

lgtm

LGTM

http://codereview.chromium.org/8198013/diff/1/content/renderer/render_widget_...
File content/renderer/render_widget_fullscreen_pepper.cc (right):

http://codereview.chromium.org/8198013/diff/1/content/renderer/render_widget_...
content/renderer/render_widget_fullscreen_pepper.cc:192:
(widget_->plugin()->GetBackingTextureId() != 0);
nit: indentation (4 spaces).

[yzshen1, 2011-10-07 22:48:14]

Thanks, Antoine!

http://codereview.chromium.org/8198013/diff/1/content/renderer/render_widget_...
File content/renderer/render_widget_fullscreen_pepper.cc (right):

http://codereview.chromium.org/8198013/diff/1/content/renderer/render_widget_...
content/renderer/render_widget_fullscreen_pepper.cc:192:
(widget_->plugin()->GetBackingTextureId() != 0);
On 2011/10/07 22:36:22, piman wrote:
> nit: indentation (4 spaces).

Done.

[commit-bot: I haz the power, 2011-10-10 03:40:49]

CQ is trying the patch. Follow status at
https://chromium-status.appspot.com/cq/yzshen@chromium.org/8198013/3001

[commit-bot: I haz the power, 2011-10-10 05:06:54]

Change committed as 104693