The "fake display port" hack is breaking sites

RESOLVED FIXED

Status

()

Firefox for Android
General
P1
normal
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: pcwalton, Assigned: pcwalton)

Tracking

unspecified
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
Created attachment 574387 [details] [diff] [review]
Proposed patch.

The "fake display port" hack will break sites because it monkey patches content CSS. This patch removes it.
Attachment #574387 - Flags: review?(kgupta)
(Assignee)

Comment 1

6 years ago
NB: This depends on bug 699351.
Depends on: 699351
(Assignee)

Comment 2

6 years ago
Created attachment 574388 [details] [diff] [review]
Proposed patch, version 2.

Oops, uploaded a bad version of the patch.
Attachment #574387 - Attachment is obsolete: true
Attachment #574387 - Flags: review?(kgupta)
Attachment #574388 - Flags: review?(kgupta)
Comment on attachment 574387 [details] [diff] [review]
Proposed patch.

>     let data = JSON.parse(aData);
>+    let browser = this.selectedBrowser;
>+    browser.scrollTo(data.x, data.y);

Should this be browser.contentWindow.scrollTo?
Attachment #574388 - Flags: review?(kgupta) → review+

Updated

6 years ago
Priority: -- → P1
http://hg.mozilla.org/projects/birch/rev/978e96c62262
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
You need to log in before you can comment on or make changes to this bug.