diff options
| author | fdegans <fdegans@chromium.org> | 2015-03-19 05:52:22 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-03-19 12:52:59 +0000 |
| commit | 6ce28f57d686337e99099a7a97354f45d22ca2f6 (patch) | |
| tree | b55f7d3dc90dd3cd25c4de1181e714b4a6de37f1 /content/browser/web_contents/web_contents_impl_unittest.cc | |
| parent | b0a90f4b6c81fc65dd0c57ebc00b0ba6fe784b22 (diff) | |
| download | chromium_src-6ce28f57d686337e99099a7a97354f45d22ca2f6.zip chromium_src-6ce28f57d686337e99099a7a97354f45d22ca2f6.tar.gz chromium_src-6ce28f57d686337e99099a7a97354f45d22ca2f6.tar.bz2 | |
Remove RenderViewHost parameter from WebContentsObserver::Did{Start|Stop}Loading.
This parameter is unused in almost every class derived from WebContentsObserver.
This will make refactoring the OnDid{Start|Stop}Loading IPCs from
WebContentsImpl to RenderFrameHostImpl simpler.
BUG=429399,450799
Review URL: https://codereview.chromium.org/1008913002
Cr-Commit-Position: refs/heads/master@{#321341}
Diffstat (limited to 'content/browser/web_contents/web_contents_impl_unittest.cc')
| -rw-r--r-- | content/browser/web_contents/web_contents_impl_unittest.cc | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/content/browser/web_contents/web_contents_impl_unittest.cc b/content/browser/web_contents/web_contents_impl_unittest.cc index d56e218..67e8e40 100644 --- a/content/browser/web_contents/web_contents_impl_unittest.cc +++ b/content/browser/web_contents/web_contents_impl_unittest.cc @@ -2765,12 +2765,8 @@ class LoadingWebContentsObserver : public WebContentsObserver { } ~LoadingWebContentsObserver() override {} - void DidStartLoading(RenderViewHost* rvh) override { - is_loading_ = true; - } - void DidStopLoading(RenderViewHost* rvh) override { - is_loading_ = false; - } + void DidStartLoading() override { is_loading_ = true; } + void DidStopLoading() override { is_loading_ = false; } bool is_loading() const { return is_loading_; } |