Closed Bug 701627 Opened 12 years ago Closed 12 years ago

slashdot.org doesn't fire MozScrolledAreaChanged events as the page gets bigger while loading

Categories

(Firefox for Android Graveyard :: General, defect, P1)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
x86
macOS
Type:
defect

Tracking

(firefox11 fixed, fennec11+)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox11 --- fixed
fennec 11+ ---

People

(Reporter: pcwalton, Assigned: dougt)

References

(
URL
)

Details

Attachments

(1 file, 1 obsolete file)

patch v.1
1.60 KB, patch
mfinkle
: review+
Details | Diff | Splinter Review 
Because we resize the page only upon receiving MozScrolledAreaChanged, we end up with a page much smaller than it should be. (How did XUL Fennec do this? It must have had the same issue...)
Blocks: native_droid_panning
I think we had similar problems with the xul fennec.  whenever we did complete loading, we asked "how big are you".  A workaround, yes.
Assignee: nobody → doug.turner
Priority: -- → P1
Attached patch patch v.1 (obsolete) — — Splinter Review 

        Attachment #577481 -
        Flags: review?(mark.finkle)

    
    

    
  

      
      
      
      

        Attached patch
          
          
        patch v.1
        — 
          — Splinter Review
      

    


  

        Attachment #577481 -
        Attachment is obsolete: true

        Attachment #577481 -
        Flags: review?(mark.finkle)

        Attachment #577483 -
        Flags: review?(mark.finkle)

    
    

    
  
Comment on attachment 577483 [details] [diff] [review]
patch v.1


>diff --git a/mobile/android/chrome/content/browser.js b/mobile/android/chrome/content/browser.js
>   get viewport() {

>-    let doc = this.browser.contentDocument.documentElement;
>+    let cdoc = this.browser.contentDocument;

Call me crazy, but I like "doc" better than "cdoc"

        Attachment #577483 -
        Flags: review?(mark.finkle) → review+

    
    

    
  
http://hg.mozilla.org/projects/birch/rev/82f3392976b4
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
tracking-fennec: --- → 11+
Product: Firefox for Android → Firefox for Android Graveyard

          You need to log in
          before you can comment on or make changes to this bug.