Make comments in byte_stream.cc gender neutral.

content/browser/byte_stream.cc

Index: content/browser/byte_stream.cc
diff --git a/content/browser/byte_stream.cc b/content/browser/byte_stream.cc
index 1c70739a9f4171f248d87f1b4d0c322f7ad0fd39..cdea993516ee29a1082c775549f8466476cf650f 100644
--- a/content/browser/byte_stream.cc
+++ b/content/browser/byte_stream.cc
@@ -22,15 +22,15 @@ ContentVector;
 
 class ByteStreamReaderImpl;
 
-// A poor man's weak pointer; a RefCountedThreadSafe boolean that can be
-// cleared in an object destructor and accessed to check for object
-// existence.  We can't use weak pointers because they're tightly tied to
-// threads rather than task runners.
+// A makeshift weak pointer; a RefCountedThreadSafe boolean that can be cleared
+// in an object destructor and accessed to check for object existence. We can't
+// use weak pointers because they're tightly tied to threads rather than task
+// runners.
 // TODO(rdsmith): A better solution would be extending weak pointers
 // to support SequencedTaskRunners.
 struct LifetimeFlag : public base::RefCountedThreadSafe<LifetimeFlag> {
  public:
-  LifetimeFlag() : is_alive(true) { }
+  LifetimeFlag() : is_alive(true) {}

[Matt Giuca, 2016/11/28 00:51:34]

nit: There's some more unnecessary formatting changes here. Put back the space
in between these braces.

[meredithl, 2016/11/28 01:13:18]

My bad, I accidentally ran clang in sublime over that part as well.

[meredithl, 2016/11/28 01:13:18]

Done.

   bool is_alive;
 
  protected: