struct WindowAction in WorkerPrivate.cpp should be marked MOZ_RAII

RESOLVED INVALID

Status

()

RESOLVED INVALID
a year ago
a year ago

People

(Reporter: baku, Assigned: baku)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

a year ago
This helps to understand why we have a raw pointer for mWindow and why that is OK.
(Assignee)

Comment 1

a year ago
Created attachment 8869027 [details] [diff] [review]
raii.patch
Assignee: nobody → amarchesini
Attachment #8869027 - Flags: review?(kyle)

Comment 2

a year ago
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3a6693755c7d
struct WindowAction in WorkerPrivate.cpp should be marked MOZ_RAII, r=qdot
Backed out for bustage at nsTArray.h:568:

https://hg.mozilla.org/integration/mozilla-inbound/rev/0e48946fd355251d61767f7ec6dd4a712bc36d9d

Push with failures (the win32 bustage is something different): https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=3a6693755c7d248bf40d6cf5caea365677da12ed&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=usercancel&filter-resultStatus=runnable
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=100225336&repo=mozilla-inbound

[task 2017-05-18T21:17:48.478612Z] 21:17:48     INFO -  In file included from /home/worker/workspace/build/src/dom/workers/WorkerLocation.cpp:7:
[task 2017-05-18T21:17:48.478842Z] 21:17:48     INFO -  In file included from /home/worker/workspace/build/src/obj-firefox/dist/include/mozilla/dom/WorkerLocation.h:10:
[task 2017-05-18T21:17:48.479018Z] 21:17:48     INFO -  In file included from /home/worker/workspace/build/src/obj-firefox/dist/include/mozilla/dom/workers/Workers.h:18:
[task 2017-05-18T21:17:48.479354Z] 21:17:48     INFO -  /home/worker/workspace/build/src/obj-firefox/dist/include/nsTArray.h:568:62: error: variable of type '(anonymous namespace)::WindowAction' is not valid in a temporary
[task 2017-05-18T21:17:48.479528Z] 21:17:48     INFO -    bool Equals(const A& aA, const B& aB) const { return aA == aB; }
[task 2017-05-18T21:17:48.479735Z] 21:17:48     INFO -                                                               ^
Flags: needinfo?(amarchesini)
(Assignee)

Comment 4

a year ago
Unfortunately, this cannot be done.
Status: NEW → RESOLVED
Last Resolved: a year ago
Flags: needinfo?(amarchesini)
Resolution: --- → FIXED
(Assignee)

Updated

a year ago
Resolution: FIXED → INVALID
You need to log in before you can comment on or make changes to this bug.