Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(313)

Issue 16140022: Issue 245137: Mouse-move events not sent after clicking on drop-down toolbar (Closed)

Created:
7 years, 6 months ago by weitao
Modified:
7 years, 6 months ago
Reviewers:
Sergey Ulanov, Jamie, Wez
CC:
chromium-reviews, jamiewalch+watch_chromium.org, dcaiafa+watch_chromium.org, hclam+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, rmsousa+watch_chromium.org, weitaosu+watch_chromium.org, alexeypa+watch_chromium.org, sergeyu+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Issue 245137: Mouse-move events not sent after clicking on drop-down toolbar Revert "Issue 236549: Inactive window should not be capturing mouseover" This reverts commit bb5596a07d45d3a4832b337c62a22e2c3f90a7ee. See issue 246335: pepper plugin no longer gets notified on of didChangeFocus. BUG=245137 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203904

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -56 lines) Patch
M remoting/client/plugin/chromoting_instance.h View 1 chunk +0 lines, -1 line 0 comments Download
M remoting/client/plugin/chromoting_instance.cc View 1 chunk +0 lines, -6 lines 0 comments Download
M remoting/client/plugin/pepper_input_handler.h View 1 chunk +0 lines, -9 lines 0 comments Download
M remoting/client/plugin/pepper_input_handler.cc View 3 chunks +27 lines, -40 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
weitao
This is a regression caused by: Issue 236549: Inactive window should not be capturing mouseover ...
7 years, 6 months ago (2013-06-03 21:05:21 UTC) #1
Wez
On 2013/06/03 21:05:21, weitaosu wrote: > This is a regression caused by: > Issue 236549: ...
7 years, 6 months ago (2013-06-03 21:40:28 UTC) #2
Wez
lgtm
7 years, 6 months ago (2013-06-03 21:41:45 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/weitaosu@chromium.org/16140022/1
7 years, 6 months ago (2013-06-03 21:41:56 UTC) #4
commit-bot: I haz the power
7 years, 6 months ago (2013-06-04 07:51:35 UTC) #5
Message was sent while issue was closed.
Change committed as 203904

Powered by Google App Engine
This is Rietveld 408576698