Disallow setting invalid values for registered properties via CSSOM

third_party/WebKit/Source/core/dom/Document.h

Index: third_party/WebKit/Source/core/dom/Document.h
diff --git a/third_party/WebKit/Source/core/dom/Document.h b/third_party/WebKit/Source/core/dom/Document.h
index d7adbbbb1db85b666a3afa85f0f9c5fc3fc1163a..a3ac691deb3bb8917b67a2da4aff2b6a5a51d1dd 100644
--- a/third_party/WebKit/Source/core/dom/Document.h
+++ b/third_party/WebKit/Source/core/dom/Document.h
@@ -1295,6 +1295,7 @@ class CORE_EXPORT Document : public ContainerNode,
   void maybeRecordLoadReason(WouldLoadReason);
   WouldLoadReason wouldLoadReason() { return m_wouldLoadReason; }
 
+  const PropertyRegistry* propertyRegistry() const;
   PropertyRegistry* propertyRegistry();
 
   // Indicates whether the user has interacted with this particular Document.
@@ -1664,7 +1665,7 @@ class CORE_EXPORT Document : public ContainerNode,
 
   WouldLoadReason m_wouldLoadReason;
 
-  Member<PropertyRegistry> m_propertyRegistry;
+  mutable Member<PropertyRegistry> m_propertyRegistry;

[alancutter (OOO until 2018), 2017/01/04 01:03:42]

Why is this mutable?

[Timothy Loh, 2017/01/04 02:00:59]

Oops, I was going to copy the logic in non-const propertyRegistry to the const
one, but then I just const_cast'd and called the non-const. Will remove.

[alancutter (OOO until 2018), 2017/01/04 02:22:10]

I don't think mutable is required for your original plan anyway. (:

 
   unsigned m_passwordCount;