| commit | b14e5789d4d1f0778ef3252fa815c13caefadebb | [log] [tgz] |
|---|---|---|
| author | Peter Kasting <[email protected]> | Mon Jan 06 23:55:04 2020 |
| committer | Commit Bot <[email protected]> | Mon Jan 06 23:55:04 2020 |
| tree | 2e921869797849891c2ea141c28f6a7954484baf | |
| parent | de165b667d781d28f311981108270f61a59e7511 [diff] [blame] |
Remove color arg from DidChangeThemeColor(). This isn't actually used except by tests, which can be rewritten to not need it. Bug: none Change-Id: Ia9f5957f161aea8bb57625729a14e1d214660ec1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1982623 Reviewed-by: Alan Cutter <[email protected]> Reviewed-by: Avi Drissman <[email protected]> Reviewed-by: Yaron Friedman <[email protected]> Commit-Queue: Peter Kasting <[email protected]> Cr-Commit-Position: refs/heads/master@{#728698}
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc index 3eac125..7074ec7 100644 --- a/content/browser/web_contents/web_contents_impl.cc +++ b/content/browser/web_contents/web_contents_impl.cc
@@ -4637,7 +4637,7 @@ if (source->did_first_visually_non_empty_paint() && last_sent_theme_color_ != source->theme_color()) { for (auto& observer : observers_) - observer.DidChangeThemeColor(source->theme_color()); + observer.DidChangeThemeColor(); last_sent_theme_color_ = source->theme_color(); } } @@ -5139,7 +5139,7 @@ if (source->theme_color() != last_sent_theme_color_) { // Theme color should have updated by now if there was one. for (auto& observer : observers_) - observer.DidChangeThemeColor(source->theme_color()); + observer.DidChangeThemeColor(); last_sent_theme_color_ = source->theme_color(); } }