Improve IDB module's SecurityError messages.

Improve IDB module's SecurityError messages.

BUG=152212
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=157369

[Mike West, 2013-08-20 08:37:23]

Hi Jochen, WDYT?

Note that I've poked at the notification tests, but they don't work. Failing
results are checked in. It's totally awesome.

-mike

[jochen (gone - plz use gerrit), 2013-08-21 08:48:14]

https://codereview.chromium.org/23135009/diff/7001/Source/testing/runner/Test...
File Source/testing/runner/TestRunner.cpp (right):

https://codereview.chromium.org/23135009/diff/7001/Source/testing/runner/Test...
Source/testing/runner/TestRunner.cpp:1863:
m_notificationPresenter->removeAllWebNotificationPermissions();
why not do this in TestRunner::reset automatically?

[jsbell, 2013-08-21 20:18:58]

IDB change lgtm

https://codereview.chromium.org/23135009/diff/7001/Source/modules/indexeddb/I...
File Source/modules/indexeddb/IDBFactory.cpp (right):

https://codereview.chromium.org/23135009/diff/7001/Source/modules/indexeddb/I...
Source/modules/indexeddb/IDBFactory.cpp:85:
es.throwSecurityError(ExceptionMessages::failedToExecute("getDatabaseNames",
"IDBFactory", "access to the Indexed Database API is denied in this context."));
nit: pull the repeated "access to the..." string out into a const char[] ?

(I'm not sure how well our compilers do on string coalescing, and I wouldn't
suggest this except I've gotten this feedback myself.)

[jsbell, 2013-08-21 20:20:36]

FYI, the lack of an IDB test for this is captured in http://crbug.com/113738 -
may be easier to write the test now than when the bug was filed

[Mike West, 2013-09-06 11:58:05]

I've dropped the TestRunner bits. They didn't work anyway.

[Mike West, 2013-09-06 12:07:36]

On 2013/09/06 11:58:05, Mike West wrote:
> I've dropped the TestRunner bits. They didn't work anyway.

Actually, I'll cull this back to just the IDB bits. Smaller patches are better.

[commit-bot: I haz the power, 2013-09-06 12:26:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mkwst@chromium.org/23135009/26001

[commit-bot: I haz the power, 2013-09-06 14:28:02]

Commit queue rejected this change because the description was changed
between the time the change entered the commit queue and the time it
was ready to commit. You can safely check the commit box again.

[commit-bot: I haz the power, 2013-09-06 14:45:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mkwst@chromium.org/23135009/26001

[commit-bot: I haz the power, 2013-09-06 14:45:48]

Change committed as 157369