Increase test coverage for Dice |
||
Issue descriptionwe should make sure we have some more integration tests in addition to the unit tests doesn't need to get merged back..
,
Oct 12
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/df554d02051d43d493d30cdf39fb1ee7566355bf commit df554d02051d43d493d30cdf39fb1ee7566355bf Author: Jochen Eisinger <jochen@chromium.org> Date: Fri Oct 12 07:53:09 2018 Add browser tests for when we show the cookie exception Depending on the sync / sign-in state. BUG= 890925 R=tangltom@chromium.org, msarda@chromium.org Change-Id: I784f13a4981250f851df770d111a13969a6892a6 Reviewed-on: https://chromium-review.googlesource.com/c/1273521 Commit-Queue: Jochen Eisinger <jochen@chromium.org> Reviewed-by: Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#599134} [add] https://crrev.com/df554d02051d43d493d30cdf39fb1ee7566355bf/chrome/browser/browsing_data/counters/browsing_data_counter_utils_browsertest.cc [modify] https://crrev.com/df554d02051d43d493d30cdf39fb1ee7566355bf/chrome/test/BUILD.gn
,
Oct 12
this were the two tests I had in mind, so closing again. |
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Oct 2