584977 - mark deprecated caps interfaces and methods with [deprecated]

Status: RESOLVED FIXED

(Core :: Security: CAPS, defect)

Description

[timeless]

We have a way to mark interfaces and methods as deprecated which some compilers
are able to use to complain about deprecation. This bug is for sprinkling that
marker into caps idl files.

Please note that at this time we only properly trigger these compile time
warnings with msvc (bug 584953 covers the fixing the gcc side).

Comment 1

[timeless]

Attachment: patch

Comment 2

[Daniel Veditz [:dveditz]]

Comment on attachment 463471 [details] [diff] [review]
patch

There are still a couple of calls to these in our own tree (CheckLoadURI in nsDocShell and a couple CheckLoadURIStr in drag-n-drop code). How noisy does this get?

I think I need to defer to bz on this one.

Comment 3

[timeless]

they become build warnings, which today only appear on msvc because for all the other platforms we're broken :).

i'm taking out way more build warnings than i'm putting in and at the moment i'm driving build warnings, so i'd like the flexibility to add some.

Comment 4

[Dave Townsend [:mossop] (PTO until Aug 21st)]

Comment on attachment 463471 [details] [diff] [review]
patch

Per bug 584998 comment 2 this doesn't work as expected yet

Comment 5

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/205e81133b5f

Comment 6

[u279076]

Can someone confirm if this is fixed?