Last Comment Bug 773127 - Need SettingsImportHelper something like AbImportHelper.
: Need SettingsImportHelper something like AbImportHelper.
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Testing Infrastructure (show other bugs)
: Trunk
: All All
: -- normal (vote)
: Thunderbird 17.0
Assigned To: Nobody; OK to take it and work on it
:
:
Mentors:
Depends on: 773124
Blocks: 772712
  Show dependency treegraph
 
Reported: 2012-07-11 19:09 PDT by Hiroyuki Ikezoe (:hiro)
Modified: 2012-07-18 18:27 PDT (History)
3 users (show)
ryanvm: in‑testsuite+
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Fix (15.62 KB, patch)
2012-07-11 19:09 PDT, Hiroyuki Ikezoe (:hiro)
mconley: review+
Details | Diff | Splinter Review

Description Hiroyuki Ikezoe (:hiro) 2012-07-11 19:09:46 PDT
Created attachment 641314 [details] [diff] [review]
Fix

To write test easier we need SettingsImportHelper something like AbImportHelper does in resources/import_helper.js.
Comment 1 Hiroyuki Ikezoe (:hiro) 2012-07-11 19:10:10 PDT
This patch depends on the fix for bug 773124.
Comment 2 Mike Conley (:mconley) 2012-07-18 12:32:55 PDT
Comment on attachment 641314 [details] [diff] [review]
Fix

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

Woo, you're on a roll! This looks great, thanks. :)
Comment 3 Ryan VanderMeulen [:RyanVM] 2012-07-18 18:27:11 PDT
https://hg.mozilla.org/comm-central/rev/62ddb46cd654

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