diff options
Diffstat (limited to 'chrome/browser')
4 files changed, 1 insertions, 160 deletions
diff --git a/chrome/browser/resources/net_internals/browser_bridge.js b/chrome/browser/resources/net_internals/browser_bridge.js index 5184f3a..7353b45 100644 --- a/chrome/browser/resources/net_internals/browser_bridge.js +++ b/chrome/browser/resources/net_internals/browser_bridge.js @@ -26,7 +26,6 @@ var BrowserBridge = (function() { this.httpThrottlingObservers_ = []; this.constantsObservers_ = []; this.crosONCFileParseObservers_ = []; - this.storeDebugLogsObservers_ = []; this.pollableDataHelpers_ = {}; this.pollableDataHelpers_.proxySettings = @@ -234,10 +233,6 @@ var BrowserBridge = (function() { this.send('importONCFile', [fileContent, passcode]); }, - storeDebugLogs: function() { - this.send('storeDebugLogs'); - }, - sendGetHttpPipeliningStatus: function() { this.send('getHttpPipeliningStatus'); }, @@ -330,11 +325,6 @@ var BrowserBridge = (function() { this.crosONCFileParseObservers_[i].onONCFileParse(error); }, - receivedStoreDebugLogs: function(status) { - for (var i = 0; i < this.storeDebugLogsObservers_.length; i++) - this.storeDebugLogsObservers_[i].onStoreDebugLogs(status); - }, - receivedHttpCacheInfo: function(info) { this.pollableDataHelpers_.httpCacheInfo.update(info); }, @@ -531,16 +521,6 @@ var BrowserBridge = (function() { }, /** - * Adds a listener for storing log file status. The observer will be called - * back with: - * - * observer.onStoreDebugLogs(status); - */ - addStoreDebugLogsObserver: function(observer) { - this.storeDebugLogsObservers_.push(observer); - }, - - /** * Adds a listener for HTTP throttling-related events. |observer| will be * called back when HTTP throttling is enabled/disabled, through: * diff --git a/chrome/browser/resources/net_internals/chromeos_view.html b/chrome/browser/resources/net_internals/chromeos_view.html index 098b04f..6b552ae 100644 --- a/chrome/browser/resources/net_internals/chromeos_view.html +++ b/chrome/browser/resources/net_internals/chromeos_view.html @@ -11,13 +11,4 @@ </div> <div id="chromeos-view-parse-status" hidden> </div> - <div id="chromeos-view-store-debug-logs-div"> - <h3>Store Logs</h3> - <input type="button" - id="chromeos-view-store-debug-logs" - value="Store Debug Logs"> - <label for="chromeos-view-store-debug-logs" - id="chromeos-view-store-debug-logs-status"> - </label> - </div> </div> diff --git a/chrome/browser/resources/net_internals/chromeos_view.js b/chrome/browser/resources/net_internals/chromeos_view.js index 61955ab..fea3e36 100644 --- a/chrome/browser/resources/net_internals/chromeos_view.js +++ b/chrome/browser/resources/net_internals/chromeos_view.js @@ -125,17 +125,7 @@ var CrosView = (function() { } /** - * Set storing debug logs status. - * - * @private - */ - function setStoreDebugLogsStatus_(status) { - $(CrosView.STORE_DEBUG_LOGS_STATUS_ID).innerText = status; - } - - /** - * Add event listeners for the file selection, passcode input - * fields and for the button for debug logs storing. + * Add event listeners for the file selection and passcode input fields. * * @private */ @@ -147,11 +137,6 @@ var CrosView = (function() { $(CrosView.PASSCODE_INPUT_ID).addEventListener('change', function(event) { setPasscode_(this.value); }, false); - - $(CrosView.STORE_DEBUG_LOGS_ID).addEventListener('click', function(event) { - $(CrosView.STORE_DEBUG_LOGS_STATUS_ID).innerText = ''; - g_browser.storeDebugLogs(); - }, false); } /** @@ -176,7 +161,6 @@ var CrosView = (function() { DivView.call(this, CrosView.MAIN_BOX_ID); g_browser.addCrosONCFileParseObserver(this); - g_browser.addStoreDebugLogsObserver(this); addEventListeners_(); } @@ -189,8 +173,6 @@ var CrosView = (function() { CrosView.PASSCODE_ID = 'chromeos-view-password-div'; CrosView.PASSCODE_INPUT_ID = 'chromeos-view-onc-password'; CrosView.PARSE_STATUS_ID = 'chromeos-view-parse-status'; - CrosView.STORE_DEBUG_LOGS_ID = 'chromeos-view-store-debug-logs'; - CrosView.STORE_DEBUG_LOGS_STATUS_ID = 'chromeos-view-store-debug-logs-status'; cr.addSingletonGetter(CrosView); @@ -199,7 +181,6 @@ var CrosView = (function() { __proto__: DivView.prototype, onONCFileParse: setParseStatus_, - onStoreDebugLogs: setStoreDebugLogsStatus_, }; return CrosView; diff --git a/chrome/browser/ui/webui/net_internals/net_internals_ui.cc b/chrome/browser/ui/webui/net_internals/net_internals_ui.cc index dc77e63..4d77bf8 100644 --- a/chrome/browser/ui/webui/net_internals/net_internals_ui.cc +++ b/chrome/browser/ui/webui/net_internals/net_internals_ui.cc @@ -14,14 +14,10 @@ #include "base/bind.h" #include "base/bind_helpers.h" #include "base/command_line.h" -#include "base/file_path.h" -#include "base/file_util.h" #include "base/memory/singleton.h" -#include "base/memory/weak_ptr.h" #include "base/message_loop.h" #include "base/message_loop_helpers.h" #include "base/path_service.h" -#include "base/platform_file.h" #include "base/string_number_conversions.h" #include "base/string_piece.h" #include "base/string_split.h" @@ -43,7 +39,6 @@ #include "chrome/common/chrome_notification_types.h" #include "chrome/common/chrome_paths.h" #include "chrome/common/chrome_version_info.h" -#include "chrome/common/logging_chrome.h" #include "chrome/common/pref_names.h" #include "chrome/common/url_constants.h" #include "content/public/browser/browser_thread.h" @@ -77,15 +72,11 @@ #include "chrome/browser/chromeos/cros/cros_library.h" #include "chrome/browser/chromeos/cros/network_library.h" #include "chrome/browser/chromeos/system/syslogs_provider.h" -#include "chromeos/dbus/dbus_thread_manager.h" -#include "chromeos/dbus/debug_daemon_client.h" #endif #ifdef OS_WIN #include "chrome/browser/net/service_providers_win.h" #endif -using base::PlatformFile; -using base::PlatformFileError; using content::BrowserThread; using content::WebContents; using content::WebUIMessageHandler; @@ -157,84 +148,6 @@ ChromeWebUIDataSource* CreateNetInternalsHTMLSource() { return source; } -#ifdef OS_CHROMEOS -// Following functions are used for getting debug logs. Logs are -// fetched from /var/log/* and put on the fileshelf. - -// Called once StoreDebugLogs is complete. Takes two parameters: -// - log_path: where the log file was saved in the case of success; -// - succeeded: was the log file saved successfully. -typedef base::Callback<void(const FilePath& log_path, - bool succeded)> StoreDebugLogsCallback; - -// Called once creation of the debug log file is completed. If -// creation failed, deletes the file by |log_path|. So, this function -// should be called on the FILE thread. After all, calls |callback| on -// the UI thread. -void CreateDebugLogFileCompleted(const StoreDebugLogsCallback& callback, - const FilePath& log_path, - bool succeeded) { - DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE)); - - if (!succeeded) - file_util::Delete(log_path, false); - - BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, - base::Bind(callback, log_path, succeeded)); -} - -// Retrieves debug logs from DebugDaemon and puts them on the -// fileshelf directory into .tgz archive. So, this function should be -// called on the FILE thread. Calls CreateDebugLogFileCompleted on the -// FILE thread when creation of archive is completed. -void CreateDebugLogFile(const StoreDebugLogsCallback& callback) { - DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE)); - - const FilePath::CharType kLogFileName[] = FILE_PATH_LITERAL("debug-log.tgz"); - - FilePath fileshelf_dir; - if (!PathService::Get(chrome::DIR_DEFAULT_DOWNLOADS, &fileshelf_dir)) { - LOG(ERROR) << "Can't get fileshelf dir"; - CreateDebugLogFileCompleted(callback, FilePath(), false); - return; - } - - FilePath log_path = fileshelf_dir.Append(kLogFileName); - log_path = logging::GenerateTimestampedName(log_path, base::Time::Now()); - - int flags = base::PLATFORM_FILE_CREATE_ALWAYS | - base::PLATFORM_FILE_WRITE; - bool created; - PlatformFileError error; - PlatformFile log_file = base::CreatePlatformFile( - log_path, flags, &created, &error); - - if (!created) { - LOG(ERROR) << - "Can't create log file: " << log_path.AsUTF8Unsafe() << ", " << - "error: " << error; - CreateDebugLogFileCompleted(callback, log_path, false); - return; - } - chromeos::DBusThreadManager::Get()->GetDebugDaemonClient()-> - GetDebugLogs(log_file, - base::Bind(&CreateDebugLogFileCompleted, - callback, log_path)); -} - -// Delegates the job of saving debug logs on the fileshelf to -// CreateDebugLogsFile on the FILE thread. Calls |callback| on the UI -// thread when saving is completed. This function should be called on -// the UI thread. -void StoreDebugLogs(const StoreDebugLogsCallback& callback) { - DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); - - BrowserThread::PostTask( - BrowserThread::FILE, FROM_HERE, - base::Bind(&CreateDebugLogFile, callback)); -} -#endif // OS_CHROMEOS - // This class receives javascript messages from the renderer. // Note that the WebUI infrastructure runs on the UI thread, therefore all of // this class's methods are expected to run on the UI thread. @@ -274,8 +187,6 @@ class NetInternalsMessageHandler void OnRefreshSystemLogs(const ListValue* list); void OnGetSystemLog(const ListValue* list); void OnImportONCFile(const ListValue* list); - void OnStoreDebugLogs(const ListValue* list); - void OnStoreDebugLogsCompleted(const FilePath& log_path, bool succeeded); #endif private: @@ -645,10 +556,6 @@ void NetInternalsMessageHandler::RegisterMessages() { "importONCFile", base::Bind(&NetInternalsMessageHandler::OnImportONCFile, base::Unretained(this))); - web_ui()->RegisterMessageCallback( - "storeDebugLogs", - base::Bind(&NetInternalsMessageHandler::OnStoreDebugLogs, - base::Unretained(this))); #endif } @@ -1361,24 +1268,6 @@ void NetInternalsMessageHandler::OnImportONCFile(const ListValue* list) { SendJavascriptCommand("receivedONCFileParse", Value::CreateStringValue(error)); } - -void NetInternalsMessageHandler::OnStoreDebugLogs(const ListValue* list) { - StoreDebugLogs( - base::Bind(&NetInternalsMessageHandler::OnStoreDebugLogsCompleted, - AsWeakPtr())); -} - -void NetInternalsMessageHandler::OnStoreDebugLogsCompleted( - const FilePath& log_path, bool succeeded) { - std::string status; - if (succeeded) - status = "Created log file: " + log_path.BaseName().AsUTF8Unsafe(); - else - status = "Failed to create log file"; - SendJavascriptCommand("receivedStoreDebugLogs", - Value::CreateStringValue(status)); -} - #endif void NetInternalsMessageHandler::IOThreadImpl::OnGetHttpPipeliningStatus( |