Issue 8202003: IndexedDB: Update calls to and implementations of WebIDBFactory.

Issue 8202003: IndexedDB: Update calls to and implementations of WebIDBFactory. (Closed)

Created:
9 years, 2 months ago by hans

Modified:
9 years, 1 month ago

Reviewers:
jam, dgrogan, jsbell, piman

CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, dpranke+watch-content_chromium.org, jam, Paweł Hajdan Jr.

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

Description

IndexedDB: Update calls to and implementations of WebIDBFactory. The BackingStoreType and maximumSize parameters have been removed. Remove the --indexeddb-use-sqlite command-line flag. BUG=none TEST=all current tests pass (layout tests, browser tests, pyautotests) Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107547

Patch Set 1 #

Patch Set 2 : rebase #

Created: 9 years, 1 month ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+6 lines, -75 lines)			Patch
M	content/browser/in_process_webkit/indexed_db_browsertest.cc	View	1	1 chunk	+0 lines, -11 lines	0 comments	Download
M	content/browser/in_process_webkit/indexed_db_dispatcher_host.cc	View	1	2 chunks	+2 lines, -26 lines	0 comments	Download
M	content/public/common/content_switches.h	View	1	1 chunk	+0 lines, -1 line	0 comments	Download
M	content/public/common/content_switches.cc	View	1	1 chunk	+0 lines, -4 lines	0 comments	Download
M	content/renderer/renderer_webidbfactory_impl.h	View		1 chunk	+2 lines, -16 lines	0 comments	Download
M	content/renderer/renderer_webidbfactory_impl.cc	View		3 chunks	+2 lines, -17 lines	0 comments	Download

Messages

Total messages: 8 (0 generated)

Expand Messages | Collapse Messages

hans

This patch depends on https://bugs.webkit.org/show_bug.cgi?id=69620

9 years, 2 months ago (2011-10-07 13:59:25 UTC) #1

hans

piman@: OWNERS approval for content/renderer? jam@: OWNERS approval for content/public?

9 years, 1 month ago (2011-10-26 12:40:22 UTC) #3

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/hans@chromium.org/8202003/3001

9 years, 1 month ago (2011-10-27 08:27:56 UTC) #7

commit-bot: I haz the power

9 years, 1 month ago (2011-10-27 08:27:57 UTC) #8

Can't apply patch for file content/public/common/content_switches.h.
While running patch -p1 --forward --force;
patching file content/public/common/content_switches.h
Hunk #1 FAILED at 143.
1 out of 1 hunk FAILED -- saving rejects to file
content/public/common/content_switches.h.rej

Expand Messages | Collapse Messages