base/i18n: Pass |str| as const reference in BreakIterator::Init().

base/i18n/break_iterator.h

 // 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.
 
 #ifndef BASE_I18N_BREAK_ITERATOR_H_
 #define BASE_I18N_BREAK_ITERATOR_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/string16.h"
@@ skipping 49 matching lines @@
     BREAK_WORD,
     BREAK_LINE,
     // TODO(jshin): Remove this after reviewing call sites.
     // If call sites really need break only on space-like characters
     // implement it separately.
     BREAK_SPACE = BREAK_LINE,
     BREAK_NEWLINE,
   };
 
   // Requires |str| to live as long as the BreakIterator does.
-  BreakIterator(const string16* str, BreakType break_type);
+  BreakIterator(const string16& str, BreakType break_type);
   ~BreakIterator();
 
   // Init() must be called before any of the iterators are valid.
   // Returns false if ICU failed to initialize.
   bool Init();
 
   // Return the current break position within the string,
   // or BreakIterator::npos when done.
   size_t pos() const { return pos_; }
 
@@ skipping 18 matching lines @@
   string16 GetString() const;
 
  private:
   // ICU iterator, avoiding ICU ubrk.h dependence.
   // This is actually an ICU UBreakiterator* type, which turns out to be
   // a typedef for a void* in the ICU headers. Using void* directly prevents
   // callers from needing access to the ICU public headers directory.
   void* iter_;
 
   // The string we're iterating over.
-  const string16* string_;
+  const string16& string_;
 
   // The breaking style (word/space/newline).
   BreakType break_type_;
 
   // Previous and current iterator positions.
   size_t prev_, pos_;
 
   DISALLOW_COPY_AND_ASSIGN(BreakIterator);
 };
 
 }  // namespace i18n
 }  // namespace base
 
 #endif  // BASE_I18N_BREAK_ITERATOR_H_