Skip to content

Commit

Permalink
Bug 1462784 - Remove the STORAGE category. r=njn
Browse files Browse the repository at this point in the history
This was used to label IndexedDB work and work in storage/mozStorage*.
I don't think this deserves its own category; categories are most useful for
the main thread, and most of the time-consuming database-related work happens
on helper threads. The main thread pieces are mostly for asynchronicity-
coordination and don't usually take up time.

This patch labels IndexedDB work as DOM instead (which is maybe debatable) and
the rest as OTHER.

MozReview-Commit-ID: 3UYhFFbi3Ry

--HG--
extra : rebase_source : 5c88dfd67274103de01fe44191f49776017738f9
  • Loading branch information
mstange committed May 18, 2018
1 parent f49c17a commit bf59099
Show file tree
Hide file tree
Showing 7 changed files with 122 additions and 123 deletions.
4 changes: 2 additions & 2 deletions dom/indexedDB/ActorsChild.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -731,7 +731,7 @@ DispatchErrorEvent(IDBRequest* aRequest,
MOZ_ASSERT(NS_FAILED(aErrorCode));
MOZ_ASSERT(NS_ERROR_GET_MODULE(aErrorCode) == NS_ERROR_MODULE_DOM_INDEXEDDB);

AUTO_PROFILER_LABEL("IndexedDB:DispatchErrorEvent", STORAGE);
AUTO_PROFILER_LABEL("IndexedDB:DispatchErrorEvent", DOM);

RefPtr<IDBRequest> request = aRequest;
RefPtr<IDBTransaction> transaction = aTransaction;
Expand Down Expand Up @@ -804,7 +804,7 @@ DispatchSuccessEvent(ResultHelper* aResultHelper,
{
MOZ_ASSERT(aResultHelper);

AUTO_PROFILER_LABEL("IndexedDB:DispatchSuccessEvent", STORAGE);
AUTO_PROFILER_LABEL("IndexedDB:DispatchSuccessEvent", DOM);

RefPtr<IDBRequest> request = aResultHelper->Request();
MOZ_ASSERT(request);
Expand Down
Loading

0 comments on commit bf59099

Please sign in to comment.