Commit 3c4d1489 authored by Denis Kuznetsov's avatar Denis Kuznetsov Committed by Commit Bot

OOBE: add extra logging on initialization

TBR=rsorokin@chromium.org

Bug: 1082670
Change-Id: I52aa684837c8328156505af273e536427c8dba62
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2236472Reviewed-by: default avatarDenis Kuznetsov [CET] <antrim@chromium.org>
Commit-Queue: Denis Kuznetsov [CET] <antrim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#776203}
parent 6ac4a0b6
...@@ -437,6 +437,9 @@ function initializeOobe() { ...@@ -437,6 +437,9 @@ function initializeOobe() {
return; return;
document.removeEventListener('DOMContentLoaded', initializeOobe); document.removeEventListener('DOMContentLoaded', initializeOobe);
// TODO(crbug.com/1082670): Remove excessive logging after investigation.
console.warn('1082670 : initializing OOBE');
try { try {
Oobe.initialize(); Oobe.initialize();
} finally { } finally {
...@@ -456,6 +459,9 @@ window.onerror = function(message, file, line, column, error) { ...@@ -456,6 +459,9 @@ window.onerror = function(message, file, line, column, error) {
console.error(error.stack); console.error(error.stack);
}; };
// TODO(crbug.com/1082670): Remove excessive logging after investigation.
console.warn('1082670 : cr_ui loaded');
/** /**
* Final initialization performed after DOM and all scripts have loaded. * Final initialization performed after DOM and all scripts have loaded.
*/ */
......
...@@ -294,6 +294,9 @@ void CoreOobeHandler::SetShelfHeight(int height) { ...@@ -294,6 +294,9 @@ void CoreOobeHandler::SetShelfHeight(int height) {
} }
void CoreOobeHandler::HandleInitialized() { void CoreOobeHandler::HandleInitialized() {
// TODO(crbug.com/1082670): Remove excessive logging after investigation.
LOG(ERROR) << "1082670 : CoreOobeHandler::HandleInitialized";
GetOobeUI()->InitializeHandlers(); GetOobeUI()->InitializeHandlers();
AllowJavascript(); AllowJavascript();
} }
......
...@@ -545,6 +545,9 @@ void OobeUI::BindInterface( ...@@ -545,6 +545,9 @@ void OobeUI::BindInterface(
OobeUI::OobeUI(content::WebUI* web_ui, const GURL& url) OobeUI::OobeUI(content::WebUI* web_ui, const GURL& url)
: ui::MojoWebUIController(web_ui, true /* enable_chrome_send */) { : ui::MojoWebUIController(web_ui, true /* enable_chrome_send */) {
// TODO(crbug.com/1082670): Remove excessive logging after investigation.
LOG(ERROR) << "1082670 : Creating new OobeUI";
display_type_ = GetDisplayType(url); display_type_ = GetDisplayType(url);
js_calls_container_ = std::make_unique<JSCallsContainer>(); js_calls_container_ = std::make_unique<JSCallsContainer>();
...@@ -643,8 +646,14 @@ void OobeUI::AddScreenHandler(std::unique_ptr<BaseScreenHandler> handler) { ...@@ -643,8 +646,14 @@ void OobeUI::AddScreenHandler(std::unique_ptr<BaseScreenHandler> handler) {
} }
void OobeUI::InitializeHandlers() { void OobeUI::InitializeHandlers() {
// TODO(crbug.com/1082670): Remove excessive logging after investigation.
LOG(ERROR) << "1082670 : OobeUI::InitializeHandlers";
js_calls_container_->ExecuteDeferredJSCalls(web_ui()); js_calls_container_->ExecuteDeferredJSCalls(web_ui());
// TODO(crbug.com/1082670): Remove excessive logging after investigation.
LOG(ERROR) << "1082670 : OobeUI::Marking as ready and executing callbacks";
ready_ = true; ready_ = true;
for (size_t i = 0; i < ready_callbacks_.size(); ++i) for (size_t i = 0; i < ready_callbacks_.size(); ++i)
ready_callbacks_[i].Run(); ready_callbacks_[i].Run();
...@@ -675,6 +684,9 @@ bool OobeUI::IsScreenInitialized(OobeScreenId screen) { ...@@ -675,6 +684,9 @@ bool OobeUI::IsScreenInitialized(OobeScreenId screen) {
} }
bool OobeUI::IsJSReady(const base::Closure& display_is_ready_callback) { bool OobeUI::IsJSReady(const base::Closure& display_is_ready_callback) {
// TODO(crbug.com/1082670): Remove excessive logging after investigation.
LOG(ERROR) << "1082670 : OobeUI::IsJSReady? = " << ready_;
if (!ready_) if (!ready_)
ready_callbacks_.push_back(display_is_ready_callback); ready_callbacks_.push_back(display_is_ready_callback);
return ready_; return ready_;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment