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

Side by Side Diff: chrome/browser/ui/webui/net_internals/net_internals_ui.cc

Issue 11275088: Remove implicit scoped_refptr operator T* Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 8 years, 1 month 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
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 "chrome/browser/ui/webui/net_internals/net_internals_ui.h" 5 #include "chrome/browser/ui/webui/net_internals/net_internals_ui.h"
6 6
7 #include <list> 7 #include <list>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 528 matching lines...) Expand 10 before | Expand all | Expand 10 after
539 539
540 //////////////////////////////////////////////////////////////////////////////// 540 ////////////////////////////////////////////////////////////////////////////////
541 // 541 //
542 // NetInternalsMessageHandler 542 // NetInternalsMessageHandler
543 // 543 //
544 //////////////////////////////////////////////////////////////////////////////// 544 ////////////////////////////////////////////////////////////////////////////////
545 545
546 NetInternalsMessageHandler::NetInternalsMessageHandler() {} 546 NetInternalsMessageHandler::NetInternalsMessageHandler() {}
547 547
548 NetInternalsMessageHandler::~NetInternalsMessageHandler() { 548 NetInternalsMessageHandler::~NetInternalsMessageHandler() {
549 if (proxy_) { 549 if (proxy_.get()) {
550 proxy_.get()->OnWebUIDeleted(); 550 proxy_.get()->OnWebUIDeleted();
551 // Notify the handler on the IO thread that the renderer is gone. 551 // Notify the handler on the IO thread that the renderer is gone.
552 BrowserThread::PostTask(BrowserThread::IO, FROM_HERE, 552 BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
553 base::Bind(&IOThreadImpl::Detach, proxy_.get())); 553 base::Bind(&IOThreadImpl::Detach, proxy_.get()));
554 } 554 }
555 } 555 }
556 556
557 void NetInternalsMessageHandler::RegisterMessages() { 557 void NetInternalsMessageHandler::RegisterMessages() {
558 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 558 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
559 559
(...skipping 1215 matching lines...) Expand 10 before | Expand all | Expand 10 after
1775 } 1775 }
1776 1776
1777 NetInternalsUI::NetInternalsUI(content::WebUI* web_ui) 1777 NetInternalsUI::NetInternalsUI(content::WebUI* web_ui)
1778 : WebUIController(web_ui) { 1778 : WebUIController(web_ui) {
1779 web_ui->AddMessageHandler(new NetInternalsMessageHandler()); 1779 web_ui->AddMessageHandler(new NetInternalsMessageHandler());
1780 1780
1781 // Set up the chrome://net-internals/ source. 1781 // Set up the chrome://net-internals/ source.
1782 Profile* profile = Profile::FromWebUI(web_ui); 1782 Profile* profile = Profile::FromWebUI(web_ui);
1783 ChromeURLDataManager::AddDataSource(profile, CreateNetInternalsHTMLSource()); 1783 ChromeURLDataManager::AddDataSource(profile, CreateNetInternalsHTMLSource());
1784 } 1784 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/webui/nacl_ui.cc ('k') | chrome/browser/ui/webui/options/search_engine_manager_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698