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

Side by Side Diff: chrome/browser/ui/webui/bookmarks_ui.cc

Issue 10067033: RefCounted types should not have public destructors, chrome/browser/ui (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge fix Created 8 years, 8 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 "chrome/browser/ui/webui/bookmarks_ui.h" 5 #include "chrome/browser/ui/webui/bookmarks_ui.h"
6 6
7 #include "base/memory/ref_counted_memory.h" 7 #include "base/memory/ref_counted_memory.h"
8 #include "base/memory/singleton.h" 8 #include "base/memory/singleton.h"
9 #include "base/message_loop.h" 9 #include "base/message_loop.h"
10 #include "base/string_number_conversions.h" 10 #include "base/string_number_conversions.h"
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 44
45 SendResponse(request_id, NULL); 45 SendResponse(request_id, NULL);
46 } 46 }
47 47
48 std::string BookmarksUIHTMLSource::GetMimeType(const std::string& path) const { 48 std::string BookmarksUIHTMLSource::GetMimeType(const std::string& path) const {
49 NOTREACHED() << "We should never get here since the extension should have" 49 NOTREACHED() << "We should never get here since the extension should have"
50 << "been triggered"; 50 << "been triggered";
51 return "text/html"; 51 return "text/html";
52 } 52 }
53 53
54 BookmarksUIHTMLSource::~BookmarksUIHTMLSource() {}
55
54 //////////////////////////////////////////////////////////////////////////////// 56 ////////////////////////////////////////////////////////////////////////////////
55 // 57 //
56 // BookmarksUI 58 // BookmarksUI
57 // 59 //
58 //////////////////////////////////////////////////////////////////////////////// 60 ////////////////////////////////////////////////////////////////////////////////
59 61
60 BookmarksUI::BookmarksUI(content::WebUI* web_ui) : WebUIController(web_ui) { 62 BookmarksUI::BookmarksUI(content::WebUI* web_ui) : WebUIController(web_ui) {
61 BookmarksUIHTMLSource* html_source = new BookmarksUIHTMLSource(); 63 BookmarksUIHTMLSource* html_source = new BookmarksUIHTMLSource();
62 64
63 // Set up the chrome://bookmarks/ source. 65 // Set up the chrome://bookmarks/ source.
64 Profile* profile = Profile::FromWebUI(web_ui); 66 Profile* profile = Profile::FromWebUI(web_ui);
65 profile->GetChromeURLDataManager()->AddDataSource(html_source); 67 profile->GetChromeURLDataManager()->AddDataSource(html_source);
66 } 68 }
67 69
68 // static 70 // static
69 RefCountedMemory* BookmarksUI::GetFaviconResourceBytes() { 71 RefCountedMemory* BookmarksUI::GetFaviconResourceBytes() {
70 return ResourceBundle::GetSharedInstance(). 72 return ResourceBundle::GetSharedInstance().
71 LoadDataResourceBytes(IDR_BOOKMARKS_FAVICON); 73 LoadDataResourceBytes(IDR_BOOKMARKS_FAVICON);
72 } 74 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/webui/bookmarks_ui.h ('k') | chrome/browser/ui/webui/chrome_web_ui_data_source.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698