summaryrefslogtreecommitdiffstats
path: root/assets
diff options
context:
space:
mode:
authorAndrew Sapperstein <asapperstein@google.com>2014-09-30 14:28:25 -0700
committerAndrew Sapperstein <asapperstein@google.com>2014-09-30 14:28:25 -0700
commit7de8dd863124e8f11f2d803ad4afdb32ddb32599 (patch)
tree89165b8e665e12e23bce4071f3d6ee1b687312cc /assets
parentfb2ed0163ce3b366f892dca5128b25d59bab2886 (diff)
downloadandroid_packages_apps_UnifiedEmail-7de8dd863124e8f11f2d803ad4afdb32ddb32599.tar.gz
android_packages_apps_UnifiedEmail-7de8dd863124e8f11f2d803ad4afdb32ddb32599.tar.bz2
android_packages_apps_UnifiedEmail-7de8dd863124e8f11f2d803ad4afdb32ddb32599.zip
Add null check in JS before replacing message body
Fixes b/13060462. Change-Id: I6c33e56cd3e99c2021d41accca2bd129429dbeee
Diffstat (limited to 'assets')
-rw-r--r--assets/script.js10
1 files changed, 8 insertions, 2 deletions
diff --git a/assets/script.js b/assets/script.js
index 2bf5d9c54..32cc85686 100644
--- a/assets/script.js
+++ b/assets/script.js
@@ -754,8 +754,14 @@ function replaceMessageBodies(messageIds) {
for (i = 0, len = messageIds.length; i < len; i++) {
id = messageIds[i];
msgContentDiv = document.querySelector("#" + id + " > .mail-message-content");
- msgContentDiv.innerHTML = window.mail.getMessageBody(id);
- processNewMessageBody(msgContentDiv);
+ // Check if we actually have a div before trying to replace this message body.
+ if (msgContentDiv) {
+ msgContentDiv.innerHTML = window.mail.getMessageBody(id);
+ processNewMessageBody(msgContentDiv);
+ } else {
+ // There's no message div, just skip it. We're in a really busted state.
+ console.log("Mail message content for msg " + id + " to replace not found.");
+ }
}
disablePostForms();
measurePositions();