Skip to content

Commit

Permalink
Merge r223438 - Unreviewed attempt to fix the Windows debug build.
Browse files Browse the repository at this point in the history
* Modules/webdatabase/DatabaseTracker.cpp:
(WebCore::DatabaseTracker::deleteOrigin):
  • Loading branch information
ryanhaddad authored and carlosgcampos committed Oct 17, 2017
1 parent 3d67434 commit 782857a
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
7 changes: 7 additions & 0 deletions Source/WebCore/ChangeLog
@@ -1,3 +1,10 @@
2017-10-16 Ryan Haddad <ryanhaddad@apple.com>

Unreviewed attempt to fix the Windows debug build.

* Modules/webdatabase/DatabaseTracker.cpp:
(WebCore::DatabaseTracker::deleteOrigin):

2017-10-16 Maureen Daum <mdaum@apple.com>

If we fail to delete any database file, don't remove its information from the tracker database
Expand Down
2 changes: 2 additions & 0 deletions Source/WebCore/Modules/webdatabase/DatabaseTracker.cpp
Expand Up @@ -848,7 +848,9 @@ bool DatabaseTracker::deleteOrigin(const SecurityOriginData& origin, DeletionMod
// If we failed to delete any database file, don't remove the origin from the tracker
// database because we didn't successfully remove all of its data.
if (failedToDeleteAnyDatabaseFile) {
#if PLATFORM(COCOA)
RELEASE_LOG_ERROR(DatabaseTracker, "Failed to delete database for origin");
#endif
return false;
}

Expand Down

0 comments on commit 782857a

Please sign in to comment.