Linux Aura: Use Aura theme on all windows when 'Use Classic theme' is used

chrome/browser/ui/views/frame/browser_view.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
 
 #include <map>
 #include <memory>
 #include <string>
@@ skipping 83 matching lines @@
                     public InfoBarContainerDelegate,
                     public LoadCompleteListener::Delegate,
                     public OmniboxPopupModelObserver,
                     public ExclusiveAccessContext,
                     public ExclusiveAccessBubbleViewsContext,
                     public extensions::ExtensionKeybindingRegistry::Delegate {
  public:
   // The browser view's class name.
   static const char kViewClassName[];
 
+  // Key used to bind an original (non-incognito) profile to the
+  // widget with which it is associated.  Only used on Linux for
+  // determining if windows should use the system theme.
+  static const char kThemeProfileKey[];

[sky, 2017/02/16 22:10:32]

As this is used in places other than browser_view, move to a separate place.
Maybe c/b/ui/views/theme_profile_key. I think it would be good if you move the
getter and the setter there and put the key in the .cc. I'm thinking of:

void SetThemeProfileForWindow(aura::Window* window, Profile* profile);
Profile* GetThemeProfileForWindow(aura::Window*);

I would also use UI_CLASS_PROPERTY rather than
SetNativeWindowProperty/GetNativeWindowProperty. Doing that means you won't need
the reinterpret_cast.

[Tom (Use chromium acct), 2017/02/17 00:35:31]

Done.

+
   BrowserView();
   ~BrowserView() override;
 
   // Takes ownership of |browser|.
   void Init(Browser* browser);
 
   void set_frame(BrowserFrame* frame) { frame_ = frame; }
   BrowserFrame* frame() const { return frame_; }
 
   // Returns a pointer to the BrowserView* interface implementation (an
@@ skipping 596 matching lines @@
   // The class that registers for keyboard shortcuts for extension commands.
   std::unique_ptr<ExtensionKeybindingRegistryViews>
       extension_keybinding_registry_;
 
   mutable base::WeakPtrFactory<BrowserView> activate_modal_dialog_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_