• rbpotter's avatar
    History Web UI: Cleanup removeVisits chrome.send call · d0da7919
    rbpotter authored
    - Move wrapping logic around chrome.send('removeVisits') into
    history_list.js instead of browser_service.js
    - Convert this call to use cr.sendWithPromise instead of calling JS
    directly from C++ after deletion succeeds/fails
    - Delete browser service test, since the logic it is testing is no
    longer in the browser service. The test is replaced by adding checks
    for the arguments passed to removeVisits() in the history list tests.
    
    Bug: 1022212
    Change-Id: I7290d97472b34a0a233ac7921c18c2a298481a3b
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1947956Reviewed-by: default avatarDemetrios Papadopoulos <dpapad@chromium.org>
    Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#721378}
    d0da7919
history_toolbar.js 2.65 KB