Closed Bug 1342760 Opened 6 years ago Closed 6 years ago

Remove remaining version parameters of script elements from dom/

Categories

(Core :: DOM: Core & HTML, defect)

50 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: emk, Assigned: emk)

References

()

Details

Attachments

(1 file)

I missed them because these files do not have one of .html/.xhtml/.js/.xul extensions.
Summary: Remove remaining version parameters from dom/ → Remove remaining version parameters of script elements from dom/
Assignee: nobody → VYV03354
Status: NEW → ASSIGNED
Comment on attachment 8841327 [details]
Bug 1342760 - Remove remaining version parameters of script elements from dom/.

https://reviewboard.mozilla.org/r/115584/#review116994

::: dom/html/MediaDocument.cpp:352
(Diff revision 1)
>  
>    RefPtr<nsGenericHTMLElement> script = NS_NewHTMLScriptElement(nodeInfo.forget());
>    NS_ENSURE_TRUE(script, NS_ERROR_OUT_OF_MEMORY);
>  
>    script->SetAttr(kNameSpaceID_None, nsGkAtoms::type,
> -                  NS_LITERAL_STRING("text/javascript;version=1.8"), true);
> +                  NS_LITERAL_STRING("text/javascript"), true);

I assume this doesn't change the behavior
Attachment #8841327 - Flags: review?(bugs) → review+
Pushed by VYV03354@nifty.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/577365c3ba18
Remove remaining version parameters of script elements from dom/. r=smaug
https://hg.mozilla.org/mozilla-central/rev/577365c3ba18
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.