Closed
Bug 1109694
Opened 9 years ago
Closed 9 years ago
Fix more bad implicit constructors in DOM
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
Attachments
(2 files)
15.88 KB,
patch
|
baku
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
5.70 KB,
patch
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8534389 -
Flags: review?(amarchesini)
Comment 2•9 years ago
|
||
Comment on attachment 8534389 [details] [diff] [review] Fix more bad implicit constructors in DOM Review of attachment 8534389 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/canvas/WebGLBindableName.h @@ +53,5 @@ > : public WebGLBindable<T> > { > public: > > + explicit WebGLBindableName(GLuint name) aName
Attachment #8534389 -
Flags: review?(amarchesini) → review+
https://hg.mozilla.org/mozilla-central/rev/aa98e8506630
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Comment 4•9 years ago
|
||
Comment on attachment 8534389 [details] [diff] [review] Fix more bad implicit constructors in DOM Approval Request Comment [Feature/regressing bug #]: MSE [User impact if declined]: Less consistent testing, sites more likely to serve Flash video. [Describe test coverage new/current, TBPL]: Landed on m-c. [Risks and why]: Minimal. Small code-correctness fix. [String/UUID change made/needed]: None.
Attachment #8534389 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
status-firefox36:
--- → affected
status-firefox37:
--- → fixed
Updated•9 years ago
|
Attachment #8534389 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 5•9 years ago
|
||
NB, I think we should backport only the subset of this patch affecting dom/media/
Comment 6•9 years ago
|
||
Backport patch for landing on Aurora in support of MSE.
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•