Fixed nits from move of web_intent_service_data.* to webkit/glue

webkit/glue/web_intent_service_data.cc

 // Copyright (c) 2011 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 <ostream>
 
 #include "base/utf_string_conversions.h"
 #include "webkit/glue/web_intent_service_data.h"
 
 WebIntentServiceData::WebIntentServiceData()
     : disposition(WebIntentServiceData::DISPOSITION_WINDOW) {
 }
 
 WebIntentServiceData::~WebIntentServiceData() {}
 
 bool WebIntentServiceData::operator==(const WebIntentServiceData& other) const {
-  return (service_url == other.service_url &&
-          action == other.action &&
-          type == other.type &&
-          title == other.title &&
-          disposition == other.disposition);
+  return service_url == other.service_url &&
+         action == other.action &&

[tony, 2011/09/23 22:39:13]

Nit: These should probably be indented 4 spaces from the return (6 total from
the start of the file).

[groby-ooo-7-16, 2011/09/23 22:48:54]

According to style guideline, it's only "indent 4" if the first argument doesn't
fit on the current line.

On 2011/09/23 22:39:13, tony wrote:

+         type == other.type &&
+         title == other.title &&
+         disposition == other.disposition;
 }
 
 std::ostream& operator<<(::std::ostream& os,
                          const WebIntentServiceData& intent) {
   return os <<
          "{" << intent.service_url <<
          ", " << UTF16ToUTF8(intent.action) <<
          ", " << UTF16ToUTF8(intent.type) <<
          ", " << UTF16ToUTF8(intent.title) <<
          ", " << intent.disposition <<
          "}";
 }