Implement bluetooth chooser for Chrome Apps on non-Mac desktops

chrome/browser/extensions/chrome_extension_chooser_dialog.h

 // Copyright 2016 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_EXTENSIONS_CHROME_EXTENSION_CHOOSER_DIALOG_H_
 #define CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_CHOOSER_DIALOG_H_
 
 #include "base/macros.h"
-#include "build/build_config.h"
 
 class ChooserController;
 
 namespace content {
 class WebContents;
 }
 
 class ChromeExtensionChooserDialog {
  public:
   explicit ChromeExtensionChooserDialog(content::WebContents* web_contents)
       : web_contents_(web_contents) {}
   ~ChromeExtensionChooserDialog() {}
 
   content::WebContents* web_contents() const { return web_contents_; }
 
-// TODO(juncai): remove this preprocessor directive once the non-Mac

[msw, 2016/06/03 19:46:24]

I'm confused, why remove this if the CL is for "non-Mac desktops", and the
chooser_dialog_view.* files are only added to
'chrome_browser_ui_views_extensions_non_mac_sources'? Does this implement the
dialog for Mac or not?

[juncai, 2016/06/07 23:19:39]

Actually, this preprocessor directive was removed at the following CL:
https://codereview.chromium.org/2036033003/

-// implementation is done.
-#if defined(OS_MACOSX)
   void ShowDialog(ChooserController* chooser_controller) const;
-#endif  // defined(OS_MACOSX)
 
  private:
   content::WebContents* web_contents_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeExtensionChooserDialog);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSION_CHOOSER_DIALOG_H_