Closed Bug 1027831 Opened 6 years ago Closed 6 years ago

Switch from deprecated FILL_PARENT to MATCH_PARENT in Java and XML

Categories

(Firefox for Android :: General, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 33

People

(Reporter: bnicholson, Assigned: hathymo, Mentored)

Details

(Whiteboard: [lang=java][good first bug])

Attachments

(1 file, 1 obsolete file)

We have tons of deprecated FILL_PARENT usages in our tree:

http://mxr.mozilla.org/mozilla-central/search?string=fill_parent&find=base&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

These should all be switched to MATCH_PARENT.
Mentor: bnicholson
Whiteboard: [mentor=bnicholson][lang=java] → [lang=java]
Whiteboard: [lang=java] → [lang=java][good first bug]
Occurrences of "fill_parent" have been replaced with "match_parent", and occurrences of FILL_PARENT with MATCH_PARENT.
Attachment #8443424 - Flags: review?(bnicholson)
Attachment #8443424 - Attachment is obsolete: true
Attachment #8443424 - Flags: review?(bnicholson)
Replaced occurrences of fill_parent with match_parent in the XML files, and FILL_PARENT with MATCH_PARENT in the .java files.
Attachment #8443443 - Flags: review?(bnicholson)
Comment on attachment 8443443 [details] [diff] [review]
1027831.patch - Replaced occurrences of fill_parent with match_parent, and FILL_PARENT with MATCH_PARENT

Review of attachment 8443443 [details] [diff] [review]:
-----------------------------------------------------------------

Looks perfect, thanks!
Attachment #8443443 - Flags: review?(bnicholson) → review+
Thanks! This is my first bug fix. Could you please help me push this code? Is there anything I need to do to make it happen?
I already took care of that for you in comment 4. :)

Normally, you would ask for try server access (https://tbpl.mozilla.org/?tree=Try), push your patch there to make sure it passes all of our tests, then set the checkin-needed flag in the bug for a volunteer to land for you.

In this case, however, you have a patch that touches many files, so I wanted to land it ASAP to minimize bitrot. The changes in the patch are also quite trivial, and have a very low chance of breaking anything.
https://hg.mozilla.org/mozilla-central/rev/426fabc9a7fa
Assignee: nobody → hathymo
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 33
You need to log in before you can comment on or make changes to this bug.