Commit ff201078 authored by Clark DuVall's avatar Clark DuVall Committed by Commit Bot

[WebLayer] Add protected constructor to CookieManager for mocking

This matches what we do with various other classes like Profile to allow
the client to mock for tests.

Bug: 1054160
Change-Id: I9dc34dbf5309ea071aeeb1d0fd2081a171c82085
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2133877
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Auto-Submit: Clark DuVall <cduvall@chromium.org>
Reviewed-by: default avatarScott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#755846}
parent 80f1201b
...@@ -34,7 +34,12 @@ public class CookieManager { ...@@ -34,7 +34,12 @@ public class CookieManager {
} }
} }
CookieManager(ICookieManager impl) { // Constructor for test mocking.
protected CookieManager() {
mImpl = null;
}
private CookieManager(ICookieManager impl) {
mImpl = impl; mImpl = impl;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment