diff options
| author | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-27 02:00:27 +0000 |
|---|---|---|
| committer | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-27 02:00:27 +0000 |
| commit | 5a1478c1b434931455337ecab126c6b6015d7eac (patch) | |
| tree | b4cd70b2f30e22f47ba85807d46f64a7d5630780 | |
| parent | bab44ba4d991eb71e696cad6afacf21d0bf1b1e8 (diff) | |
| download | chromium_src-5a1478c1b434931455337ecab126c6b6015d7eac.zip chromium_src-5a1478c1b434931455337ecab126c6b6015d7eac.tar.gz chromium_src-5a1478c1b434931455337ecab126c6b6015d7eac.tar.bz2 | |
Not DefProc'ing the NCActivate seems to cause the process not to be shown as activate when originally shown.
BUG=4513
TEST=Start Chrome, the Chrome entry in the task bar should be shown as active.
R=ben
Review URL: http://codereview.chromium.org/28240
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10566 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/views/custom_frame_window.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/chrome/views/custom_frame_window.cc b/chrome/views/custom_frame_window.cc index 79ce069..110750f 100644 --- a/chrome/views/custom_frame_window.cc +++ b/chrome/views/custom_frame_window.cc @@ -997,7 +997,10 @@ LRESULT CustomFrameWindow::OnNCActivate(BOOL active) { PaintNow(root_view_->GetScheduledPaintRect()); } - return TRUE; + // Defering to our parent as it is important that the NCActivate message gets + // DefProc'ed or the task bar won't show our process as active. + // See bug http://crbug.com/4513. + return Window::OnNCActivate(active); } LRESULT CustomFrameWindow::OnNCCalcSize(BOOL mode, LPARAM l_param) { |