Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(32)

Issues Search
Open Issues | Closed Issues | All Issues | Sign in with your Google Account to create issues and add comments

Issue 1178593002: WebSQL: Added database open time statistics reporting. (Closed)

Created:
4 years, 10 months ago by cmumford
Modified:
4 years, 10 months ago
Reviewers:
michaeln, Mike West
CC:
blink-reviews, dglazkov+blink
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

WebSQL: Added database open time statistics reporting. BUG=497960 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197387

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -11 lines) Patch
M Source/modules/webdatabase/Database.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/modules/webdatabase/Database.cpp View 8 chunks +11 lines, -9 lines 0 comments Download
M public/platform/WebDatabaseObserver.h View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 14 (5 generated)
cmumford
4 years, 10 months ago (2015-06-10 18:36:38 UTC) #2
cmumford
Adding mkwst@ for review of public/platform/WebDatabaseObserver.h
4 years, 10 months ago (2015-06-10 18:45:57 UTC) #4
michaeln
lgtm 2
4 years, 10 months ago (2015-06-10 19:03:49 UTC) #5
cmumford
ping mkwst@ for review of public/platform/WebDatabaseObserver.h
4 years, 10 months ago (2015-06-11 15:26:21 UTC) #6
Mike West
LGTM.
4 years, 10 months ago (2015-06-12 07:29:38 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1178593002/1
4 years, 10 months ago (2015-06-17 20:11:06 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: mac_blink_rel on tryserver.blink (JOB_FAILED, http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/59322)
4 years, 10 months ago (2015-06-17 22:50:40 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1178593002/1
4 years, 10 months ago (2015-06-18 17:55:48 UTC) #13
commit-bot: I haz the power
4 years, 10 months ago (2015-06-18 18:57:28 UTC) #14
Message was sent while issue was closed. 
Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=197387

Powered by Google App Engine
This is Rietveld 408576698