diff options
author | ilevy@chromium.org <ilevy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-15 01:28:29 +0000 |
---|---|---|
committer | ilevy@chromium.org <ilevy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-15 01:28:29 +0000 |
commit | 11bfe30b4bd2656e07765cae0253eff65121b225 (patch) | |
tree | cbd33d135e57d0dc50ea2d8bd4d6ad81ad4afa1c /remoting | |
parent | 2fcafa0b3526619bbb10f0aebd7e6a22ee9f965b (diff) | |
download | chromium_src-11bfe30b4bd2656e07765cae0253eff65121b225.zip chromium_src-11bfe30b4bd2656e07765cae0253eff65121b225.tar.gz chromium_src-11bfe30b4bd2656e07765cae0253eff65121b225.tar.bz2 |
Revert 167794 - Native remoting host configurer for linux.
This is initial version of the remoting host configurer for linux. It doesn't look pretty yet,
but it works.
This requires a new dependency and a PSA was not sent in advance to chromium-dev. It breaks
out bots because someone needs to manually run install-build-deps on each of them.
Please wait a couple days before landing. yfriedman has sent the required PSA.
Review URL: https://chromiumcodereview.appspot.com/11260004
TBR=sergeyu@chromium.org
Review URL: https://codereview.chromium.org/11362266
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167811 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'remoting')
-rw-r--r-- | remoting/base/resources.cc | 4 | ||||
-rw-r--r-- | remoting/base/resources.h | 6 | ||||
-rw-r--r-- | remoting/host/setup/linux/linux_host_setup_wizard.cc | 308 | ||||
-rw-r--r-- | remoting/remoting.gyp | 24 | ||||
-rw-r--r-- | remoting/resources/string_resources.grd | 4 |
5 files changed, 4 insertions, 342 deletions
diff --git a/remoting/base/resources.cc b/remoting/base/resources.cc index a43e328..32e62c4 100644 --- a/remoting/base/resources.cc +++ b/remoting/base/resources.cc @@ -17,14 +17,14 @@ const char kCommonResourcesFileName[] = "chrome_remote_desktop.pak"; } // namespace // Loads chromoting resources. -bool LoadResources(const std::string& pref_locale) { +bool LoadResources(const std::string& locale) { FilePath path; if (!PathService::Get(base::DIR_MODULE, &path)) return false; PathService::Override(ui::DIR_LOCALES, path.AppendASCII(kLocaleResourcesDirName)); - ui::ResourceBundle::InitSharedInstanceLocaleOnly(pref_locale, NULL); + ui::ResourceBundle::InitSharedInstanceLocaleOnly(locale, NULL); ui::ResourceBundle::GetSharedInstance().AddDataPackFromPath( path.AppendASCII(kCommonResourcesFileName), ui::SCALE_FACTOR_100P); diff --git a/remoting/base/resources.h b/remoting/base/resources.h index b08eb8d..a8fdfd6 100644 --- a/remoting/base/resources.h +++ b/remoting/base/resources.h @@ -9,10 +9,8 @@ namespace remoting { -// Loads chromoting resources. Returns false in case of a failure. |pref_locale| -// is passed to l10n_util::GetApplicationLocale(), so the default system locale -// is used if |pref_locale| is empty. -bool LoadResources(const std::string& pref_locale); +// Loads chromoting resources. Returns false in case of a failure. +bool LoadResources(const std::string& locale); } // namespace remoting diff --git a/remoting/host/setup/linux/linux_host_setup_wizard.cc b/remoting/host/setup/linux/linux_host_setup_wizard.cc deleted file mode 100644 index 98dde3b..0000000 --- a/remoting/host/setup/linux/linux_host_setup_wizard.cc +++ /dev/null @@ -1,308 +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 <gdk/gdk.h> -#include <gtk/gtk.h> -#include <webkit/webkitwebview.h> - -#include "base/at_exit.h" -#include "base/basictypes.h" -#include "base/command_line.h" -#include "base/message_loop.h" -#include "base/run_loop.h" -#include "base/threading/thread.h" -#include "net/base/net_util.h" -#include "remoting/base/common_resources.h" -#include "remoting/base/resources.h" -#include "remoting/base/string_resources.h" -#include "remoting/host/setup/host_starter.h" -#include "remoting/host/setup/oauth_helper.h" -#include "remoting/host/setup/pin_validator.h" -#include "remoting/host/url_request_context.h" -#include "ui/base/gtk/gtk_signal.h" -#include "ui/base/gtk/scoped_gobject.h" -#include "ui/base/l10n/l10n_util.h" -#include "ui/base/resource/resource_bundle.h" -#include "ui/base/ui_base_paths.h" - -namespace remoting { - -class LinuxHostSetupWizard { - public: - enum Pages { - kAuthPage = 0, - kPinPage = 1, - kProgressPage = 2, - kStartedPage = 3, - kErrorPage = 4, - kStoppedPage = 5, - }; - - LinuxHostSetupWizard(); - ~LinuxHostSetupWizard(); - - void Init(); - void Run(); - - private: - CHROMEGTK_CALLBACK_1(LinuxHostSetupWizard, gboolean, OnDelete, GdkEvent*); - CHROMEGTK_CALLBACK_1(LinuxHostSetupWizard, void, OnAuthUriChanged, - GParamSpec*); - CHROMEGTK_CALLBACK_0(LinuxHostSetupWizard, void, OnPinOkClicked); - - GtkWidget* AddPage(int expected_index); - GtkWidget* AddTextFieldWithLabel(GtkContainer* container, - const std::string& message); - void AddButtonsBox(GtkContainer* container, GCallback ok_callback); - - void ShowAuthenticationPrompt(); - void ShowProgressMessage(const std::string& message); - void ShowErrorMessage(const std::string& message); - - void ShowMessageBox(const std::string& message); - - void OnHostStarted(HostStarter::Result result); - - MessageLoopForUI main_message_loop_; - base::RunLoop run_loop_; - - scoped_ptr<base::Thread> io_thread_; - - scoped_refptr<URLRequestContextGetter> url_context_getter_; - - GtkWidget* main_window_; - GtkWidget* notebook_; - - // TODO(sergeyu): Replace auth code with authentication. - GtkWidget* auth_web_view_; - GtkWidget* pin_entry_; - GtkWidget* pin_repeat_entry_; - GtkWidget* progress_message_; - GtkWidget* error_message_; - - std::string auth_code_; - - scoped_ptr<HostStarter> starter_; - - DISALLOW_COPY_AND_ASSIGN(LinuxHostSetupWizard); -}; - -LinuxHostSetupWizard::LinuxHostSetupWizard() { -} - -LinuxHostSetupWizard::~LinuxHostSetupWizard() { - gtk_widget_destroy(main_window_); -} - -void LinuxHostSetupWizard::Init() { - io_thread_.reset(new base::Thread("IOThread")); - io_thread_->StartWithOptions( - base::Thread::Options(MessageLoop::TYPE_IO, 0)); - - url_context_getter_ = new URLRequestContextGetter( - main_message_loop_.message_loop_proxy(), - io_thread_->message_loop_proxy()); - - main_window_ = gtk_window_new(GTK_WINDOW_TOPLEVEL); - g_signal_connect(main_window_, "delete-event", - G_CALLBACK(OnDeleteThunk), this); - gtk_window_set_title( - GTK_WINDOW(main_window_), - l10n_util::GetStringUTF8(IDR_PRODUCT_NAME).c_str()); - gtk_window_set_resizable(GTK_WINDOW(main_window_), FALSE); - - gfx::Image logo = ui::ResourceBundle::GetSharedInstance().GetImageNamed( - IDR_PRODUCT_LOGO_32); - ui::ScopedGObject<GdkPixbuf>::Type pixbuf(logo.ToGdkPixbuf()); - gtk_window_set_icon(GTK_WINDOW(main_window_), pixbuf.get()); - - notebook_ = gtk_notebook_new(); - gtk_container_add(GTK_CONTAINER(main_window_), notebook_); - gtk_notebook_set_show_tabs(GTK_NOTEBOOK(notebook_), FALSE); - gtk_notebook_set_show_border(GTK_NOTEBOOK(notebook_), FALSE); - - GtkWidget* auth_page = AddPage(kAuthPage); - auth_web_view_ = webkit_web_view_new(); - gtk_container_add(GTK_CONTAINER(auth_page), auth_web_view_); - g_signal_connect(auth_web_view_, "notify::uri", - G_CALLBACK(OnAuthUriChangedThunk), this); - - GtkWidget* pin_page = AddPage(kPinPage); - pin_entry_ = AddTextFieldWithLabel( - GTK_CONTAINER(pin_page), - l10n_util::GetStringUTF8(IDR_ASK_PIN_DIALOG_LABEL)); - pin_repeat_entry_ = AddTextFieldWithLabel( - GTK_CONTAINER(pin_page), - l10n_util::GetStringUTF8(IDR_ASK_PIN_DIALOG_CONFIRM_LABEL)); - AddButtonsBox(GTK_CONTAINER(pin_page), G_CALLBACK(OnPinOkClickedThunk)); - - GtkWidget* progress_page = AddPage(kProgressPage); - progress_message_ = gtk_label_new(""); - gtk_container_add(GTK_CONTAINER(progress_page), progress_message_); - - GtkWidget* started_page = AddPage(kStartedPage); - GtkWidget* started_message = gtk_label_new( - l10n_util::GetStringUTF8(IDR_HOST_SETUP_STARTED).c_str()); - gtk_container_add(GTK_CONTAINER(started_page), started_message); - - GtkWidget* error_page = AddPage(kErrorPage); - error_message_ = gtk_label_new(""); - gtk_container_add(GTK_CONTAINER(error_page), error_message_); - - GtkWidget* stopped_page = AddPage(kStoppedPage); - GtkWidget* stopped_message = gtk_label_new( - l10n_util::GetStringUTF8(IDR_HOST_NOT_STARTED).c_str()); - gtk_container_add(GTK_CONTAINER(stopped_page), stopped_message); - - gtk_widget_show_all(main_window_); - - ShowAuthenticationPrompt(); -} - -void LinuxHostSetupWizard::Run() { - gtk_widget_show_all(main_window_); - run_loop_.Run(); -} - -GtkWidget* LinuxHostSetupWizard::AddPage(int expected_index) { - GtkWidget* page = gtk_vbox_new(FALSE, 8); - - int page_index = - gtk_notebook_append_page(GTK_NOTEBOOK(notebook_), page, NULL); - DCHECK_EQ(page_index, expected_index); - - return page; -} - -GtkWidget* LinuxHostSetupWizard::AddTextFieldWithLabel( - GtkContainer* container, - const std::string& message) { - GtkWidget* box = gtk_hbox_new(FALSE, 8); - gtk_container_add(container, box); - - GtkWidget* label = gtk_label_new(message.c_str()); - gtk_container_add(GTK_CONTAINER(box), label); - - GtkWidget* entry = gtk_entry_new(); - gtk_container_add(GTK_CONTAINER(box), entry); - - return entry; -} - -void LinuxHostSetupWizard::AddButtonsBox(GtkContainer* container, - GCallback ok_callback) { - GtkWidget* box = gtk_hbox_new(FALSE, 8); - gtk_container_add(container, box); - - GtkWidget* button = - gtk_button_new_with_label(l10n_util::GetStringUTF8(IDR_OK).c_str()); - gtk_container_add(GTK_CONTAINER(box), button); - g_signal_connect(button, "clicked", ok_callback, this); -} - -void LinuxHostSetupWizard::ShowAuthenticationPrompt() { - std::string url = GetOauthStartUrl(GetDefaultOauthRedirectUrl()); - webkit_web_view_load_uri(WEBKIT_WEB_VIEW(auth_web_view_), url.c_str()); - gtk_notebook_set_current_page(GTK_NOTEBOOK(notebook_), kAuthPage); -} - -void LinuxHostSetupWizard::ShowProgressMessage(const std::string& message) { - gtk_label_set_text(GTK_LABEL(progress_message_), message.c_str()); - gtk_notebook_set_current_page(GTK_NOTEBOOK(notebook_), kProgressPage); -} - -void LinuxHostSetupWizard::ShowErrorMessage(const std::string& message) { - gtk_label_set_text(GTK_LABEL(error_message_), message.c_str()); - gtk_notebook_set_current_page(GTK_NOTEBOOK(notebook_), kErrorPage); -} - -void LinuxHostSetupWizard::ShowMessageBox(const std::string& message) { - GtkWidget* dialog = gtk_message_dialog_new( - GTK_WINDOW(main_window_), static_cast<GtkDialogFlags>( - GTK_DIALOG_MODAL | GTK_DIALOG_DESTROY_WITH_PARENT), - GTK_MESSAGE_ERROR, GTK_BUTTONS_OK, "%s", message.c_str()); - g_signal_connect(dialog, "response", G_CALLBACK(gtk_widget_destroy), NULL); - gtk_widget_show_all(dialog); -} - -gboolean LinuxHostSetupWizard::OnDelete(GtkWidget* window, GdkEvent* event) { - run_loop_.Quit(); - return TRUE; -} - -void LinuxHostSetupWizard::OnAuthUriChanged(GtkWidget* web_view, - GParamSpec* param_spec) { - std::string url(webkit_web_view_get_uri(WEBKIT_WEB_VIEW(auth_web_view_))); - std::string auth_code = GetOauthCodeInUrl(url, GetDefaultOauthRedirectUrl()); - if (!auth_code.empty()) { - auth_code_ = auth_code; - gtk_notebook_set_current_page(GTK_NOTEBOOK(notebook_), kPinPage); - } -} - -void LinuxHostSetupWizard::OnPinOkClicked(GtkWidget* button) { - std::string host_name = net::GetHostName(); - std::string pin = gtk_entry_get_text(GTK_ENTRY(pin_entry_)); - std::string pin_repeat = gtk_entry_get_text(GTK_ENTRY(pin_repeat_entry_)); - - if (!IsPinValid(pin)) { - ShowMessageBox(l10n_util::GetStringUTF8(IDR_INVALID_PIN)); - return; - } - - if (pin != pin_repeat) { - ShowMessageBox(l10n_util::GetStringUTF8(IDR_PINS_NOT_EQUAL)); - return; - } - - starter_ = HostStarter::Create(url_context_getter_.get()); - - starter_->StartHost(host_name, pin, false, auth_code_, - GetDefaultOauthRedirectUrl(), - base::Bind(&LinuxHostSetupWizard::OnHostStarted, - base::Unretained(this))); - ShowProgressMessage(l10n_util::GetStringUTF8(IDR_HOST_SETUP_STARTING)); -} - -void LinuxHostSetupWizard::OnHostStarted(HostStarter::Result result) { - switch (result) { - case HostStarter::START_COMPLETE: - gtk_notebook_set_current_page(GTK_NOTEBOOK(notebook_), kStartedPage); - break; - - case HostStarter::NETWORK_ERROR: - case HostStarter::OAUTH_ERROR: - ShowErrorMessage(l10n_util::GetStringUTF8( - IDR_HOST_SETUP_REGISTRATION_FAILED)); - break; - - case HostStarter::START_ERROR: - ShowErrorMessage(l10n_util::GetStringUTF8(IDR_HOST_SETUP_HOST_FAILED)); - break; - } -} - -} // namespace remoting - -int main(int argc, char** argv) { - CommandLine::Init(argc, argv); - base::AtExitManager at_exit_manager; - ui::RegisterPathProvider(); - - remoting::LoadResources(std::string()); - - InitLogging(NULL, - logging::LOG_ONLY_TO_SYSTEM_DEBUG_LOG, - logging::DONT_LOCK_LOG_FILE, - logging::APPEND_TO_OLD_LOG_FILE, - logging::DISABLE_DCHECK_FOR_NON_OFFICIAL_RELEASE_BUILDS); - - gtk_init(&argc, &argv); - - remoting::LinuxHostSetupWizard setup_ui; - setup_ui.Init(); - setup_ui.Run(); - - return 0; -} diff --git a/remoting/remoting.gyp b/remoting/remoting.gyp index 993382d..4b037e7 100644 --- a/remoting/remoting.gyp +++ b/remoting/remoting.gyp @@ -285,30 +285,6 @@ 'host/setup/start_host.cc', ], }, # end of target 'remoting_start_host' - { - 'target_name': 'remoting_configurer', - 'type': 'executable', - 'variables': { 'enable_wexit_time_destructors': 1, }, - 'cflags': [ - '<!@(pkg-config --cflags webkit-1.0)', - ], - 'link_settings': { - 'ldflags': [ - '<!@(pkg-config --libs-only-L --libs-only-other webkit-1.0)', - ], - 'libraries': [ - '<!@(pkg-config --libs-only-l webkit-1.0)', - ], - }, - 'dependencies': [ - '../base/base.gyp:base', - 'remoting_host_setup_base', - 'remoting_resources' - ], - 'sources': [ - 'host/setup/linux/linux_host_setup_wizard.cc', - ], - }, # end of target 'remoting_configurer' ], # end of 'targets' }], # 'OS=="linux"' diff --git a/remoting/resources/string_resources.grd b/remoting/resources/string_resources.grd index 7a9a3c1..52456fc 100644 --- a/remoting/resources/string_resources.grd +++ b/remoting/resources/string_resources.grd @@ -522,10 +522,6 @@ <message desc="Help link displayed when the user enables the host on this computer. Clicking this link opens a page that explains why this operation is safe. This string appears in a UI with limited horizontal space. Please try to keep translations no more than about 30 characters." name="IDR_WHY_IS_THIS_SAFE"> Why is this safe? </message> - - <message name="IDR_HOST_NOT_STARTED" desc="Message displayed when the current computer is not accepting remote connections, instructing the user how to enable them."> - Remote access to this computer is disabled. - </message> </messages> </release> </grit> |