Skip to content

Commit

Permalink
Revert "Test for 'interactive' readyState during app bootstrap"
Browse files Browse the repository at this point in the history
This reverts commit f47ba0e.

Reason for revert: Introduces errors in IE9, this event can go off too
soon.

Change-Id: I16256254d3a2e5b88415720fc5dd2ab83335cc56
  • Loading branch information
niloc132 committed Oct 18, 2017
1 parent d9fe189 commit faff18e
Show file tree
Hide file tree
Showing 8 changed files with 8 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
__MAIN__
};

if (/loaded|interactive|complete/.test($doc.readyState)) {
if (/loaded|complete/.test($doc.readyState)) {
executeMain();
} else {
//defer app script insertion until the body is ready
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@
$head.insertBefore(devModeScript, $head.firstElementChild || $head.children[0]);
};

if (/loaded|interactive|complete/.test($doc.readyState)) {
if (/loaded|complete/.test($doc.readyState)) {
injectScriptTag();
} else {
//defer app script insertion until the body is ready
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -440,7 +440,7 @@ function __MODULE_FUNC__() {

// Fallback. If onBodyDone() gets fired twice, it's not a big deal.
var onBodyDoneTimerId = setInterval(function() {
if (/loaded|interactive|complete/.test($doc.readyState)) {
if (/loaded|complete/.test($doc.readyState)) {
maybeInjectFrame();
onBodyDone();
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@ function isBodyLoaded() {
// in isBodyLoadedFf35Fix.js
return (typeof $doc.body != "undefined" && $doc.body != null);
}
return (/loaded|interactive|complete/.test($doc.readyState));
return (/loaded|complete/.test($doc.readyState));
}
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,5 @@ function isBodyLoaded() {
// the body is loaded.
return false;
}
return (/loaded|interactive|complete/.test($doc.readyState));
return (/loaded|complete/.test($doc.readyState));
}
2 changes: 1 addition & 1 deletion dev/core/src/com/google/gwt/core/linker/IFrameTemplate.js
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,7 @@ function __MODULE_FUNC__() {

// Fallback. If onBodyDone() gets fired twice, it's not a big deal.
var onBodyDoneTimerId = setInterval(function() {
if (/loaded|interactive|complete/.test($doc.readyState)) {
if (/loaded|complete/.test($doc.readyState)) {
maybeInjectFrame();
onBodyDone();
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,7 @@ function __MODULE_FUNC__() {

// Fallback. If onBodyDone() gets fired twice, it's not a big deal.
var onBodyDoneTimerId = setInterval(function() {
if (/loaded|interactive|complete/.test($doc.readyState)) {
if (/loaded|complete/.test($doc.readyState)) {
onBodyDone();
}
}, 50);
Expand Down
2 changes: 1 addition & 1 deletion dev/core/src/com/google/gwt/core/linker/XSTemplate.js
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ function __MODULE_FUNC__() {

// Fallback. If onBodyDone() gets fired twice, it's not a big deal.
var onBodyDoneTimerId = setInterval(function() {
if (/loaded|interactive|complete/.test($doc.readyState)) {
if (/loaded|complete/.test($doc.readyState)) {
onBodyDone();
}
}, 50);
Expand Down

0 comments on commit faff18e

Please sign in to comment.