My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 94: Can't view change https://review.source.android.com/8981
  Back to list
Status:  Released
Owner:  code-rev...@gtempaccount.com
Closed:  Oct 2012


Sign in to add a comment
 
Reported by code-rev...@gtempaccount.com, Sep 24, 2009
Reported by Shawn Pearce <sop@google.com> on Wed Mar 04 10:32:56 PST 2009
Source: JIRA GERRIT-94
Affected Version: 2.0.6

Change https://review.source.android.com/8981 crashes with a JavaScript error:

Error: uncaught exception: java.lang.IndexOutOfBoundsException: Row index: 41,
Row size: 1
Sep 24, 2009
#1 code-rev...@gtempaccount.com
Comment by Shawn Pearce <sop@google.com> on Wed Mar 04 12:15:33 PST 2009

This was caused by a change in timing in the table construction.

A race condition existed where if the entire table could be composed before
the progress meter was initialized, but then delayed execution one cycle in
order to finish binding the row items, the progress meter replaced the table
content and the row items couldn't be set as the table had only 1 row in it
(the progress meter).

This whole delayed execution of creating the patch table is a mess.
Sep 24, 2009
#2 code-rev...@gtempaccount.com
Comment by Shawn Pearce <sop@google.com> on Wed Mar 04 13:17:39 PST 2009

Fixed by:

  https://review.source.android.com/9073
  https://review.source.android.com/9074
Sep 24, 2009
#3 code-rev...@gtempaccount.com
Update by Shawn Pearce <sop@google.com> on Wed Mar 04 13:17:39 PST 2009

Fixed in version 2.0.6.
Status: Fixed
Sep 25, 2009
#4 code-rev...@gtempaccount.com
(No comment was entered for this change.)
Labels: FixedIn-2.0.6
Oct 25, 2012
#5 sop@google.com
(No comment was entered for this change.)
Status: Released
Sign in to add a comment

Powered by Google Project Hosting