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

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

Issue 2753933003: 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 422 matching lines...) Expand 10 before | Expand all | Expand 10 after
433 bool DrawInfoBarArrows(int* x) const override; 433 bool DrawInfoBarArrows(int* x) const override;
434 434
435 // Overridden from views::View: 435 // Overridden from views::View:
436 const char* GetClassName() const override; 436 const char* GetClassName() const override;
437 void Layout() override; 437 void Layout() override;
438 void ViewHierarchyChanged( 438 void ViewHierarchyChanged(
439 const ViewHierarchyChangedDetails& details) override; 439 const ViewHierarchyChangedDetails& details) override;
440 void ChildPreferredSizeChanged(View* child) override; 440 void ChildPreferredSizeChanged(View* child) override;
441 void GetAccessibleNodeData(ui::AXNodeData* node_data) override; 441 void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
442 void OnThemeChanged() override; 442 void OnThemeChanged() override;
443 void OnNativeThemeChanged(const ui::NativeTheme* theme) override;
444 443
445 // Overridden from ui::AcceleratorTarget: 444 // Overridden from ui::AcceleratorTarget:
446 bool AcceleratorPressed(const ui::Accelerator& accelerator) override; 445 bool AcceleratorPressed(const ui::Accelerator& accelerator) override;
447 446
448 // OmniboxPopupModelObserver overrides 447 // OmniboxPopupModelObserver overrides
449 void OnOmniboxPopupShownOrHidden() override; 448 void OnOmniboxPopupShownOrHidden() override;
450 449
451 // ExclusiveAccessContext overrides 450 // ExclusiveAccessContext overrides
452 Profile* GetProfile() override; 451 Profile* GetProfile() override;
453 content::WebContents* GetActiveWebContents() override; 452 content::WebContents* GetActiveWebContents() override;
(...skipping 17 matching lines...) Expand all
471 // Creates an accessible tab label for screen readers that includes the tab 470 // Creates an accessible tab label for screen readers that includes the tab
472 // status for the given tab index. This takes the form of 471 // status for the given tab index. This takes the form of
473 // "Page title - Tab state". 472 // "Page title - Tab state".
474 base::string16 GetAccessibleTabLabel(bool include_app_name, int index) const; 473 base::string16 GetAccessibleTabLabel(bool include_app_name, int index) const;
475 474
476 // Testing interface: 475 // Testing interface:
477 views::View* GetContentsContainerForTest() { return contents_container_; } 476 views::View* GetContentsContainerForTest() { return contents_container_; }
478 views::WebView* GetContentsWebViewForTest() { return contents_web_view_; } 477 views::WebView* GetContentsWebViewForTest() { return contents_web_view_; }
479 views::WebView* GetDevToolsWebViewForTest() { return devtools_web_view_; } 478 views::WebView* GetDevToolsWebViewForTest() { return devtools_web_view_; }
480 479
480 // Called by BrowserFrame during theme changes.
481 void NativeThemeUpdated(const ui::NativeTheme* theme);
482
481 private: 483 private:
482 // Do not friend BrowserViewLayout. Use the BrowserViewLayoutDelegate 484 // Do not friend BrowserViewLayout. Use the BrowserViewLayoutDelegate
483 // interface to keep these two classes decoupled and testable. 485 // interface to keep these two classes decoupled and testable.
484 friend class BrowserViewLayoutDelegateImpl; 486 friend class BrowserViewLayoutDelegateImpl;
485 FRIEND_TEST_ALL_PREFIXES(BrowserViewTest, BrowserView); 487 FRIEND_TEST_ALL_PREFIXES(BrowserViewTest, BrowserView);
486 488
487 // Appends to |toolbars| a pointer to each AccessiblePaneView that 489 // Appends to |toolbars| a pointer to each AccessiblePaneView that
488 // can be traversed using F6, in the order they should be traversed. 490 // can be traversed using F6, in the order they should be traversed.
489 void GetAccessiblePanes(std::vector<views::AccessiblePaneView*>* panes); 491 void GetAccessiblePanes(std::vector<views::AccessiblePaneView*>* panes);
490 492
(...skipping 219 matching lines...) Expand 10 before | Expand all | Expand 10 after
710 // The class that registers for keyboard shortcuts for extension commands. 712 // The class that registers for keyboard shortcuts for extension commands.
711 std::unique_ptr<ExtensionKeybindingRegistryViews> 713 std::unique_ptr<ExtensionKeybindingRegistryViews>
712 extension_keybinding_registry_; 714 extension_keybinding_registry_;
713 715
714 mutable base::WeakPtrFactory<BrowserView> activate_modal_dialog_factory_; 716 mutable base::WeakPtrFactory<BrowserView> activate_modal_dialog_factory_;
715 717
716 DISALLOW_COPY_AND_ASSIGN(BrowserView); 718 DISALLOW_COPY_AND_ASSIGN(BrowserView);
717 }; 719 };
718 720
719 #endif // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_ 721 #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