Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(629)

Side by Side Diff: content/browser/dom_storage/dom_storage_message_filter.cc

Issue 16294003: Update content/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/dom_storage/dom_storage_message_filter.h" 5 #include "content/browser/dom_storage/dom_storage_message_filter.h"
6 6
7 #include "base/auto_reset.h" 7 #include "base/auto_reset.h"
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/nullable_string16.h" 9 #include "base/nullable_string16.h"
10 #include "base/threading/sequenced_worker_pool.h" 10 #include "base/threading/sequenced_worker_pool.h"
(...skipping 14 matching lines...) Expand all
25 : context_(context->context()), 25 : context_(context->context()),
26 connection_dispatching_message_for_(0) { 26 connection_dispatching_message_for_(0) {
27 } 27 }
28 28
29 DOMStorageMessageFilter::~DOMStorageMessageFilter() { 29 DOMStorageMessageFilter::~DOMStorageMessageFilter() {
30 DCHECK(!host_.get()); 30 DCHECK(!host_.get());
31 } 31 }
32 32
33 void DOMStorageMessageFilter::InitializeInSequence() { 33 void DOMStorageMessageFilter::InitializeInSequence() {
34 DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::IO)); 34 DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::IO));
35 host_.reset(new dom_storage::DomStorageHost(context_)); 35 host_.reset(new dom_storage::DomStorageHost(context_.get()));
36 context_->AddEventObserver(this); 36 context_->AddEventObserver(this);
37 } 37 }
38 38
39 void DOMStorageMessageFilter::UninitializeInSequence() { 39 void DOMStorageMessageFilter::UninitializeInSequence() {
40 // TODO(michaeln): Restore this DCHECK once crbug/166470 and crbug/164403 40 // TODO(michaeln): Restore this DCHECK once crbug/166470 and crbug/164403
41 // are resolved. 41 // are resolved.
42 // DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::IO)); 42 // DCHECK(!BrowserThread::CurrentlyOn(BrowserThread::IO));
43 context_->RemoveEventObserver(this); 43 context_->RemoveEventObserver(this);
44 host_.reset(); 44 host_.reset();
45 } 45 }
(...skipping 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
203 params.connection_id = connection_dispatching_message_for_; 203 params.connection_id = connection_dispatching_message_for_;
204 params.key = key; 204 params.key = key;
205 params.new_value = new_value; 205 params.new_value = new_value;
206 params.old_value = old_value; 206 params.old_value = old_value;
207 params.namespace_id = area->namespace_id(); 207 params.namespace_id = area->namespace_id();
208 Send(new DOMStorageMsg_Event(params)); 208 Send(new DOMStorageMsg_Event(params));
209 } 209 }
210 } 210 }
211 211
212 } // namespace content 212 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/dom_storage/dom_storage_context_impl.cc ('k') | content/browser/download/download_resource_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698