Initial Implementation of Iframe Attribute for Feature Policy (Part 1)

third_party/WebKit/Source/core/html/HTMLIFrameElementPermissions.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSDstyle license that can be
 // found in the LICENSE file.
 
 #include "core/html/HTMLIFrameElementPermissions.h"
 
 #include "core/html/HTMLIFrameElement.h"
 #include "wtf/HashMap.h"
 #include "wtf/text/StringBuilder.h"
 
@@ skipping 45 matching lines @@
   Vector<blink::mojom::blink::PermissionName> permissions;
   unsigned numTokenErrors = 0;
   StringBuilder tokenErrors;
   const SpaceSplitString& tokens = this->tokens();
 
   for (size_t i = 0; i < tokens.size(); ++i) {
     blink::mojom::blink::PermissionName type;
     if (getPermissionType(tokens[i], &type)) {
       permissions.push_back(type);
     } else {
-      if (numTokenErrors)
-        tokenErrors.append(", '");
-      else
-        tokenErrors.append('\'');
+      tokenErrors.append(tokenErrors.isEmpty() ? "'" : ", '");
       tokenErrors.append(tokens[i]);
-      tokenErrors.append('\'');
+      tokenErrors.append("'");
       ++numTokenErrors;
     }
   }
 
   if (numTokenErrors) {
-    if (numTokenErrors > 1)
-      tokenErrors.append(" are invalid permissions flags.");
-    else
-      tokenErrors.append(" is an invalid permissions flag.");
+    tokenErrors.append(numTokenErrors > 1 ? " are invalid permissions flags."
+                                          : " is an invalid permissions flag.");
     invalidTokensErrorMessage = tokenErrors.toString();
   }
 
   return permissions;
 }
 
 bool HTMLIFrameElementPermissions::validateTokenValue(
     const AtomicString& tokenValue,
     ExceptionState&) const {
   mojom::blink::PermissionName unused;
   return getPermissionType(tokenValue, &unused);
 }
 
 void HTMLIFrameElementPermissions::valueWasSet() {
   if (m_element)
     m_element->permissionsValueWasSet();
 }
 
 }  // namespace blink