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

Side by Side Diff: chrome/utility/importer/edge_database_reader_win.cc

Issue 2341693002: Re-write many calls to WrapUnique() with MakeUnique() (Closed)
Patch Set: Fix one under-change and one mysterious revert Created 4 years, 3 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/utility/importer/edge_database_reader_win.h" 5 #include "chrome/utility/importer/edge_database_reader_win.h"
6 6
7 #include <windows.h> 7 #include <windows.h>
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 238 matching lines...) Expand 10 before | Expand all | Expand 10 after
249 249
250 if (!IsOpen()) { 250 if (!IsOpen()) {
251 SetLastError(JET_errDatabaseNotFound); 251 SetLastError(JET_errDatabaseNotFound);
252 return nullptr; 252 return nullptr;
253 } 253 }
254 254
255 if (!SetLastError(JetOpenTable(session_id_, db_id_, table_name.c_str(), 255 if (!SetLastError(JetOpenTable(session_id_, db_id_, table_name.c_str(),
256 nullptr, 0, JET_bitTableReadOnly, &table_id))) 256 nullptr, 0, JET_bitTableReadOnly, &table_id)))
257 return nullptr; 257 return nullptr;
258 258
259 return base::WrapUnique( 259 return base::MakeUnique<EdgeDatabaseTableEnumerator>(table_name, session_id_,
260 new EdgeDatabaseTableEnumerator(table_name, session_id_, table_id)); 260 table_id);
261 } 261 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698