Closed Bug 1219132 Opened 6 years ago Closed 3 years ago

[FFOS2.0][Woodduck][System]The url in homescreen iframe is added by timestamp

Categories

(Firefox OS Graveyard :: Gaia::System, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: wenjie.chen.hz, Unassigned)

References

Details

Attachments

(1 file)

Attached image woodduckbug.png
Dear Mozilla:
  please see the atachement.
  In homescreen, if press home key,the url in iframe will grew.Cause this effect codes:
   gaia/apps/system/js/homescreen_window.js 176:
     HomescreenWindow.prototype.ensure = function hw_ensure(reset) {
    this.debug('ensuring homescreen...', this.frontWindow);
    if (!this.element) {
      this.render();
    } else if (reset) {
      if (this.frontWindow) {
        // Just kill front window but not switch to the first page.
        this.frontWindow.kill();
      } else {
        this.browser.element.src = this.browser_config.url + Date.now();
      }
    }

    return this.element;
  };
========================================
this.browser.element.src = this.browser_config.url + Date.now();
======================================
above code cause it.

so I modify it to:
/////////////////////////
this.browser.element.src = this.url + Date.now();
//////////////////////////
Is there any risk?
Wesly,
would you please help to support this?
thanks.
Flags: needinfo?(wehuang)
Please arrange some engineers to analyze the issue.
In addition, if we take the patch above, no frozen issues such as
https://bugzilla.mozilla.org/show_bug.cgi?id=1217725
any more reported from our PA team.

Could someone tell me something about this?
Hi Etienne:

Do you think it's possible for you to give some suggestion to our partner TCL's patch above, from Gaia:system point of view? Or actually this is unrelated to Gaia:system so you probably can point us a proper module?

Thanks!
Flags: needinfo?(wehuang) → needinfo?(etienne)
Assignee: wehuang → nobody
QA Contact: wehuang
See Also: → 1217725
I think this is a dupe of bug 1079699.
My strong suggestion is to take the patch from bug 1079699, it comes with a test :)
Flags: needinfo?(etienne)
Blocks: Woodduck
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.