Make comments in byte_stream.cc gender neutral.

content/browser/byte_stream.cc

 // 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.
 
 #include "content/browser/byte_stream.h"
 
 #include <deque>
 #include <set>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/sequenced_task_runner.h"
 
 namespace content {
 namespace {
 
 typedef std::deque<std::pair<scoped_refptr<net::IOBuffer>, size_t> >
 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.
-// TODO(rdsmith): A better solution would be extending weak pointers
-// to support SequencedTaskRunners.
+// This is a RefCountedThreadSafe boolean that can be cleared in an object

[Matt Giuca, 2016/11/25 00:35:42]

drive-by: I don't think we should lose this sentence. The idea that it is a
"poor man's weak pointer" is important to give the reader context about what
type of thing this class is trying to emulate, poorly.

How about "a simplistic weak pointer" or "a poor-person's weak pointer"? (Also
see email I just sent about "poor-man" in general.)

[meredithl, 2016/11/27 23:49:21]

Done.

+// 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) { }
   bool is_alive;
 
  protected:
   friend class base::RefCountedThreadSafe<LifetimeFlag>;
   virtual ~LifetimeFlag() { }
 
  private:
@@ skipping 415 matching lines @@
   ByteStreamReaderImpl* out = new ByteStreamReaderImpl(
       output_task_runner, output_flag, buffer_size);
 
   in->SetPeer(out, output_task_runner, output_flag);
   out->SetPeer(in, input_task_runner, input_flag);
   input->reset(in);
   output->reset(out);
 }
 
 }  // namespace content