Issue 3472 - Made sure that "Block element" option is shown after page load

popup.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 66 matching lines @@
   }
 }
 
 function onUnload()
 {
   ext.onMessage.removeListener(onMessage);
 }
 
 function onMessage(message, sender, callback)
 {
-  if (message.type == "report-html-page"
-      && backgroundPage.htmlPages.has(page))

[Sebastian Noack, 2016/01/09 00:01:50]

This check is redundant. Or if you want to keep it for consistence, please move
that logic including the DOM manipulation below to a function that is called
here as well when checking initially.

[Thomas Greiner, 2016/01/12 11:35:17]

Unfortunately, it's not redundant. Any page could've sent this message so there
needs to be an extra check to ensure that it concerns the page the popup is
associated with.

Generally, I'd like to avoid creating a function for one line of code,
especially since I can't use `document.body.classList.toggle("nohtml",
!backgroundPage.htmlPages.has(page))` due to Safari 6.1. Therefore I'm checking
the origin of the page because comparing `sender.page == page` doesn't work and
implementing it doesn't appear to be worth it.

[Sebastian Noack, 2016/01/12 16:09:16]

Acknowledged.
I don't think that I agree. But it's probably not too important. LGTM

+  if (message.type == "report-html-page" && sender.page._id == page._id)
     document.body.classList.remove("nohtml");
 }
 
 function toggleEnabled()
 {
   var disabled = document.body.classList.toggle("disabled");
   if (disabled)
   {
     var host = getDecodedHostname(page.url).replace(/^www\./, "");
     var filter = Filter.fromText("@@||" + host + "^$document");
@@ skipping 41 matching lines @@
 
 function toggleCollapse(event)
 {
   var collapser = event.currentTarget;
   Prefs[collapser.dataset.option] = !Prefs[collapser.dataset.option];
   collapser.parentNode.classList.toggle("collapsed");
 }
 
 document.addEventListener("DOMContentLoaded", onLoad, false);
 window.addEventListener("unload", onUnload, false);