1425437 - Improve view batching calculations when dropping/pasting to tags

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, enhancement)

Description

[Mark Banner (:standard8)]

For tagging operations (move/paste etc), we're currently deciding to batch the view updates based on the amount of places transactions created. 

Since this is only ever a single transaction for tag operations, we'll never batch them currently. We should base it on the actual amount of urls that we're tagging.

It is a small patch, but split out from bug 1404909, as it has been in my queue for a while, and we might as well get it landed.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1425437 - Improve view batching when dropping/pasting to tags, use the actual bookmarks count, not the count of transactions.

Review commit: https://reviewboard.mozilla.org/r/207754/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/207754/

Comment 2

[Marco Bonardo [:mak]]

Comment on attachment 8937018 [details]
Bug 1425437 - Improve view batching when dropping/pasting to tags, use the actual bookmarks count, not the count of transactions.

https://reviewboard.mozilla.org/r/207754/#review213646

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6ab3ab7f0aa9
Improve view batching when dropping/pasting to tags, use the actual bookmarks count, not the count of transactions. r=mak

Comment 4

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/6ab3ab7f0aa9