Bug 1441932 P4 Treat same-value assignments to the LoadInfo reserved and initial ClientInfo values as no-op changes. r=mayhemer

dom/clients/manager/ClientInfo.cpp

@@ -60,6 +60,12 @@ ClientInfo::~ClientInfo()
 {
 }
 
+bool
+ClientInfo::operator==(const ClientInfo& aRight) const
+{
+  return *mData == *aRight.mData;
+}
+
 const nsID&
 ClientInfo::Id() const
 {

dom/clients/manager/ClientInfo.h

@@ -47,6 +47,9 @@ public:
 
   ~ClientInfo();
 
+  bool
+  operator==(const ClientInfo& aRight) const;
+
   // Get the unique identifier chosen at the time of the global's creation.
   const nsID&
   Id() const;

netwerk/base/LoadInfo.cpp

@@ -1317,6 +1317,11 @@ void
 LoadInfo::SetReservedClientInfo(const ClientInfo& aClientInfo)
 {
   MOZ_DIAGNOSTIC_ASSERT(mInitialClientInfo.isNothing());
+  // Treat assignments of the same value as a no-op.  The emplace below
+  // will normally assert when overwriting an existing value.
+  if (mReservedClientInfo.isSome() && mReservedClientInfo.ref() == aClientInfo) {
+    return;
+  }
   mReservedClientInfo.emplace(aClientInfo);
 }
 
@@ -1331,6 +1336,11 @@ LoadInfo::SetInitialClientInfo(const ClientInfo& aClientInfo)
 {
   MOZ_DIAGNOSTIC_ASSERT(!mReservedClientSource);
   MOZ_DIAGNOSTIC_ASSERT(mReservedClientInfo.isNothing());
+  // Treat assignments of the same value as a no-op.  The emplace below
+  // will normally assert when overwriting an existing value.
+  if (mInitialClientInfo.isSome() && mInitialClientInfo.ref() == aClientInfo) {
+    return;
+  }
   mInitialClientInfo.emplace(aClientInfo);
 }