[tracing] Show the local storage databases under dom_storage

[tracing] Show the local storage databases under dom_storage

The sqlite memory usage was being reported under "sqlite" column. After
this CL the sql databases opened by DOM storage will be shown under the
"dom_storage" column. This is done by adding an ownership edge
(implemented in sql::Connection::ReportMemoryUsage).

BUG=605785
Committed: https://crrev.com/9270bd2a45d85f4810cafedc6f25e188edf2d671
Cr-Commit-Position: refs/heads/master@{#437426}

[ssid, 2016-12-03 01:08:47]

ssid@chromium.org changed reviewers:
+ dskiba@chromium.org, michaeln@chromium.org

[ssid, 2016-12-03 01:08:47]

ptal, thanks!

[ssid, 2016-12-03 01:09:10]

Description was changed from

==========
[tracing] Add the local storage databases under dom_storage

The sqlite memory usage was being reported under "sqlite" column. After
this CL the sql databases opened by DOM storage will be shown under the
"dom_storage" column. This is done by adding an ownership edge
(implemented in sql::Connection::ReportMemoryUsage).

BUG=605785
==========

to

==========
[tracing] Show the local storage databases under dom_storage

The sqlite memory usage was being reported under "sqlite" column. After
this CL the sql databases opened by DOM storage will be shown under the
"dom_storage" column. This is done by adding an ownership edge
(implemented in sql::Connection::ReportMemoryUsage).

BUG=605785
==========

[michaeln, 2016-12-05 22:42:53]

https://codereview.chromium.org/2548063002/diff/1/content/browser/dom_storage...
File content/browser/dom_storage/dom_storage_database.cc (right):

https://codereview.chromium.org/2548063002/diff/1/content/browser/dom_storage...
content/browser/dom_storage/dom_storage_database.cc:140: const std::string&
name) {
i think you need to check IsOpen() here

[ssid, 2016-12-08 18:09:47]

Thanks, fixed. ptal

https://codereview.chromium.org/2548063002/diff/1/content/browser/dom_storage...
File content/browser/dom_storage/dom_storage_database.cc (right):

https://codereview.chromium.org/2548063002/diff/1/content/browser/dom_storage...
content/browser/dom_storage/dom_storage_database.cc:140: const std::string&
name) {
On 2016/12/05 22:42:53, michaeln wrote:
> i think you need to check IsOpen() here

Done.

[michaeln, 2016-12-08 23:26:11]

lgtm

[ssid, 2016-12-09 00:30:55]

The CQ bit was checked by ssid@chromium.org

[commit-bot: I haz the power, 2016-12-09 00:31:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-09 01:57:04]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481243455742480,
"parent_rev": "446e618afd0a05f9b4d09ce76a6b6bca6313b1f2", "commit_rev":
"22a5f3561a72b3f8480234c679899b77005604d0"}

[commit-bot: I haz the power, 2016-12-09 01:57:31]

Description was changed from

==========
[tracing] Show the local storage databases under dom_storage

The sqlite memory usage was being reported under "sqlite" column. After
this CL the sql databases opened by DOM storage will be shown under the
"dom_storage" column. This is done by adding an ownership edge
(implemented in sql::Connection::ReportMemoryUsage).

BUG=605785
==========

to

==========
[tracing] Show the local storage databases under dom_storage

The sqlite memory usage was being reported under "sqlite" column. After
this CL the sql databases opened by DOM storage will be shown under the
"dom_storage" column. This is done by adding an ownership edge
(implemented in sql::Connection::ReportMemoryUsage).

BUG=605785
==========

[commit-bot: I haz the power, 2016-12-09 01:57:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-09 02:01:08]

Description was changed from

==========
[tracing] Show the local storage databases under dom_storage

The sqlite memory usage was being reported under "sqlite" column. After
this CL the sql databases opened by DOM storage will be shown under the
"dom_storage" column. This is done by adding an ownership edge
(implemented in sql::Connection::ReportMemoryUsage).

BUG=605785
==========

to

==========
[tracing] Show the local storage databases under dom_storage

The sqlite memory usage was being reported under "sqlite" column. After
this CL the sql databases opened by DOM storage will be shown under the
"dom_storage" column. This is done by adding an ownership edge
(implemented in sql::Connection::ReportMemoryUsage).

BUG=605785

Committed: https://crrev.com/9270bd2a45d85f4810cafedc6f25e188edf2d671
Cr-Commit-Position: refs/heads/master@{#437426}
==========

[commit-bot: I haz the power, 2016-12-09 02:01:08]

Patchset 2 (id:??) landed as
https://crrev.com/9270bd2a45d85f4810cafedc6f25e188edf2d671
Cr-Commit-Position: refs/heads/master@{#437426}