Closed
Bug 1183902
Opened 9 years ago
Closed 9 years ago
_shareStringWithDefault is unused
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox42 fixed)
RESOLVED
FIXED
Firefox 42
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: mcomella, Assigned: gioyik, Mentored)
Details
(Whiteboard: [lang=js][good first bug])
Attachments
(1 file)
1.12 KB,
patch
|
mcomella
:
review+
|
Details | Diff | Splinter Review |
A definition and no uses: https://mxr.mozilla.org/mozilla-central/search?string=_shareStringWithDefault&find=&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central Margaret, I'm not missing anything, right?
Flags: needinfo?(margaret.leibovic)
Comment 1•9 years ago
|
||
Looking at blame, this was added in this changeset: http://hg.mozilla.org/mozilla-central/rev/02b6677f5f8d And then its consumers were removed here: http://hg.mozilla.org/mozilla-central/rev/c437a98128f4 I guess that "showAsActions" property just takes care of this for us now. So yeah, I think you can kill this.
Flags: needinfo?(margaret.leibovic)
Reporter | ||
Comment 2•9 years ago
|
||
To start, set up a build environment - you can see the instructions here: https://wiki.mozilla.org/Mobile/Fennec/Android Then, you'll need to create a patch to upload - see https://wiki.mozilla.org/Mobile/Fennec/Android#Creating_commits_and_submitting_patches If you need any help, you can reply to this bug, or feel free to message me on IRC - my nick is "mcomella" and you can find me in #mobile. If you need IRC setup instructions, see https://wiki.mozilla.org/IRC Thanks and happy coding! ^_^
Mentor: michael.l.comella, margaret.leibovic
Whiteboard: [lang=js][good first bug]
Assignee | ||
Comment 3•9 years ago
|
||
Attachment #8637647 -
Flags: review?(michael.l.comella)
Assignee | ||
Updated•9 years ago
|
Attachment #8637647 -
Flags: review?(michael.l.comella)
Assignee | ||
Updated•9 years ago
|
Attachment #8637647 -
Flags: review?(michael.l.comella) → review?(margaret.leibovic)
Assignee | ||
Comment 4•9 years ago
|
||
Hi, I submitted a patch for this bug. Let me know if is a good one or needs modifications. Thanks P.S Sorry for the double review with the same person.
Reporter | ||
Comment 5•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=67157c1a858c
Reporter | ||
Comment 6•9 years ago
|
||
Comment on attachment 8637647 [details] [diff] [review] 1183902.patch Review of attachment 8637647 [details] [diff] [review]: ----------------------------------------------------------------- I made a push to our try test servers (above). Once the push goes green, you can add the "checkin-needed" keyword [1] to get your patch checked in. Note that all patches added via checkin-needed keyword need an associated green try run. Let me know if you need help reading the results. [1]: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch#Getting_the_patch_checked_into_the_tree
Attachment #8637647 -
Flags: review?(michael.l.comella)
Attachment #8637647 -
Flags: review?(margaret.leibovic)
Attachment #8637647 -
Flags: review+
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → gioyik
Assignee | ||
Comment 7•9 years ago
|
||
Michael, Treeherder is showing 3 test as failed. Could you help me to understand how fix them? Thanks
Flags: needinfo?(michael.l.comella)
Reporter | ||
Comment 8•9 years ago
|
||
(In reply to Giovanny Gongora [:gioyik] from comment #7) > Treeherder is showing 3 test as failed. Could you help me to understand how > fix them? Our test suites often get intermittent failures and this looks like one of them. To check, you can see if the suite has been re-run (i.e. there is another test suite with the same name on the same device) and if that was green. Additionally, if you click a failing test suite, you can look at the bar on the bottom of the screen and see if it matches any existing known intermittent failures, which are also listed on that bar. So this looks good to me and is ready for check-in!
Flags: needinfo?(michael.l.comella)
Reporter | ||
Updated•9 years ago
|
Flags: needinfo?(gioyik)
Assignee | ||
Comment 9•9 years ago
|
||
That's good to know. Thank you for the feedback.
Flags: needinfo?(gioyik)
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 10•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/f846219745ee
Keywords: checkin-needed
Comment 11•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f846219745ee
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•