Don't sync typed URL modification that doesn't have valid visits, which could

Don't sync typed URL modification that doesn't have valid visits, which could
happen if the URL is bookmarked and all its visits are deleted.

BUG=377711
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273925

[rlarocque, 2014-05-27 21:56:44]

I think that looks OK, but I'd feel better about it if someone who knows more
about type URLs would take a look.  +Nicolas for further review.

Also, should there be tests for this?

[haitaol1, 2014-05-27 23:32:51]

On 2014/05/27 21:56:44, rlarocque wrote:
> I think that looks OK, but I'd feel better about it if someone who knows more
> about type URLs would take a look.  +Nicolas for further review.
> 
> Also, should there be tests for this?

Added test.

[haitaol1, 2014-05-29 17:29:46]

ping

[rlarocque, 2014-05-30 17:32:33]

Nicolas claims that he doesn't know this code very well, either.  He's OOO
today, so I guess this falls back to me.

The change looks sane, and it couldn't be any much than the existing code. 
LGTM.

Sorry for the delays.

[haitaol1, 2014-05-30 17:38:58]

The CQ bit was checked by haitaol@chromium.org

[commit-bot: I haz the power, 2014-05-30 17:41:32]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/haitaol@chromium.org/300843004/20001

[commit-bot: I haz the power, 2014-05-30 23:10:06]

Change committed as 273925