diff options
author | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-01 03:14:09 +0000 |
---|---|---|
committer | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-01 03:14:09 +0000 |
commit | 9c009098a0f38f9a00b8c00de4f32398e4361221 (patch) | |
tree | 783e9bce53169c847ab877319bd7bc106804f197 /chrome/renderer | |
parent | c274c75edef56e91a2104c8ddac8d20b6efeb628 (diff) | |
download | chromium_src-9c009098a0f38f9a00b8c00de4f32398e4361221.zip chromium_src-9c009098a0f38f9a00b8c00de4f32398e4361221.tar.gz chromium_src-9c009098a0f38f9a00b8c00de4f32398e4361221.tar.bz2 |
chrome: Remove the remaining use of ALLOW_THIS_IN_INITIALIZER_LIST.
It's no longer providing value as the MSVC warning is disabled during
compilation. Refer to bug for details.
BUG=234765
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/14712004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@197552 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer')
10 files changed, 11 insertions, 12 deletions
diff --git a/chrome/renderer/chrome_render_process_observer.cc b/chrome/renderer/chrome_render_process_observer.cc index 8b6b7b7..ff24a73 100644 --- a/chrome/renderer/chrome_render_process_observer.cc +++ b/chrome/renderer/chrome_render_process_observer.cc @@ -65,7 +65,7 @@ static const int kCacheStatsDelayMS = 2000; class RendererResourceDelegate : public content::ResourceDispatcherDelegate { public: RendererResourceDelegate() - : ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) { + : weak_factory_(this) { } virtual webkit_glue::ResourceLoaderBridge::Peer* OnRequestComplete( diff --git a/chrome/renderer/extensions/user_script_scheduler.cc b/chrome/renderer/extensions/user_script_scheduler.cc index f194a23..c4802a2 100644 --- a/chrome/renderer/extensions/user_script_scheduler.cc +++ b/chrome/renderer/extensions/user_script_scheduler.cc @@ -41,7 +41,7 @@ namespace extensions { UserScriptScheduler::UserScriptScheduler(WebFrame* frame, Dispatcher* dispatcher) - : ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)), + : weak_factory_(this), frame_(frame), current_location_(UserScript::UNDEFINED), has_run_idle_(false), diff --git a/chrome/renderer/net/net_error_helper.cc b/chrome/renderer/net/net_error_helper.cc index e53a906..18a1cc4 100644 --- a/chrome/renderer/net/net_error_helper.cc +++ b/chrome/renderer/net/net_error_helper.cc @@ -97,9 +97,8 @@ WebKit::WebURLError NetErrorToWebURLError(int net_error) { NetErrorHelper::NetErrorHelper(RenderView* render_view) : RenderViewObserver(render_view), - ALLOW_THIS_IN_INITIALIZER_LIST(tracker_(base::Bind( - &NetErrorHelper::TrackerCallback, - base::Unretained(this)))), + tracker_(base::Bind(&NetErrorHelper::TrackerCallback, + base::Unretained(this))), dns_error_page_state_(NetErrorTracker::DNS_ERROR_PAGE_NONE), updated_error_page_(false), is_failed_post_(false) { diff --git a/chrome/renderer/net/renderer_net_predictor.cc b/chrome/renderer/net/renderer_net_predictor.cc index 746cd22..b9ee8b9 100644 --- a/chrome/renderer/net/renderer_net_predictor.cc +++ b/chrome/renderer/net/renderer_net_predictor.cc @@ -24,7 +24,7 @@ static const size_t kMAX_SUBMISSION_PER_TASK = 30; RendererNetPredictor::RendererNetPredictor() : c_string_queue_(1000), - ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) { + weak_factory_(this) { Reset(); } diff --git a/chrome/renderer/pepper/pepper_flash_renderer_host.cc b/chrome/renderer/pepper/pepper_flash_renderer_host.cc index c76f7c6..768690f 100644 --- a/chrome/renderer/pepper/pepper_flash_renderer_host.cc +++ b/chrome/renderer/pepper/pepper_flash_renderer_host.cc @@ -41,7 +41,7 @@ PepperFlashRendererHost::PepperFlashRendererHost( PP_Instance instance, PP_Resource resource) : ResourceHost(host->GetPpapiHost(), instance, resource), - weak_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)), + weak_factory_(this), host_(host) { } diff --git a/chrome/renderer/safe_browsing/phishing_classifier.cc b/chrome/renderer/safe_browsing/phishing_classifier.cc index 7e9dba1..21b6486 100644 --- a/chrome/renderer/safe_browsing/phishing_classifier.cc +++ b/chrome/renderer/safe_browsing/phishing_classifier.cc @@ -41,7 +41,7 @@ PhishingClassifier::PhishingClassifier(content::RenderView* render_view, : render_view_(render_view), scorer_(NULL), clock_(clock), - ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) { + weak_factory_(this) { Clear(); } diff --git a/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc b/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc index 17ce0e8..feee111 100644 --- a/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc +++ b/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc @@ -103,7 +103,7 @@ PhishingDOMFeatureExtractor::PhishingDOMFeatureExtractor( FeatureExtractorClock* clock) : render_view_(render_view), clock_(clock), - ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) { + weak_factory_(this) { Clear(); } diff --git a/chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc b/chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc index 8d6237d..ea4cbc0 100644 --- a/chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc +++ b/chrome/renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc @@ -45,7 +45,7 @@ class PhishingDOMFeatureExtractorTest protected: PhishingDOMFeatureExtractorTest() : content::RenderViewFakeResourcesTest(), - ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) {} + weak_factory_(this) {} virtual ~PhishingDOMFeatureExtractorTest() {} diff --git a/chrome/renderer/safe_browsing/phishing_term_feature_extractor.cc b/chrome/renderer/safe_browsing/phishing_term_feature_extractor.cc index fa3b237..dd17057 100644 --- a/chrome/renderer/safe_browsing/phishing_term_feature_extractor.cc +++ b/chrome/renderer/safe_browsing/phishing_term_feature_extractor.cc @@ -102,7 +102,7 @@ PhishingTermFeatureExtractor::PhishingTermFeatureExtractor( murmurhash3_seed_(murmurhash3_seed), negative_word_cache_(kMaxNegativeWordCacheSize), clock_(clock), - ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) { + weak_factory_(this) { Clear(); } diff --git a/chrome/renderer/translate/translate_helper.cc b/chrome/renderer/translate/translate_helper.cc index 68cf1be..1664233 100644 --- a/chrome/renderer/translate/translate_helper.cc +++ b/chrome/renderer/translate/translate_helper.cc @@ -73,7 +73,7 @@ TranslateHelper::TranslateHelper(content::RenderView* render_view) : content::RenderViewObserver(render_view), translation_pending_(false), page_id_(-1), - ALLOW_THIS_IN_INITIALIZER_LIST(weak_method_factory_(this)) { + weak_method_factory_(this) { } TranslateHelper::~TranslateHelper() { |