793425 - Bookmarks Manager Tools-> Import HTML Wizard not working due to sytax error in migration.js

Status: RESOLVED FIXED

(SeaMonkey :: Bookmarks & History, defect)

Description

[Philip Chee]

Sun Sep 23 2012 01:31:10
Error: SyntaxError: missing ; before statement
Source file: chrome://communicator/content/migration/migration.js
Line: 44, Column: 45
Source code:
this._migrator = window.arguments[1] instance of nsISuiteProfileMigrator ?

Regression from Bug 779716

Comment 1

[Philip Chee]

Attachment: Patch v1.0 fixit.

Comment 2

[[e-mail]]

Comment on attachment 663723 [details] [diff] [review]
Patch v1.0 fixit.

[Triage Comment]

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: 
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky):
String changes made by this patch:

Comment 3

[Philip Chee]

Regression caused by (bug #): Bug 779716
User impact if declined: Bookmarks Import Wizard fails to work properly.
Testing completed (on m-c, etc.): yes
Risk to taking this patch (and alternatives if risky): one line typo fix. No risk.
String changes made by this patch: None.

Comment 4

[Philip Chee]

Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/1c7bf0fd95c7
and comm-aurora:
http://hg.mozilla.org/releases/comm-aurora/rev/c6c65a22ceef

Comment 6

[Justin Wood (:Callek)]

Comment on attachment 663723 [details] [diff] [review]
Patch v1.0 fixit.

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

http://hg.mozilla.org/releases/comm-beta/rev/82714f12926b