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

Side by Side Diff: chrome/browser/ui/views/frame/browser_view.h

Issue 2753813003: Fix theme change paint issue on linux. (Closed)
Patch Set: Created 3 years, 9 months 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 CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
6 #define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_ 6 #define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
7 7
8 #include <map> 8 #include <map>
9 #include <memory> 9 #include <memory>
10 #include <string> 10 #include <string>
(...skipping 424 matching lines...) Expand 10 before | Expand all | Expand 10 after
435 435
436 // Overridden from views::View: 436 // Overridden from views::View:
437 const char* GetClassName() const override; 437 const char* GetClassName() const override;
438 void Layout() override; 438 void Layout() override;
439 void OnGestureEvent(ui::GestureEvent* event) override; 439 void OnGestureEvent(ui::GestureEvent* event) override;
440 void ViewHierarchyChanged( 440 void ViewHierarchyChanged(
441 const ViewHierarchyChangedDetails& details) override; 441 const ViewHierarchyChangedDetails& details) override;
442 void ChildPreferredSizeChanged(View* child) override; 442 void ChildPreferredSizeChanged(View* child) override;
443 void GetAccessibleNodeData(ui::AXNodeData* node_data) override; 443 void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
444 void OnThemeChanged() override; 444 void OnThemeChanged() override;
445 void OnNativeThemeChanged(const ui::NativeTheme* theme) override;
446 445
447 // Overridden from ui::AcceleratorTarget: 446 // Overridden from ui::AcceleratorTarget:
448 bool AcceleratorPressed(const ui::Accelerator& accelerator) override; 447 bool AcceleratorPressed(const ui::Accelerator& accelerator) override;
449 448
450 // OmniboxPopupModelObserver overrides 449 // OmniboxPopupModelObserver overrides
451 void OnOmniboxPopupShownOrHidden() override; 450 void OnOmniboxPopupShownOrHidden() override;
452 451
453 // ExclusiveAccessContext overrides 452 // ExclusiveAccessContext overrides
454 Profile* GetProfile() override; 453 Profile* GetProfile() override;
455 content::WebContents* GetActiveWebContents() override; 454 content::WebContents* GetActiveWebContents() override;
(...skipping 17 matching lines...) Expand all
473 // Creates an accessible tab label for screen readers that includes the tab 472 // Creates an accessible tab label for screen readers that includes the tab
474 // status for the given tab index. This takes the form of 473 // status for the given tab index. This takes the form of
475 // "Page title - Tab state". 474 // "Page title - Tab state".
476 base::string16 GetAccessibleTabLabel(bool include_app_name, int index) const; 475 base::string16 GetAccessibleTabLabel(bool include_app_name, int index) const;
477 476
478 // Testing interface: 477 // Testing interface:
479 views::View* GetContentsContainerForTest() { return contents_container_; } 478 views::View* GetContentsContainerForTest() { return contents_container_; }
480 views::WebView* GetContentsWebViewForTest() { return contents_web_view_; } 479 views::WebView* GetContentsWebViewForTest() { return contents_web_view_; }
481 views::WebView* GetDevToolsWebViewForTest() { return devtools_web_view_; } 480 views::WebView* GetDevToolsWebViewForTest() { return devtools_web_view_; }
482 481
482 // Called by BrowserFrame during theme changes.
483 void NativeThemeUpdated(const ui::NativeTheme* theme);
484
483 private: 485 private:
484 // Do not friend BrowserViewLayout. Use the BrowserViewLayoutDelegate 486 // Do not friend BrowserViewLayout. Use the BrowserViewLayoutDelegate
485 // interface to keep these two classes decoupled and testable. 487 // interface to keep these two classes decoupled and testable.
486 friend class BrowserViewLayoutDelegateImpl; 488 friend class BrowserViewLayoutDelegateImpl;
487 FRIEND_TEST_ALL_PREFIXES(BrowserViewTest, BrowserView); 489 FRIEND_TEST_ALL_PREFIXES(BrowserViewTest, BrowserView);
488 490
489 // Appends to |toolbars| a pointer to each AccessiblePaneView that 491 // Appends to |toolbars| a pointer to each AccessiblePaneView that
490 // can be traversed using F6, in the order they should be traversed. 492 // can be traversed using F6, in the order they should be traversed.
491 void GetAccessiblePanes(std::vector<views::AccessiblePaneView*>* panes); 493 void GetAccessiblePanes(std::vector<views::AccessiblePaneView*>* panes);
492 494
(...skipping 219 matching lines...) Expand 10 before | Expand all | Expand 10 after
712 // The class that registers for keyboard shortcuts for extension commands. 714 // The class that registers for keyboard shortcuts for extension commands.
713 std::unique_ptr<ExtensionKeybindingRegistryViews> 715 std::unique_ptr<ExtensionKeybindingRegistryViews>
714 extension_keybinding_registry_; 716 extension_keybinding_registry_;
715 717
716 mutable base::WeakPtrFactory<BrowserView> activate_modal_dialog_factory_; 718 mutable base::WeakPtrFactory<BrowserView> activate_modal_dialog_factory_;
717 719
718 DISALLOW_COPY_AND_ASSIGN(BrowserView); 720 DISALLOW_COPY_AND_ASSIGN(BrowserView);
719 }; 721 };
720 722
721 #endif // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_ 723 #endif // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/frame/browser_frame.cc ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698