Now mark delete notifications for archived URLs specially

Now mark delete notifications for archived URLs specially

BUG=124508
TEST=existing tests suffice

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

[Andrew T Wilson (Slow), 2012-05-04 23:32:34]

PTAL. This is the first step towards letting Sync ignore archived URLs (we don't
want to sync archives as deletions to other clients, because otherwise clock
skew on one client could result in premature deletion of entries from other
clients).

[sky, 2012-05-04 23:43:39]

I'm passing this one Brett's way.

[brettw, 2012-05-08 19:56:56]

lgtm

http://codereview.chromium.org/10380015/diff/1/chrome/browser/history/expire_...
File chrome/browser/history/expire_history_backend.h (right):

http://codereview.chromium.org/10380015/diff/1/chrome/browser/history/expire_...
chrome/browser/history/expire_history_backend.h:201: USER_INITIATED,
Can you prefix these with something. Like DELETION_USER_INITIATED?

[commit-bot: I haz the power, 2012-05-08 21:26:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/atwilson@chromium.org/10380015/9001

[commit-bot: I haz the power, 2012-05-09 00:25:29]

Try job failure for 10380015-9001 (retry) (retry) (retry) on win_rel for step
"update".
It's a second try, previously, step "runhooks" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

Step "update" is always a major failure.
Look at the try server FAQ for more details.