Fix some grammatical errors in comments in net/

Fix some grammatical errors in comments in net/


BUG=None

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

[gavinp, 2012-03-28 23:36:16]

wdyt?

[mmenke, 2012-03-28 23:54:37]

LGTM.

https://chromiumcodereview.appspot.com/9909004/diff/1/net/cookies/cookie_mons...
File net/cookies/cookie_monster.cc (right):

https://chromiumcodereview.appspot.com/9909004/diff/1/net/cookies/cookie_mons...
net/cookies/cookie_monster.cc:2670: // The Mozilla code broke it into 3 cases,
if its strings lengths
"its strings lengths" should presumably be "its strings' lengths", though I'd go
with "the length of its strings".  Too many s's scare me.

[gavinp, 2012-03-30 01:29:31]

On 2012/03/28 23:54:37, Matt Menke wrote:
> LGTM.
> 
>
https://chromiumcodereview.appspot.com/9909004/diff/1/net/cookies/cookie_mons...
> File net/cookies/cookie_monster.cc (right):
> 
>
https://chromiumcodereview.appspot.com/9909004/diff/1/net/cookies/cookie_mons...
> net/cookies/cookie_monster.cc:2670: // The Mozilla code broke it into 3 cases,
> if its strings lengths
> "its strings lengths" should presumably be "its strings' lengths", though I'd
go
> with "the length of its strings".  Too many s's scare me.

I agree.  Fixed in this latest upload; I also moved the vulgar 3 to "three."

My plan right now is to land all of my its CLs via dcommit early Tuesday
morning.

[mmenke, 2012-03-30 01:39:57]

Tuesday?  Do you think we'll have that one comment making sense that soon?  :)

http://codereview.chromium.org/9909004/diff/4001/net/cookies/cookie_monster.cc
File net/cookies/cookie_monster.cc (right):

http://codereview.chromium.org/9909004/diff/4001/net/cookies/cookie_monster.c...
net/cookies/cookie_monster.cc:2671: // are less than, equal to, or greater. I
think the approach below is simpler.
This still doesn't make any sense.  I think this actually means the length of
|url_path| is less than, equal to, or greater than |path_|.  Two lengths of two
strings can't be each be less than the other...  Assuming string lengths are
determinate.

[gavinp, 2012-03-30 01:53:50]

On 2012/03/30 01:39:57, Matt Menke wrote:
> Tuesday?  Do you think we'll have that one comment making sense that soon?  :)
> 
> http://codereview.chromium.org/9909004/diff/4001/net/cookies/cookie_monster.cc
> File net/cookies/cookie_monster.cc (right):
> 
>
http://codereview.chromium.org/9909004/diff/4001/net/cookies/cookie_monster.c...
> net/cookies/cookie_monster.cc:2671: // are less than, equal to, or greater. I
> think the approach below is simpler.
> This still doesn't make any sense.  I think this actually means the length of
> |url_path| is less than, equal to, or greater than |path_|.  Two lengths of
two
> strings can't be each be less than the other...  Assuming string lengths are
> determinate.

You're, err, right.  How's this?

I'm just saying Tuesday am because I have thirty-three of these, and I want to
land them all via dcommit before anyone gets in.  Mondays I can't get to the
office before 9:45am, so that clearly won't work.  Friday I can't be in before
8:30am, also a loss.  But Tuesday?  I can be in pretty early then.

But, yes, we should have had a few chances to rewrite this (completely
unintelligible) comment by then.

[mmenke, 2012-03-30 01:56:44]

LGTM.