• beaudoin@chromium.org's avatar
    - Most visited iframe now uses postMessage() to signal the iframing page that... · 1e3247ea
    beaudoin@chromium.org authored
    - Most visited iframe now uses postMessage() to signal the iframing page that the link has been displayed.
    - Refactor most_visited_thumbnail.js to eliminate client-side secondary thumbnail fallback.
    - Refactor UMA logging of NTP suggestions events.
    
    The new postMessage() signal is sent whenever the iframe has failed loading the thumbnails, or when the thumbnail has been successfully loaded. This way it's impossible for the iframing page to learn whether the user has a given URL in its TopSites. The iframing page can use that signal to know when it's possible to display a fallback visual under the iframe.
    
    The refactor significantly simplifies the iframe javascript by dropping support for multiple thumbnails, which can now be handled by the iframing page. UMA logging of NTP suggestions related statistics have also been both simplified and augmented. The patch also includes better unit tests for the UMA logging.
    
    BUG=None
    
    Review URL: https://codereview.chromium.org/102433009
    
    git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245579 0039d316-1c4b-4281-b951-d872f2087c98
    1e3247ea
most_visited_util.js 7.3 KB