[WebSocketExtensionParser] Have Consume.* methods return bool instead of using has_error()

net/server/web_socket_encoder.cc

 // Copyright 2014 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.
 
 #include "net/server/web_socket_encoder.h"
 
 #include "base/logging.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "net/base/io_buffer.h"
@@ skipping 233 matching lines @@
   *has_client_window_bits = false;
   *client_window_bits = 15;
   *server_window_bits = 15;
   *client_no_context_takeover = false;
   *server_no_context_takeover = false;
 
   if (header_value.empty())
     return;
 
   WebSocketExtensionParser parser;
-  parser.Parse(header_value);
-  if (parser.has_error())
+  if (!parser.Parse(header_value))
     return;
   const std::vector<WebSocketExtension>& extensions = parser.extensions();
   // TODO(tyoshino): Fail if this method is used for parsing a response and
   // there are multiple permessage-deflate extensions or there are any unknown
   // extensions.
   for (const auto& extension : extensions) {
     if (extension.name() != "permessage-deflate") {
       continue;
     }
 
@@ skipping 111 matching lines @@
     return false;
   scoped_refptr<IOBufferWithSize> buffer =
       deflater_->GetOutput(deflater_->CurrentOutputSize());
   if (!buffer.get())
     return false;
   *output = std::string(buffer->data(), buffer->size());
   return true;
 }
 
 }  // namespace net