diff options
| author | danakj <danakj@chromium.org> | 2015-03-09 10:45:39 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-03-09 17:46:34 +0000 |
| commit | 4ef352de3f24142d38b7d11fe3f9f637b43ee834 (patch) | |
| tree | 7f6986711ebbd45505f1c5f8b4e4e2f383f4ff69 /ppapi | |
| parent | d415c687891e096b3e2d10e656842d59a86a51ea (diff) | |
| download | chromium_src-4ef352de3f24142d38b7d11fe3f9f637b43ee834.zip chromium_src-4ef352de3f24142d38b7d11fe3f9f637b43ee834.tar.gz chromium_src-4ef352de3f24142d38b7d11fe3f9f637b43ee834.tar.bz2 | |
base: Stop passing a non-const ref to ObserverListBase::Iterator.
Pass a pointer instead, as per the style guide. Also makes the
constructor explicit (all callsites already are explicit).
BUG=464816
TBR=blundell@chromium.org,agl,dmichael,garykac
Review URL: https://codereview.chromium.org/982413002
Cr-Commit-Position: refs/heads/master@{#319674}
Diffstat (limited to 'ppapi')
| -rw-r--r-- | ppapi/proxy/ppapi_proxy_test.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/ppapi/proxy/ppapi_proxy_test.cc b/ppapi/proxy/ppapi_proxy_test.cc index 5a7867d..14362d9 100644 --- a/ppapi/proxy/ppapi_proxy_test.cc +++ b/ppapi/proxy/ppapi_proxy_test.cc @@ -65,8 +65,7 @@ PPB_Proxy_Private ppb_proxy_private = { ObserverList<ProxyTestHarnessBase> get_interface_handlers_; const void* MockGetInterface(const char* name) { - ObserverList<ProxyTestHarnessBase>::Iterator it = - get_interface_handlers_; + ObserverList<ProxyTestHarnessBase>::Iterator it(&get_interface_handlers_); while (ProxyTestHarnessBase* observer = it.GetNext()) { const void* interface = observer->GetInterface(name); if (interface) |