Network Setup from Shark to Remora.

chrome/browser/chromeos/login/helper.h

 // Copyright (c) 2012 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.
 //
 // This file contains helper functions used by Chromium OS login.
 
 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_HELPER_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_HELPER_H_
 
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "base/strings/string16.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/views/view.h"
 
 class GURL;
 
 namespace gfx {
 class Rect;
 class Size;
 }  // namespace gfx
 
 namespace content {
 class StoragePartition;
-}
+}  // namespace content

[achuithb, 2015/11/05 23:26:12]

I think we don't do this when the namespace is small - it's obvious what the
closing brace is for, and the comment just adds clutter.

I would revert the changes in this file.

[stevenjb, 2015/11/05 23:48:25]

+1

[xdai1, 2015/11/06 23:23:02]

Done.

 
 namespace net {
 class URLRequestContextGetter;
-}
+}  // namespace net
 
 namespace chromeos {
 
 // Returns bounds of the screen to use for login wizard.
 // The rect is centered within the default monitor and sized accordingly if
 // |size| is not empty. Otherwise the whole monitor is occupied.
 gfx::Rect CalculateScreenBounds(const gfx::Size& size);
 
 // Returns the size of user image required for proper display under current DPI.
 int GetCurrentUserImageSize();
@@ skipping 53 matching lines @@
 // Returns the request context that contains sign-in cookies. For old iframe
 // based flow, the context of the sign-in profile is returned. For webview based
 // flow, the context of the sign-in webview storage partition is returned.
 net::URLRequestContextGetter* GetSigninContext();
 
 }  // namespace login
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_HELPER_H_