i#107771: sw: make SwXReferenceMark a client of its format poolitem

In other words, stop registering at SwDoc's "UnoCallBack"

Change-Id: I9e08966cf8e2d2a373867d81549c8887f73993c8

sw/inc/fmtrfmrk.hxx

@@ -22,11 +22,15 @@
 #include <rtl/ustring.hxx>
 #include <svl/poolitem.hxx>
 
+#include <calbck.hxx>
+
 class SwTxtRefMark;
 
 // ATT_REFMARK
 
-class SwFmtRefMark : public SfxPoolItem
+class SwFmtRefMark
+    : public SfxPoolItem
+    , public SwModify
 {
     friend class SwTxtRefMark;
     SwTxtRefMark* pTxtAttr;
@@ -44,6 +48,8 @@ public:
     virtual bool            operator==( const SfxPoolItem& ) const SAL_OVERRIDE;
     virtual SfxPoolItem*    Clone( SfxItemPool* pPool = 0 ) const SAL_OVERRIDE;
 
+    void InvalidateRefMark();
+
     const SwTxtRefMark *GetTxtRefMark() const   { return pTxtAttr; }
     SwTxtRefMark *GetTxtRefMark()               { return pTxtAttr; }
 

sw/inc/hintids.hxx

@@ -330,7 +330,6 @@ RES_MSG_BEGIN = RES_FMT_END,
     RES_FINDNEARESTNODE,
     RES_CONTENT_VISIBLE,
     RES_FOOTNOTE_DELETED,
-    RES_REFMARK_DELETED,
     RES_GRAPHIC_SWAPIN,
     RES_FIELD_DELETED,
     RES_NAME_CHANGED,

sw/source/core/attr/calbck.cxx

@@ -174,7 +174,6 @@ void SwModify::NotifyClients( const SfxPoolItem* pOldValue, const SfxPoolItem* p
             break;
 
         case RES_FOOTNOTE_DELETED:
-        case RES_REFMARK_DELETED:
         case RES_FIELD_DELETED:
             bLockClientList = false;
             break;

sw/source/core/docnode/nodes.cxx

@@ -39,6 +39,7 @@
 #include <frame.hxx>
 #include <txtatr.hxx>
 #include <tox.hxx>
+#include <fmtrfmrk.hxx>
 
 #include <docsh.hxx>
 #include <svl/smplhint.hxx>
@@ -303,7 +304,8 @@ void SwNodes::ChgNode( SwNodeIndex& rDelPos, sal_uLong nSz,
                                 break;
 
                             case RES_TXTATR_REFMARK:
-                                nDelMsg = RES_REFMARK_DELETED;
+                                static_cast<SwFmtRefMark&>(pAttr->GetAttr())
+                                    .InvalidateRefMark();
                                 break;
 
                             case RES_TXTATR_META:

sw/source/core/inc/unorefmark.hxx

@@ -55,10 +55,10 @@ private:
 
 public:
 
-    SwXReferenceMark(SwDoc *const pDoc, const SwFmtRefMark *const pMark);
+    SwXReferenceMark(SwDoc *const pDoc, SwFmtRefMark *const pMark);
 
     static SwXReferenceMark *
-        CreateXReferenceMark(SwDoc & rDoc, SwFmtRefMark const& rMarkFmt);
+        CreateXReferenceMark(SwDoc & rDoc, SwFmtRefMark & rMarkFmt);
     /// may return 0
     static SwXReferenceMark *
         GetReferenceMark(SwModify const& rUnoCB, SwFmtRefMark const& rMarkFmt);

sw/source/core/txtnode/atrref.cxx

@@ -17,9 +17,11 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
+#include <fmtrfmrk.hxx>
+
 #include <hintids.hxx>
+#include <hints.hxx>
 #include <txtrfmrk.hxx>
-#include <fmtrfmrk.hxx>
 #include <swfont.hxx>
 
 SwFmtRefMark::~SwFmtRefMark( )
@@ -27,16 +29,18 @@ SwFmtRefMark::~SwFmtRefMark( )
 }
 
 SwFmtRefMark::SwFmtRefMark( const OUString& rName )
-    : SfxPoolItem( RES_TXTATR_REFMARK ),
-    pTxtAttr( 0 ),
-    aRefName( rName )
+    : SfxPoolItem(RES_TXTATR_REFMARK)
+    , SwModify(0)
+    , pTxtAttr(0)
+    , aRefName(rName)
 {
 }
 
 SwFmtRefMark::SwFmtRefMark( const SwFmtRefMark& rAttr )
-    : SfxPoolItem( RES_TXTATR_REFMARK ),
-    pTxtAttr( 0 ),
-    aRefName( rAttr.aRefName )
+    : SfxPoolItem(RES_TXTATR_REFMARK)
+    , SwModify(0)
+    , pTxtAttr(0)
+    , aRefName(rAttr.aRefName)
 {
 }
 
@@ -51,6 +55,13 @@ SfxPoolItem* SwFmtRefMark::Clone( SfxItemPool* ) const
     return new SwFmtRefMark( *this );
 }
 
+void SwFmtRefMark::InvalidateRefMark()
+{
+    SwPtrMsgPoolItem const item(RES_REMOVE_UNO_OBJECT,
+            &static_cast<SwModify&>(*this)); // cast to base class (void*)
+    NotifyClients(&item, &item);
+}
+
 // Attribut fuer Inhalts-/Positions-Referenzen im Text
 
 SwTxtRefMark::SwTxtRefMark( SwFmtRefMark& rAttr,

sw/source/core/txtnode/thints.cxx

@@ -32,6 +32,7 @@
 #include <txtinet.hxx>
 #include <txtflcnt.hxx>
 #include <fmtfld.hxx>
+#include <fmtrfmrk.hxx>
 #include <fmtanchr.hxx>
 #include <fmtinfmt.hxx>
 #include <txtatr.hxx>
@@ -1185,7 +1186,7 @@ void SwTxtNode::DestroyAttr( SwTxtAttr* pAttr )
             break;
 
         case RES_TXTATR_REFMARK:
-            nDelMsg = RES_REFMARK_DELETED;
+            static_cast<SwFmtRefMark&>(pAttr->GetAttr()).InvalidateRefMark();
             break;
 
         case RES_TXTATR_META:

sw/source/core/unocore/unocoll.cxx

@@ -1988,7 +1988,8 @@ SwXReferenceMark* SwXReferenceMarks::GetObject( SwDoc* pDoc, const SwFmtRefMark*
 {
     SolarMutexGuard aGuard;
 
-    return SwXReferenceMark::CreateXReferenceMark(*pDoc, *pMark);
+    return SwXReferenceMark::CreateXReferenceMark(
+                *pDoc, *const_cast<SwFmtRefMark*>(pMark));
 }
 
 void SwUnoCollection::Invalidate()

sw/source/core/unocore/unoportenum.cxx

@@ -491,8 +491,8 @@ lcl_CreateRefMarkPortion(
     const SwTxtAttr & rAttr, const bool bEnd)
 {
     SwDoc* pDoc = pUnoCrsr->GetDoc();
-    const SwFmtRefMark& rRefMark =
-        static_cast<const SwFmtRefMark&>(rAttr.GetAttr());
+    SwFmtRefMark& rRefMark = const_cast<SwFmtRefMark&>(
+            static_cast<const SwFmtRefMark&>(rAttr.GetAttr()));
     Reference<XTextContent> xContent;
     if (!xContent.is())
     {

sw/source/core/unocore/unorefmk.cxx

@@ -53,8 +53,8 @@ public:
     OUString             m_sMarkName;
 
     Impl(   SwXReferenceMark & rThis,
-            SwDoc *const pDoc, SwFmtRefMark const*const pRefMark)
-        : SwClient((pDoc) ? pDoc->GetUnoCallBack() : 0)
+            SwDoc *const pDoc, SwFmtRefMark *const pRefMark)
+        : SwClient(pRefMark)
         , m_rThis(rThis)
         , m_EventListeners(m_Mutex)
         , m_bIsDescriptor(0 == pRefMark)
@@ -96,23 +96,10 @@ void SwXReferenceMark::Impl::Modify( const SfxPoolItem* pOld, const SfxPoolItem
     {
         Invalidate();
     }
-    else if (pOld)
-    {
-        switch (pOld->Which())
-        {
-            case RES_REFMARK_DELETED:
-                if (static_cast<const void*>(m_pMarkFmt) ==
-                        static_cast<const SwPtrMsgPoolItem *>(pOld)->pObject)
-                {
-                    Invalidate();
-                }
-                break;
-        }
-    }
 }
 
 SwXReferenceMark::SwXReferenceMark(
-        SwDoc *const pDoc, SwFmtRefMark const*const pRefMark)
+        SwDoc *const pDoc, SwFmtRefMark *const pRefMark)
     : m_pImpl( new SwXReferenceMark::Impl(*this, pDoc, pRefMark) )
 {
 }
@@ -133,10 +120,10 @@ SwXReferenceMark::GetReferenceMark(
 
 SwXReferenceMark *
 SwXReferenceMark::CreateXReferenceMark(
-        SwDoc & rDoc, SwFmtRefMark const& rMarkFmt)
+        SwDoc & rDoc, SwFmtRefMark & rMarkFmt)
 {
     SwXReferenceMark *const pXMark(
-        GetReferenceMark(*rDoc.GetUnoCallBack(), rMarkFmt) );
+        GetReferenceMark(rMarkFmt, rMarkFmt) );
     return (pXMark)
         ?   pXMark
         :   new SwXReferenceMark(&rDoc, &rMarkFmt);
@@ -266,7 +253,7 @@ void SwXReferenceMark::Impl::InsertRefMark(SwPaM& rPam,
 
     m_pMarkFmt = &pTxtAttr->GetRefMark();
 
-    pDoc2->GetUnoCallBack()->Add(this);
+    const_cast<SwFmtRefMark*>(m_pMarkFmt)->Add(this);
 }
 
 void SAL_CALL