Remove redundant 'escape to quit fullscreen/mouse lock' code.

Remove redundant 'escape to quit fullscreen/mouse lock' code.

BUG=41781
TEST=Escape key should still exit mouse lock / tabbed fullscreen.

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

[yzshen1, 2011-10-15 03:25:53]

You may need to add the owner of these files as reviewer, too.

http://codereview.chromium.org/8301021/diff/1/content/browser/renderer_host/r...
File content/browser/renderer_host/render_widget_host_view_win.cc (left):

http://codereview.chromium.org/8301021/diff/1/content/browser/renderer_host/r...
content/browser/renderer_host/render_widget_host_view_win.cc:1295: if
(is_fullscreen_) {
This is the flash-style fullscreen. You should not remove this logic.

[koz (OOO until 15th September), 2011-10-15 04:12:34]

http://codereview.chromium.org/8301021/diff/1/content/browser/renderer_host/r...
File content/browser/renderer_host/render_widget_host_view_win.cc (left):

http://codereview.chromium.org/8301021/diff/1/content/browser/renderer_host/r...
content/browser/renderer_host/render_widget_host_view_win.cc:1295: if
(is_fullscreen_) {
On 2011/10/15 03:25:53, yzshen1 wrote:
> This is the flash-style fullscreen. You should not remove this logic.

Done.

[yzshen1, 2011-10-15 04:32:30]

http://codereview.chromium.org/8301021/diff/4001/content/browser/renderer_hos...
File content/browser/renderer_host/render_widget_host_view_win.cc (right):

http://codereview.chromium.org/8301021/diff/4001/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_win.cc:1292: absorbed =
true;
you could remove |absorbed| and return directly.

[koz (OOO until 15th September), 2011-10-15 04:36:55]

http://codereview.chromium.org/8301021/diff/4001/content/browser/renderer_hos...
File content/browser/renderer_host/render_widget_host_view_win.cc (right):

http://codereview.chromium.org/8301021/diff/4001/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_win.cc:1292: absorbed =
true;
On 2011/10/15 04:32:30, yzshen1 wrote:
> you could remove |absorbed| and return directly.

Done.

[brettw, 2011-10-15 04:37:00]

lgtm

[yzshen1, 2011-10-15 04:44:14]

lgtm