Last Comment Bug 762801 - Merge nsIEmbeddingSiteWindow2 into nsIEmbeddingSiteWindow
: Merge nsIEmbeddingSiteWindow2 into nsIEmbeddingSiteWindow
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Embedding: APIs (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla16
Assigned To: David Zbarsky (:dzbarsky)
:
Mentors:
Depends on:
Blocks:
  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:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


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

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

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

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

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

https://hg.mozilla.org/integration/mozilla-inbound/rev/6fa19f2c32a9

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