Closed Bug 1259928 Opened 4 years ago Closed 4 years ago

"View Selection Source" throws TypeError: args is null Source File: chrome://global/content/viewPartialSource.js

Categories

(SeaMonkey :: General, defect)

SeaMonkey 2.40 Branch
Unspecified
Windows 7
defect
Not set
major

Tracking

(seamonkey2.41? affected, seamonkey2.42? fixed, seamonkey2.43+ fixed, seamonkey2.44+ fixed, seamonkey2.46+ fixed)

RESOLVED FIXED
seamonkey2.45
Tracking Status
seamonkey2.41 ? affected
seamonkey2.42 ? fixed
seamonkey2.43 + fixed
seamonkey2.44 + fixed
seamonkey2.46 + fixed

People

(Reporter: hhofer42, Assigned: philip.chee)

References

Details

Attachments

(1 file, 1 obsolete file)

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:43.0) Gecko/20100101 Firefox/43.0 SeaMonkey/2.40
Build ID: 20160120202951

Steps to reproduce:

Select text from a web page and select "View Selection Source" from the context menu.


Actual results:

An empty window opens and the following is logged in the error console:

> Error: TypeError: args is null
> Source File: chrome://global/content/viewPartialSource.js
> Line: 20


Expected results:

A window showing the partial HTML-source should have opened.

I'm almost sure this still worked in SeaMonkey 2.39
Probably DUP of "Bug 1203395 - [window mode] View Selection Source fails"

@Reporter: Thank you for your attention. 
Please feel free to reopen this Bug if you find out that the fix for the other bug does not solve your problem or if you see other well founded indication that you fond an independent bug.
Status: UNCONFIRMED → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1203395
Sorry, but I don't think bug 1203395 fixed the issue for SeaMonkey.

I can see the changes from https://hg.mozilla.org/releases/mozilla-aurora/rev/d6f44c82219b ,
and the error mentioned in bug 12093395#c0 is different.

So, on the contrary, I think this issue is a _consequence_ of the fix for this bug.
(Probably the part with "let args = window.arguments[0]" instead of "let args = window.arguments" in "viewPartialSource.js")

It affects SeaMonkey 2.40 as well as the latest available nightly (2.45a1, Build ID: 20160308001946) under Win 7.
Status: RESOLVED → UNCONFIRMED
Resolution: DUPLICATE → ---
(In reply to H. Hofer from comment #2)
> So, on the contrary, I think this issue is a _consequence_ of the fix for
> this bug.
Indeed! 
I will do some additional investigation in the evening.
Steps how to reproduce with en-US SeaMonkey 2.40 final Mozilla/5.0 (Windows NT 6.1; WOW64; rv:43.0) from official download page,  Gecko/20100101  Firefox/ 43.0  Build 20160120202951, (Classic Theme) on German WIN7 64bit:

0. 'Launch Browser to show this page → Launch Error Console'
1. Select (highlight) first paragraph of oritinal report here
2. 'Rightclick → View selection source'
   Bug: Empty window will be opened instead of Window showing selections source
   Error Console: as reported

Works fine with FF 48.0a1 (2016-03-27)

It seems that fix for "Bug 1203395 - [window mode] View Selection Source fails " does not work here, but causes additional problem?
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Unspecified → Windows 7
See Also: → 1203395
Still REPRODUCIBLE with  English SeaMonkey 2.45a1  (Windows NT 6.1; WOW64; rv:48.0)  Gecko/20100101 Firefox/48.0 Build 20160308001946  (Default Classic Theme)  on German WIN7 64bit.

Probably 2.41 ... 2.44 also are affected.
A previously existing capability is now broken without any acceptable workaround.  Thus, this is a Major bug, not a Normal bug.
Severity: normal → major
Depends on: 1203395, 1134585
Summary: "View Selection Source" throws TypeError → "View Selection Source" throws TypeError: args is null Source File: chrome://global/content/viewPartialSource.js
Attached patch Working patch WIP (obsolete) — Splinter Review
This patch fixes the error. Now doing a build to confirm.
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
This is a minimal fix for the view selection source. Enhancements and other fancy stuff can wait for another patch. Meanwhile we need to get this into all branches.
Attachment #8736805 - Attachment is obsolete: true
Attachment #8736916 - Flags: review?(iann_bugzilla)
Duplicate of this bug: 1261700
Comment on attachment 8736916 [details] [diff] [review]
Patch v1.0 simplified fix

[Triage Comment]
a/r=me for c-c
a=me for c-a/c-b
a=me for esr if required
Attachment #8736916 - Flags: review?(iann_bugzilla)
Attachment #8736916 - Flags: review+
Attachment #8736916 - Flags: approval-comm-beta+
Attachment #8736916 - Flags: approval-comm-aurora+
Status: ASSIGNED → RESOLVED
Closed: 4 years ago4 years ago
Resolution: --- → FIXED
Duplicate of this bug: 1320881
You need to log in before you can comment on or make changes to this bug.