Closed
Bug 649825
Opened 13 years ago
Closed 13 years ago
refactor DeviceManager to allow multiple underlying implementations
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: blassey, Assigned: blassey)
References
Details
Attachments
(1 file, 1 obsolete file)
46.24 KB,
patch
|
cmtalbert
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Comment 2•13 years ago
|
||
this changes how you initialize the devicemanager, but does not change runtestsremote.py to do it right (the patch in bug 650205 does that). So if these land separately there will have to be additional changes to make things work.
Assignee: nobody → blassey.bugs
Attachment #526005 -
Attachment is obsolete: true
Attachment #530753 -
Flags: review?(ctalbert)
Comment 3•13 years ago
|
||
this patch applied directly to a m-c tree has 7/8 hunks failed to apply.
Comment 4•13 years ago
|
||
oops, I downloaded and applied the wrong patch, the latest patch works just fine
Comment 5•13 years ago
|
||
I ran this with the patch from bug 650205 and I passed all my devicemanager unit tests (using SUT agent) as well as a handful of reftest and mochitest runs. In looking at this patch I don't see anything that throws up red flags.
Comment on attachment 530753 [details] [diff] [review] patch This looks great. Thanks Blassey. I like the way you created pure virutal methods in the DeviceManager class. r+ Joel, I can get the patch together to push this to remotetesting/ and droidfarm/. Do you want to do the talos one?
Attachment #530753 -
Flags: review?(ctalbert) → review+
Comment 7•13 years ago
|
||
once this lands on m-c, I will do the talos patch.
Comment 8•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/8ad0dfefa6fa
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•