Closed Bug 1073971 Opened 10 years ago Closed 10 years ago

avoid some warnings in "about:" page used in page-mod tests

Categories

(Add-on SDK Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla35

People

(Reporter: zombie, Assigned: zombie)

References

Details

Attachments

(1 file)

      No description provided.
Blocks: 1058698
Summary: shut → avoid some warnings in "about:" page used in page-mod tests
20 warnings logged:

  console.log: addon-sdk: [JavaScript Error: "formatURLPref: Couldn't get pref: 
  app.releaseNotesURL" {file: "resource://gre/components/nsURLFormatter.js" line: 134}]

makes it hard to read the test logs..
Assignee: nobody → tomica+amo
Status: NEW → ASSIGNED
(i'm breaking out mostly-independent changes from my big PR for bug 1058698..)
Comment on attachment 8496573 [details] [review]
Link to Github pull-request: https://github.com/mozilla/addon-sdk/pull/1660

Thanks!

Looks good, just a minor nit to use a url more like `http://localhost/app-dummy/`
Attachment #8496573 - Flags: review?(evold) → review-
Comment on attachment 8496573 [details] [review]
Link to Github pull-request: https://github.com/mozilla/addon-sdk/pull/1660

don't think it makes a difference -- this does not result in a network request, just a read from prefs assigned to a href of a link on the "about:" page, but whatever..
Attachment #8496573 - Flags: review- → review?(evold)
Attachment #8496573 - Flags: review?(evold) → review?(jsantell)
Attachment #8496573 - Flags: review?(jsantell) → review+
Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/9f20e29e16b8ae0cd6a7f8781c8e4abb740ea7b4
bug 1073971 - avoid some warnings on about: page

https://github.com/mozilla/addon-sdk/commit/5fbe6782acec44badc8f6d6268d5ad37366e557e
Merge pull request #1660 from zombie/1073971-about-warnings

bug 1073971 - avoid some warnings on about: page, r=@jsantell
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: