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

Side by Side Diff: content/browser/download/save_file.cc

Issue 2333923004: Extracting NetLog inner classes into their own classes. (Closed)
Patch Set: Some nit fixes and better, impl-agnostic naming of net_log_parameters_callback_typedef.h -> net/log… Created 4 years, 2 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
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/download/save_file.h" 5 #include "content/browser/download/save_file.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "content/public/browser/browser_thread.h" 8 #include "content/public/browser/browser_thread.h"
9 #include "net/log/net_log_with_source.h"
9 10
10 namespace content { 11 namespace content {
11 12
12 // TODO(asanka): SaveFile should use the target directory of the save package as 13 // TODO(asanka): SaveFile should use the target directory of the save package as
13 // the default download directory when initializing |file_|. 14 // the default download directory when initializing |file_|.
14 // Unfortunately, as it is, constructors of SaveFile don't always 15 // Unfortunately, as it is, constructors of SaveFile don't always
15 // have access to the SavePackage at this point. 16 // have access to the SavePackage at this point.
16 SaveFile::SaveFile(const SaveFileCreateInfo* info, bool calculate_hash) 17 SaveFile::SaveFile(const SaveFileCreateInfo* info, bool calculate_hash)
17 : file_(net::NetLogWithSource()), info_(info) { 18 : file_(net::NetLogWithSource()), info_(info) {
18 DCHECK_CURRENTLY_ON(BrowserThread::FILE); 19 DCHECK_CURRENTLY_ON(BrowserThread::FILE);
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
67 68
68 int64_t SaveFile::BytesSoFar() const { 69 int64_t SaveFile::BytesSoFar() const {
69 return file_.bytes_so_far(); 70 return file_.bytes_so_far();
70 } 71 }
71 72
72 std::string SaveFile::DebugString() const { 73 std::string SaveFile::DebugString() const {
73 return file_.DebugString(); 74 return file_.DebugString();
74 } 75 }
75 76
76 } // namespace content 77 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698