As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 762801 - Merge nsIEmbeddingSiteWindow2 into nsIEmbeddingSiteWindow
: Merge nsIEmbeddingSiteWindow2 into nsIEmbeddingSiteWindow
Product: Core
Classification: Components
Component: Embedding: APIs (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla16
Assigned To: David Zbarsky (:dzbarsky)
: Myk Melez [:myk] [@mykmelez]
Depends on:
  Show dependency treegraph
Reported: 2012-06-08 00:40 PDT by David Zbarsky (:dzbarsky)
Modified: 2012-06-09 19:48 PDT (History)
3 users (show)
dzbarsky: in‑testsuite-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch (14.47 KB, patch)
2012-06-08 00:40 PDT, David Zbarsky (:dzbarsky)
jst: review+
Details | Diff | Splinter Review

Description User image David Zbarsky (:dzbarsky) 2012-06-08 00:40:46 PDT
Created attachment 631295 [details] [diff] [review]
Comment 1 User image Johnny Stenback (:jst, 2012-06-09 00:59:21 PDT
Comment on attachment 631295 [details] [diff] [review]

-class nsSiteWindow2 : public nsIEmbeddingSiteWindow2
+class nsSiteWindow2 : public nsIEmbeddingSiteWindow

Any reason not to drop the '2' in the class name here as well?

Comment 2 User image David Zbarsky (:dzbarsky) 2012-06-09 15:20:50 PDT
I changed the class name.

Note You need to log in before you can comment on or make changes to this bug.