Change-Id: Id12dbb9219a9581dd52d1f820c47f3da47b62187

Change-Id: I62aa8ca0f07dce9a820527ac1aef9c44cd70223a

Change-Id: I1468fc94aa0c709528a767b96bc26eefdb7ed08f

Change-Id: Ibc091cefab7fb0b628db0896af077a03fcda4094

Change-Id: Iea2124d0b485e051df1136f63ba4113b2dbc6f84

Change-Id: Ib0b9b17010f7c1b0814b48f6fb0144e5296418df

Change-Id: I8be9141b9653e73ebd23a5a3d810f240c376f97e

Change-Id: I616c8c28255e0d90ae90033a128bd34d7570530c

Change-Id: If73bb41bfa805e22609748f25971724b4778edb3

Change-Id: I667852d1c854259ed698541b868a49322c6fbe64

Change-Id: If1926c8ecc94c3b9786a3f255371b440c482a155

Change-Id: I9a1d47202e2794461f6ec44f3e72ee1dd2fde09d

Change-Id: I364c5eb176d5003deb1938810cccf4f2aaedbd59

Now its ok to reference / un-reference ourself during construction.

Change-Id: Ib3b302719c7d40d7a42272f93db5e4c437859cad

Document that in README.lifecycle; the problem is that our constructors
currently take and release references left/right on the object being
created, which ... means we need an initial reference.

Change-Id: I5de952b73ac67888c3fbb150d4a7cde2a7bc9abf

Change-Id: I4a28ba4740e9c84666cd85c046598862f20a7a96

This pointer is used to pass extra information through dialog
constructors, and (as such) if implemented with a VclPtr it causes
us to take and then release a reference on our in-construction
object, before it can return it's 'this' into the safety of its
calling VclPtr<> constructor; not good. cf.
Dialog::doDeferredInit vs. SystemWindow::loadUI.

Change-Id: Idcab40cedcdebed560077cfaa1a14395e6e01cd6

Change-Id: I33ec0318a257929583aa4bb0d107f16d1f8edbcd

Change-Id: Ia7085cf98141fdbd50cf9cc9ed45e2e0c09e448e

Change-Id: Ie6f135d9e91d0831717c2a0726161825d0a06665

also audit commit: "start wrapping OutputDevice in VclPtr"

Change-Id: I6e018d7c3e3c8f941c184ad1aaa8593f7750c620

Change-Id: I0e9460cb33b7cb5da9ddb950ff27bac8cbf7fed8

No need to pay a per-instance cost for this, also avoid a self
reference for every Window sub-class.

Change-Id: I040a0ede9481229e43a39bc23297969e90b95f5e

Change-Id: Iffdc9f73520a97ccc284ecba1b2468dc229506c1

Change-Id: I2437b4d3c532be6cbb31dccfa9e267498111c183

Emitting events acquires and passes references conservatively, and the
VCLEVENT_OBJECT_DYING event hit this hard.

Change-Id: If86991ead63b05425a237bf213583c020ba4e582

Change-Id: I094163c894ff9accf70050d5a4fab83e905a282b

Change-Id: I5fa19ec6161ab97eb7df8b52a268917f41ae2205

Change-Id: I25346435d2129086281783636624b1bb430d0096

Change-Id: I2b154e0ed9eee0a45900ada7f805a1d444a31bc0

Change-Id: Ic5bc5c3325b460cba0d95484454c9bb826684890

Change-Id: I1c8ded2e5c5e8072111d3e1480ba7086db135810

Change-Id: I954c6b0db3ae69146763fa6c54e2d8015b00ee4e

Change-Id: I2149511f958ba75e81dc41b10b01eb9d19610037

Without this, assigning to a ScopedVclPtr instance thus:

    pScopedVclPtr = new Foo();

constructed a new intermediate ScopedVCLPtr, used a default assignment
operator, unhelpfully disposing the new Foo before it could make it to
pScopedVclPtr => add operator, and hide problematic constructors.

Change-Id: Icc0da962938bf115eac0c24a6a76cfeb66ddf23a

Change-Id: Iebdf2af3866d884289f6913646106fe3f2d4dde5

Change-Id: I2f0d8a4fe426d8ee1ac55a05ae8dd0b44f9aab2b

Change-Id: Ia742c47399231bc5914b6586132ad3daf694fdb0

Change-Id: I2353d8e9f2b30bf00cf5229613793d9605854b76