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

Side by Side Diff: ui/views/win/hwnd_message_handler.h

Issue 1406403007: Eliminate HICON leaks caused by creating icons from bitmap image. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef UI_VIEWS_WIN_HWND_MESSAGE_HANDLER_H_ 5 #ifndef UI_VIEWS_WIN_HWND_MESSAGE_HANDLER_H_
6 #define UI_VIEWS_WIN_HWND_MESSAGE_HANDLER_H_ 6 #define UI_VIEWS_WIN_HWND_MESSAGE_HANDLER_H_
7 7
8 #include <windows.h> 8 #include <windows.h>
9 9
10 #include <set> 10 #include <set>
(...skipping 499 matching lines...) Expand 10 before | Expand all | Expand 10 after
510 // true. 510 // true.
511 bool restored_enabled_; 511 bool restored_enabled_;
512 512
513 // The current cursor. 513 // The current cursor.
514 HCURSOR current_cursor_; 514 HCURSOR current_cursor_;
515 515
516 // The last cursor that was active before the current one was selected. Saved 516 // The last cursor that was active before the current one was selected. Saved
517 // so that we can restore it. 517 // so that we can restore it.
518 HCURSOR previous_cursor_; 518 HCURSOR previous_cursor_;
519 519
520 // The icon created from the bitmap image of the window icon.
521 base::win::ScopedHICON window_icon_;
522
523 // The icon created from the bitmap image of the app icon.
524 base::win::ScopedHICON app_icon_;
525
520 // Event handling ------------------------------------------------------------ 526 // Event handling ------------------------------------------------------------
521 527
522 // The flags currently being used with TrackMouseEvent to track mouse 528 // The flags currently being used with TrackMouseEvent to track mouse
523 // messages. 0 if there is no active tracking. The value of this member is 529 // messages. 0 if there is no active tracking. The value of this member is
524 // used when tracking is canceled. 530 // used when tracking is canceled.
525 DWORD active_mouse_tracking_flags_; 531 DWORD active_mouse_tracking_flags_;
526 532
527 // Set to true when the user presses the right mouse button on the caption 533 // Set to true when the user presses the right mouse button on the caption
528 // area. We need this so we can correctly show the context menu on mouse-up. 534 // area. We need this so we can correctly show the context menu on mouse-up.
529 bool is_right_mouse_pressed_on_caption_; 535 bool is_right_mouse_pressed_on_caption_;
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
616 622
617 // The factory used with BEGIN_SAFE_MSG_MAP_EX. 623 // The factory used with BEGIN_SAFE_MSG_MAP_EX.
618 base::WeakPtrFactory<HWNDMessageHandler> weak_factory_; 624 base::WeakPtrFactory<HWNDMessageHandler> weak_factory_;
619 625
620 DISALLOW_COPY_AND_ASSIGN(HWNDMessageHandler); 626 DISALLOW_COPY_AND_ASSIGN(HWNDMessageHandler);
621 }; 627 };
622 628
623 } // namespace views 629 } // namespace views
624 630
625 #endif // UI_VIEWS_WIN_HWND_MESSAGE_HANDLER_H_ 631 #endif // UI_VIEWS_WIN_HWND_MESSAGE_HANDLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698