Push LayoutState from RenderSVGText

Source/core/rendering/style/DataRef.h

 /*
  * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
  *           (C) 2000 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2003, 2005, 2008 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 44 matching lines @@
         return m_data == o.m_data || *m_data == *o.m_data;
     }
 
     bool operator!=(const DataRef<T>& o) const
     {
         ASSERT(m_data);
         ASSERT(o.m_data);
         return m_data != o.m_data && *m_data != *o.m_data;
     }
 
+    void operator=(std::nullptr_t) { m_data = nullptr; }

[leviw_travelin_and_unemployed, 2014/11/10 21:07:23]

This makes a lot more sense as a solution than your previous patch. This part is
the only part I find surprising... Do we really not reset DataRefs elsewhere?
Ideally, we'd just use access() and copy in a default StyleMultiColData (which
could potentially avoid some RefPtr churn). Seems like a lot of yak shaving,
though. WDYT?

[leviw_travelin_and_unemployed, 2014/11/10 21:07:23]

This makes a lot more sense as a solution than your previous patch. This part is
the only part I find surprising... Do we really not reset DataRefs elsewhere?
Ideally, we'd just use access() and copy in a default StyleMultiColData (which
could potentially avoid some RefPtr churn). That may be more yak shaving than
this patch is worth, though.

 private:
     RefPtr<T> m_data;
 };
 
 } // namespace blink
 
 #endif // DataRef_h