[Harmony] Harmonize Bluetooth dialogs.

chrome/browser/chromeos/bluetooth/bluetooth_pairing_view.h

+// Copyright 2017 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_CHROMEOS_BLUETOOTH_BLUETOOTH_PAIRING_VIEW_H_
+#define CHROME_BROWSER_CHROMEOS_BLUETOOTH_BLUETOOTH_PAIRING_VIEW_H_
+
+#include "ui/views/controls/webview/web_dialog_view.h"
+
+namespace content {
+class BrowserContext;
+}
+
+namespace device {
+class BluetoothDevice;
+}
+
+namespace ui {
+class WebDialogDelegate;
+}
+
+namespace chromeos {
+
+// A customized dialog view for the Bluetooth pairing dialog.
+class BluetoothPairingView : public views::WebDialogView {
+ public:
+  BluetoothPairingView(content::BrowserContext* context,
+                       ui::WebDialogDelegate* delegate,
+                       WebContentsHandler* handler);
+  ~BluetoothPairingView() override;
+
+  // Shows the customarized Bluetooth pairing dialog.

[James Cook, 2017/02/09 15:18:02]

nit: "customized"

Also, document parameters, especially container_id.

[xdai1, 2017/02/09 22:58:28]

Done.

+  static void ShowDialog(int container_id,

[James Cook, 2017/02/09 15:18:02]

It's kind of weird that there's a "ShowDialog" method on a View class. I think
the Show method should be on the Dialog class.

[xdai1, 2017/02/09 22:58:28]

In this case we don't need this view class at all. Removed it...

+                         content::BrowserContext* context,
+                         WebContentsHandler* handler,
+                         const device::BluetoothDevice* device);
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(BluetoothPairingView);

[James Cook, 2017/02/09 15:18:02]

#include macros.h

[xdai1, 2017/02/09 22:58:28]

Done.

+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_BLUETOOTH_BLUETOOTH_PAIRING_VIEW_H_