Last Comment Bug 735490 - Delete mobile/android/chrome/tests/
: Delete mobile/android/chrome/tests/
Status: RESOLVED FIXED
:
Product: Firefox for Android
Classification: Client Software
Component: General (show other bugs)
: unspecified
: ARM Android
: -- normal (vote)
: Firefox 14
Assigned To: Richard Newman [:rnewman]
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-13 15:35 PDT by Richard Newman [:rnewman]
Modified: 2012-03-24 09:24 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---
fixed
fixed


Attachments
Proposed patch. v1 (9.16 KB, patch)
2012-03-13 15:38 PDT, Richard Newman [:rnewman]
mbrubeck: review+
Details | Diff | Splinter Review
Fix! (620 bytes, patch)
2012-03-13 16:00 PDT, Richard Newman [:rnewman]
mbrubeck: review+
Details | Diff | Splinter Review

Description Richard Newman [:rnewman] 2012-03-13 15:35:16 PDT
15:33:56 < mbrubeck> rnewman: Left over from XUL Fennec, never cleaned up.
15:34:02 < mbrubeck> rnewman: It's unused and can be deleted.
Comment 1 Richard Newman [:rnewman] 2012-03-13 15:38:22 PDT
Created attachment 605571 [details] [diff] [review]
Proposed patch. v1
Comment 2 Matt Brubeck (:mbrubeck) 2012-03-13 15:43:35 PDT
Comment on attachment 605571 [details] [diff] [review]
Proposed patch. v1

You can actually delete the entire mobile/android/chrome/tests directory if you'd like -- all of these browser-chrome tests are left over from when we forked XUL Fennec, and none of them will run.
Comment 3 Richard Newman [:rnewman] 2012-03-13 15:50:09 PDT
Nuked!

https://hg.mozilla.org/integration/mozilla-inbound/rev/62c8e6a841e7
Comment 4 Richard Newman [:rnewman] 2012-03-13 16:00:32 PDT
Created attachment 605579 [details] [diff] [review]
Fix!
Comment 5 Richard Newman [:rnewman] 2012-03-13 16:03:37 PDT
Follow-up to remove test directory from makefile.sh:

https://hg.mozilla.org/integration/mozilla-inbound/rev/97741a41d74f
Comment 6 Richard Newman [:rnewman] 2012-03-13 18:24:50 PDT
Missed a spot in the chrome makefile.

https://hg.mozilla.org/integration/mozilla-inbound/rev/ae6e1959cbb9
Comment 9 Mark Finkle (:mfinkle) (use needinfo?) 2012-03-24 09:24:52 PDT
https://hg.mozilla.org/releases/mozilla-aurora/rev/bb9cf955a89b

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