summaryrefslogtreecommitdiffstats
path: root/chrome/browser/dom_ui
diff options
context:
space:
mode:
authorpinkerton@google.com <pinkerton@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2010-07-19 14:43:07 +0000
committerpinkerton@google.com <pinkerton@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2010-07-19 14:43:07 +0000
commit38889cb24f897bf651601ab28810bb1c283253a5 (patch)
tree1693b3c7a00aafe1d45f3869d0473495ea9310b8 /chrome/browser/dom_ui
parent399a8da882ca5366d865b04bfc211081f1d924a1 (diff)
downloadchromium_src-38889cb24f897bf651601ab28810bb1c283253a5.zip
chromium_src-38889cb24f897bf651601ab28810bb1c283253a5.tar.gz
chromium_src-38889cb24f897bf651601ab28810bb1c283253a5.tar.bz2
Revert 52902 - Add getUserInfo, getRegistrationUrl message callback for register screen.
Restrict chrome://register to OOBE wizard only. Actually delete about:register. BUG= http://crosbug.com/4813 TEST=none Review URL: http://codereview.chromium.org/3044001 TBR=nkostylev@google.com Review URL: http://codereview.chromium.org/3026007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52905 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/dom_ui')
-rw-r--r--chrome/browser/dom_ui/dom_ui_factory.cc2
-rw-r--r--chrome/browser/dom_ui/register_page_ui.cc107
2 files changed, 3 insertions, 106 deletions
diff --git a/chrome/browser/dom_ui/dom_ui_factory.cc b/chrome/browser/dom_ui/dom_ui_factory.cc
index f51fe22..3dd1a50 100644
--- a/chrome/browser/dom_ui/dom_ui_factory.cc
+++ b/chrome/browser/dom_ui/dom_ui_factory.cc
@@ -17,6 +17,7 @@
#include "chrome/browser/dom_ui/options_ui.h"
#include "chrome/browser/dom_ui/remoting_ui.h"
#include "chrome/browser/dom_ui/plugins_ui.h"
+#include "chrome/browser/dom_ui/register_page_ui.h"
#include "chrome/browser/dom_ui/slideshow_ui.h"
#include "chrome/browser/extensions/extension_dom_ui.h"
#include "chrome/browser/extensions/extensions_service.h"
@@ -32,7 +33,6 @@
#if defined(OS_CHROMEOS)
#include "chrome/browser/dom_ui/filebrowse_ui.h"
#include "chrome/browser/dom_ui/mediaplayer_ui.h"
-#include "chrome/browser/dom_ui/register_page_ui.h"
#endif
const DOMUITypeID DOMUIFactory::kNoDOMUI = NULL;
diff --git a/chrome/browser/dom_ui/register_page_ui.cc b/chrome/browser/dom_ui/register_page_ui.cc
index 9c9bdab..7811b21 100644
--- a/chrome/browser/dom_ui/register_page_ui.cc
+++ b/chrome/browser/dom_ui/register_page_ui.cc
@@ -1,35 +1,20 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2009 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/dom_ui/register_page_ui.h"
-#include <string>
-
#include "app/l10n_util.h"
#include "app/resource_bundle.h"
#include "base/callback.h"
-#include "base/logging.h"
#include "base/string_piece.h"
-#include "base/utf_string_conversions.h"
-#include "base/values.h"
#include "base/weak_ptr.h"
-#include "chrome/browser/browser_process.h"
#include "chrome/browser/chrome_thread.h"
-#include "chrome/browser/chromeos/cros/cros_library.h"
-#include "chrome/browser/chromeos/customization_document.h"
-#include "chrome/browser/chromeos/login/wizard_controller.h"
-#include "chrome/browser/chromeos/version_loader.h"
#include "chrome/browser/dom_ui/chrome_url_data_manager.h"
#include "chrome/browser/tab_contents/tab_contents.h"
#include "chrome/common/url_constants.h"
#include "grit/browser_resources.h"
-namespace {
-// Constant value for os_name sent in setUserInfo.
-const wchar_t kOSName[] = L"ChromeOS";
-} // namespace
-
class RegisterPageUIHTMLSource : public ChromeURLDataManager::DataSource {
public:
RegisterPageUIHTMLSource();
@@ -64,23 +49,6 @@ class RegisterPageHandler : public DOMMessageHandler,
virtual void RegisterMessages();
private:
- // Handlers for JS DOMUI messages.
- void HandleGetRegistrationUrl(const Value* value);
- void HandleGetUserInfo(const Value* value);
-
- // Callback from chromeos::VersionLoader giving the version.
- void OnVersion(chromeos::VersionLoader::Handle handle, std::string version);
-
- // Sends message to host registration page with system/user info data.
- void SendUserInfo();
-
- // Handles asynchronously loading the version.
- chromeos::VersionLoader version_loader_;
- // Used to request the version.
- CancelableRequestConsumer version_consumer_;
-
- std::string version_;
-
DISALLOW_COPY_AND_ASSIGN(RegisterPageHandler);
};
@@ -97,15 +65,6 @@ RegisterPageUIHTMLSource::RegisterPageUIHTMLSource()
void RegisterPageUIHTMLSource::StartDataRequest(const std::string& path,
bool is_off_the_record,
int request_id) {
- // Make sure that chrome://register is available only during
- // OOBE wizard lifetime.
- if (!WizardController::default_controller() &&
- !WizardController::default_controller()->is_oobe()) {
- scoped_refptr<RefCountedBytes> empty_bytes(new RefCountedBytes);
- SendResponse(request_id, empty_bytes);
- return;
- }
-
static const base::StringPiece register_html(
ResourceBundle::GetSharedInstance().GetRawDataResource(
IDR_HOST_REGISTRATION_PAGE_HTML));
@@ -141,69 +100,7 @@ void RegisterPageHandler::Init() {
}
void RegisterPageHandler::RegisterMessages() {
- dom_ui_->RegisterMessageCallback("getRegistrationUrl",
- NewCallback(this, &RegisterPageHandler::HandleGetRegistrationUrl));
- dom_ui_->RegisterMessageCallback("getUserInfo",
- NewCallback(this, &RegisterPageHandler::HandleGetUserInfo));
-}
-
-void RegisterPageHandler::HandleGetRegistrationUrl(const Value* value) {
- if (WizardController::default_controller() &&
- WizardController::default_controller()->GetCustomization()) {
- StringValue url_value(WizardController::default_controller()->
- GetCustomization()->registration_url());
- dom_ui_->CallJavascriptFunction(L"setRegistrationUrl", url_value);
- } else {
- LOG(ERROR) << "Startup manifest not defined.";
- }
-}
-
-void RegisterPageHandler::HandleGetUserInfo(const Value* value) {
- if (chromeos::CrosLibrary::Get()->EnsureLoaded()) {
- version_loader_.GetVersion(
- &version_consumer_, NewCallback(this,
- &RegisterPageHandler::OnVersion));
- } else {
- LOG(ERROR) << "Error loading cros library.";
- }
-}
-
-void RegisterPageHandler::OnVersion(chromeos::VersionLoader::Handle handle,
- std::string version) {
- version_ = version;
- SendUserInfo();
-}
-
-void RegisterPageHandler::SendUserInfo() {
- DictionaryValue value;
- // TODO(nkostylev): Extract all available system/user info.
- // http://crosbug.com/4813
-
- std::string system_sku;
- if (WizardController::default_controller() &&
- WizardController::default_controller()->GetCustomization()) {
- system_sku = WizardController::default_controller()->
- GetCustomization()->product_sku();
- } else {
- LOG(ERROR) << "Startup manifest not defined.";
- }
-
- // Required info.
- value.SetString(L"system_hwqual", L"hardware qual identifier");
- value.SetString(L"system_sku", UTF8ToWide(system_sku));
- value.SetString(L"system_serial", L"serial number");
- value.SetString(L"os_language",
- UTF8ToWide(g_browser_process->GetApplicationLocale()));
- value.SetString(L"os_name", kOSName);
- value.SetString(L"os_version", UTF8ToWide(version_));
- value.SetString(L"os_connection", L"connection type");
- value.SetString(L"user_email", L"");
-
- // Optional info.
- value.SetString(L"user_first_name", L"");
- value.SetString(L"user_last_name", L"");
-
- dom_ui_->CallJavascriptFunction(L"setUserInfo", value);
+ // TODO(nkostylev): Register callback messages.
}
////////////////////////////////////////////////////////////////////////////////