diff options
author | Andrew Sapperstein <asapperstein@google.com> | 2014-09-30 14:28:25 -0700 |
---|---|---|
committer | Andrew Sapperstein <asapperstein@google.com> | 2014-09-30 14:28:25 -0700 |
commit | 7de8dd863124e8f11f2d803ad4afdb32ddb32599 (patch) | |
tree | 89165b8e665e12e23bce4071f3d6ee1b687312cc /assets | |
parent | fb2ed0163ce3b366f892dca5128b25d59bab2886 (diff) | |
download | android_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.js | 10 |
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(); |