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

Unified Diff: chrome/browser/ui/webui/options/advanced_options_utils_win.cc

Issue 1789543003: MD Settings: move AdvancedOptionsUtilities to a settings-accessible place (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/options/advanced_options_utils_win.cc
diff --git a/chrome/browser/ui/webui/options/advanced_options_utils_win.cc b/chrome/browser/ui/webui/options/advanced_options_utils_win.cc
deleted file mode 100644
index 7c5c16c1295b64e835cb1aa251aaf1d4eacec774..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/webui/options/advanced_options_utils_win.cc
+++ /dev/null
@@ -1,118 +0,0 @@
-// 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.
-
-#include "chrome/browser/ui/webui/options/advanced_options_utils.h"
-
-#include <windows.h>
-#include <cryptuiapi.h>
-#pragma comment(lib, "cryptui.lib")
-#include <shellapi.h>
-
-#include "base/bind.h"
-#include "base/bind_helpers.h"
-#include "base/location.h"
-#include "base/macros.h"
-#include "base/message_loop/message_loop.h"
-#include "base/path_service.h"
-#include "base/threading/thread.h"
-#include "chrome/browser/browser_process.h"
-#include "content/public/browser/browser_thread.h"
-#include "content/public/browser/web_contents.h"
-#include "ui/shell_dialogs/base_shell_dialog_win.h"
-#include "ui/views/win/hwnd_util.h"
-
-using content::BrowserThread;
-using content::WebContents;
-
-namespace options {
-
-namespace {
-
-// Shows a Windows certificate management dialog on the dialog thread.
-class ManageCertificatesDialog : public ui::BaseShellDialogImpl {
- public:
- ManageCertificatesDialog() {}
-
- // Shows the dialog and calls |callback| when the dialog closes. The caller
- // must ensure the ManageCertificatesDialog remains valid until then.
- void Show(HWND parent, const base::Closure& callback) {
- if (IsRunningDialogForOwner(parent)) {
- base::MessageLoop::current()->PostTask(FROM_HERE, callback);
- return;
- }
-
- RunState run_state = BeginRun(parent);
- run_state.dialog_thread->task_runner()->PostTaskAndReply(
- FROM_HERE, base::Bind(&ManageCertificatesDialog::ShowOnDialogThread,
- base::Unretained(this), run_state),
- base::Bind(&ManageCertificatesDialog::OnDialogClosed,
- base::Unretained(this), run_state, callback));
- }
-
- private:
- void ShowOnDialogThread(const RunState& run_state) {
- CRYPTUI_CERT_MGR_STRUCT cert_mgr = {0};
- cert_mgr.dwSize = sizeof(CRYPTUI_CERT_MGR_STRUCT);
- cert_mgr.hwndParent = run_state.owner;
- ::CryptUIDlgCertMgr(&cert_mgr);
- }
-
- void OnDialogClosed(const RunState& run_state,
- const base::Closure& callback) {
- EndRun(run_state);
- // May delete |this|.
- callback.Run();
- }
-
- DISALLOW_COPY_AND_ASSIGN(ManageCertificatesDialog);
-};
-
-} // namespace
-
-// Callback that opens the Internet Options control panel dialog with the
-// Connections tab selected.
-void OpenConnectionDialogCallback() {
- // Using rundll32 seems better than LaunchConnectionDialog which causes a
- // new dialog to be made for each call. rundll32 uses the same global
- // dialog and it seems to share with the shortcut in control panel.
- base::FilePath rundll32;
- PathService::Get(base::DIR_SYSTEM, &rundll32);
- rundll32 = rundll32.AppendASCII("rundll32.exe");
-
- base::FilePath shell32dll;
- PathService::Get(base::DIR_SYSTEM, &shell32dll);
- shell32dll = shell32dll.AppendASCII("shell32.dll");
-
- base::FilePath inetcpl;
- PathService::Get(base::DIR_SYSTEM, &inetcpl);
- inetcpl = inetcpl.AppendASCII("inetcpl.cpl,,4");
-
- std::wstring args(shell32dll.value());
- args.append(L",Control_RunDLL ");
- args.append(inetcpl.value());
-
- ShellExecute(NULL, L"open", rundll32.value().c_str(), args.c_str(), NULL,
- SW_SHOWNORMAL);
-}
-
-void AdvancedOptionsUtilities::ShowNetworkProxySettings(
- WebContents* web_contents) {
- DCHECK(BrowserThread::IsMessageLoopValid(BrowserThread::FILE));
- BrowserThread::PostTask(BrowserThread::FILE,
- FROM_HERE,
- base::Bind(&OpenConnectionDialogCallback));
-}
-
-void AdvancedOptionsUtilities::ShowManageSSLCertificates(
- WebContents* web_contents) {
- HWND parent =
- views::HWNDForNativeWindow(web_contents->GetTopLevelNativeWindow());
-
- ManageCertificatesDialog* dialog = new ManageCertificatesDialog;
- dialog->Show(
- parent,
- base::Bind(&base::DeletePointer<ManageCertificatesDialog>, dialog));
-}
-
-} // namespace options

Powered by Google App Engine
This is Rietveld 408576698