Disable some flaky IndexedDB browser tests.

Disable some flaky IndexedDB browser tests.

BUG=123685
TEST=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=134672

[dgrogan, 2012-04-30 21:50:38]

Josh, could you review this?

[jsbell, 2012-04-30 22:48:53]

lgtm with formatting nits

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
File content/browser/in_process_webkit/indexed_db_layout_browsertest.cc (left):

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
content/browser/in_process_webkit/indexed_db_layout_browsertest.cc:104: //
http://crbug.com/120924
Is 120924 a duplicate of 123685? If not, keep this bug comment somewhere near
index-basics-workers.html reference.

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
File content/browser/in_process_webkit/indexed_db_layout_browsertest.cc (right):

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
content/browser/in_process_webkit/indexed_db_layout_browsertest.cc:42: // Flaky:
http://crbug.com/123685
Formatting-wise, take a look at content/browser/appcache/appcache_browsertest.cc
- possibly more verbose than needed, but very readable.

Otherwise, based on content/browser/worker_host/test/worker_browsertest.cc ...

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
content/browser/in_process_webkit/indexed_db_layout_browsertest.cc:43: // 
"pending-version-change-stuck-works-with-terminate.html",
leading // should be indented 2 spaces to align with " on non-commented line,
and followed by 1 space before "

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
content/browser/in_process_webkit/indexed_db_layout_browsertest.cc:50: //  
"index-basics-workers.html",
Same formatting as above: 2 spaces before //, 1 after

[dgrogan, 2012-05-01 00:46:08]

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
File content/browser/in_process_webkit/indexed_db_layout_browsertest.cc (left):

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
content/browser/in_process_webkit/indexed_db_layout_browsertest.cc:104: //
http://crbug.com/120924
On 2012/04/30 22:48:54, jsbell wrote:
> Is 120924 a duplicate of 123685? If not, keep this bug comment somewhere near
> index-basics-workers.html reference.

It was kind of a duplicate.  I closed it in any case so we don't need to refer
to it anymore.

120924 was originally about only index-basics-workers.html flakiness, of which
there were two causes. One is fixed, the other seems like the general flakiness
described by 123685.  So I decided to interpret it as being just about the cause
that is now fixed, and closed it.

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
File content/browser/in_process_webkit/indexed_db_layout_browsertest.cc (right):

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
content/browser/in_process_webkit/indexed_db_layout_browsertest.cc:43: // 
"pending-version-change-stuck-works-with-terminate.html",
On 2012/04/30 22:48:54, jsbell wrote:
> leading // should be indented 2 spaces to align with " on non-commented line,
> and followed by 1 space before "

Done.

https://chromiumcodereview.appspot.com/10261019/diff/1/content/browser/in_pro...
content/browser/in_process_webkit/indexed_db_layout_browsertest.cc:50: //  
"index-basics-workers.html",
On 2012/04/30 22:48:54, jsbell wrote:
> Same formatting as above: 2 spaces before //, 1 after

Done.

[commit-bot: I haz the power, 2012-05-01 00:46:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dgrogan@chromium.org/10261019/4001

[commit-bot: I haz the power, 2012-05-01 03:56:59]

Change committed as 134672