Remove CertErrors::Add(); use CertErrors::AddError() instead.

net/cert/internal/cert_errors.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // ----------------------------
 // Overview of error design
 // ----------------------------
 //
 // Certificate path validation/parsing may emit a sequence of
 // errors/warnings/context. These are represented by a tree of CertErrorNodes.
 // Each node is comprised of:
 //
 //   * A unique identifier.
 //
 //     This serves similarly to an error code, and is useful for querying if a
 //     particular error occurred.
 //
 //   * [optional] A parameters object.
 //
 //     Nodes may attach a heap-allocated subclass of CertErrorParams, to carry
 //     extra information that is useful when reporting the error. For instance
 //     a parsing error may want to describe where in the DER the failure
 //     happened, or what the unexpected value was.
 //
 //   * [optional] Child nodes.
 //
-//     Error nodes are arranged in a tree. The parent/child hiearchy is used to
+//     Error nodes are arranged in a tree. The parent/child hierarchy is used to
 //     group errors that share some common state.
 //     For instance during path processing it is useful to group the
 //     errors/warnings that happened while processing certificate "i" as
 //     children of a shared "context" node. The context node in this case
 //     doesn't describe a particular error, but rather some shared event and
 //     its parameters.
 //
 // ----------------------------
 // Using errors in other APIs
 // ----------------------------
@@ skipping 80 matching lines @@
  public:
   CertErrors();
   ~CertErrors();
 
   // Adds a node to the current insertion point in the error tree. |params| may
   // be null.
   void Add(CertErrorNodeType node_type,
            CertErrorId id,
            std::unique_ptr<CertErrorParams> params);
 
-  // TODO(crbug.com/634443): Eliminate this and use AddError() instead (which
-  // is less ambiguous).
-  void Add(CertErrorId id);
-
   void AddError(CertErrorId id, std::unique_ptr<CertErrorParams> params);
   void AddError(CertErrorId id);
 
   void AddWarning(CertErrorId id, std::unique_ptr<CertErrorParams> params);
   void AddWarning(CertErrorId id);
 
   // Returns true if the tree is empty. Note that emptiness of the error tree
   // is NOT equivalent to success for some call, and vice versa. (For instance
   // consumers may forget to emit errors on failures, or some errors may be
   // non-fatal warnings).
@@ skipping 17 matching lines @@
   // The top-most CertErrorScoper that is currently in scope (and which affects
   // the parent node for newly added errors).
   CertErrorScoper* current_scoper_ = nullptr;
 
   DISALLOW_COPY_AND_ASSIGN(CertErrors);
 };
 
 }  // namespace net
 
 #endif  // NET_CERT_INTERNAL_CERT_ERRORS_H_