Fix range comparison. Get space to accept keyword work.

Fix range comparison. Get space to accept keyword work.

 I overlooked this placeholder code to fix.

BUG=none
TEST=enabled AcceptKeywordBySpaceTest for views-implementation

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72185

[oshima, 2011-01-21 19:08:20]

red trybots are update failures. I kicked off new jobs with newer revistion.

[Peter Kasting, 2011-01-21 19:10:10]

http://codereview.chromium.org/6266015/diff/2001/views/controls/textfield/tex...
File views/controls/textfield/textfield.h (right):

http://codereview.chromium.org/6266015/diff/2001/views/controls/textfield/tex...
views/controls/textfield/textfield.h:72: // Returns true if the selected text
will be the same.
I think the functionality here is confusing.  I'd prefer to leave Equals() and
add the empty range checks to the caller.

[oshima, 2011-01-21 20:05:01]

http://codereview.chromium.org/6266015/diff/2001/views/controls/textfield/tex...
File views/controls/textfield/textfield.h (right):

http://codereview.chromium.org/6266015/diff/2001/views/controls/textfield/tex...
views/controls/textfield/textfield.h:72: // Returns true if the selected text
will be the same.
On 2011/01/21 19:10:10, Peter Kasting wrote:
> I think the functionality here is confusing.  I'd prefer to leave Equals() and
> add the empty range checks to the caller.
moved empty check to caller, but i prefer to keep the
non directional euqal check here.

[Peter Kasting, 2011-01-21 20:08:49]

OK