Skip to content
This repository has been archived by the owner on Nov 7, 2018. It is now read-only.

Commit

Permalink
Bug 1393424 - Don't try to autoscroll in Print Preview mode. r=kats
Browse files Browse the repository at this point in the history
It doesn't work properly anyways due to a more general coordinate-system
problem on the Print Preview page (see bug 1393494).

MozReview-Commit-ID: D4i2H8z5mtW
  • Loading branch information
theres-waldo committed Sep 1, 2017
1 parent 2965e24 commit f86dce2
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions toolkit/content/browser-content.js
Original file line number Diff line number Diff line change
Expand Up @@ -139,6 +139,12 @@ var ClickEventHandler = {
if (!this._scrollable)
return;

// In some configurations like Print Preview, content.performance
// (which we use below) is null. Autoscrolling is broken in Print
// Preview anyways (see bug 1393494), so just don't start it at all.
if (!content.performance)
return;

let domUtils = content.QueryInterface(Ci.nsIInterfaceRequestor)
.getInterface(Ci.nsIDOMWindowUtils);
let scrollable = this._scrollable;
Expand Down

0 comments on commit f86dce2

Please sign in to comment.