Since this is just a mixin now, it should be named "UserPageDataGridMixin".

The suffix for this class is "Grid," but should be "DataGrid" to match the others.

Trailing whitespace.

Too many blank lines.

Some older views are inconsistent in this way, but docstrings must follow the following format:
"""One-line summary."""

or:
"""One-line summary.

Multi-line description.
"""

Reading through this code, it looks like most of the functionality is here. Can you specify in the description what is left to do, and if it's pretty close, remove the WIP tag?

Please wrap this to 80 columns.

This should use _() for localization.

Please wrap at 80 columns.

This should use _() for localization.

Docstrings should start with a single line for a summary, then (optionally) a blank line and paragraphs of description. Please also wrap to 80 columns.

Your code here uses 2-space indentation instead of 4.

I don't know how permanent this is, but it should probably be formatted more like this:

datagrid.tabs = [
    (UserPageReviewRequestDataGrid.tab_title(username),
     reverse('user', args=[username])),
    (UserPageReviewsDataGrid.tab_title(username),
     reverse('user', args=[username, 'reviews'])),
]

Please wrap to 80 columns.

Please put a space ofter the :. Here and throughout this file.

Remove this line.

Remove this line.

Indent 2 spaces in LESS files.

Indent 1 space in HTML. Also note that we add spaces between {% and the tag names for block tags (separate from the HTML indentation). This file hasn't been updated to indent the tags correctly -- would you mind doing so while you're changing this? See reviewboard/templates/reviews/review_request_box.html for a good example.

This is looking pretty good! I have a bunch of style comments.

If this isn't necessary, can you remove it?

If this isn't necessary, can you remove it?

There's an extra space between 'review_summary' and ,

Please capitalize the "Do"

Maybe mention that this is an abstract class?

Wrap to 80 columns.

This doesn't match between here and UserPageReviewRequestDataGrid (one has "user page", one has "user's page"). Can you also include a mention of reviews vs. review requests in the docstring summary?

Trailing whitespace. Can you also rewrap this to put one item per line?

Indent 1 more space.

Indent 1 more space.

Indent 1 more space.

Indent 1 more space.

Indent 1 more space.

Remove these two lines.

Can you switch the order of these two URLs?

Comments should have a space between # and the comment text, and start with a capital letter.

Maybe put these all on the same line?

Maybe put these all on the same line? Also, watch out for trailing whitespace.

Comments should have a space between # and the comment text, and start with a capital letter.

Indent 1 more space.

Indent 1 more space.

Indent 1 more space.

Hmm. It looks like it's been quite some time since you pulled from the origin repository. We've made some changes to the ReviewRequest version of this query to improve performance. Can you rebase on top of the latest master code, and add the accessible_repo_ids and accessible_group_ids mechanism that ReviewRequestManager._query uses?

Almost there!

Indent 2 more spaces.

Can you add a blank line here?

"Group" hasn't been imported here. You should import it at the top of this method (like in ReviewRequestManager._query)

These are pretty generic class names, and the !important makes me scared. Can you maybe make these a bit more specific, so that these rules won't bleed out into other parts of the UI?

This looks good to me. Because it's a big change, Christian is also going to take a look before we push it. Nice work!

Missing trailing period.

This can be one line, like the one above.

This is all a copy/paste of the load_extra_data in ReviewRequestDataGrid. We should pull that out into a ShowClosedReviewRequestsMixin.

Should remove the blank line.

Wrong class.

Space before %

Should remove the blank line.

Space before %

The latter part of that regex is too verbose. We know it'll just be [a-z-]

Should be on one line.

No blank line.

No need for this comment.

Let's do review-requests instead.
Also, is None

The parameters are the same for both classes. Let's compute the class here, and initialize it after the conditionals.

No need for this comment.

Blank line between these.

Missing period.

There's a bunch of !important rules in here. We should remove all of them, and instead nest the rules better so that the browser will naturally choose them.

Should just list the class, not the element.
Same with all the ones below.

The &. ones go above the child element/class rules.

THis should go above the .datagrid-title rule above.

Too many spaces before endif.

Shouldn't have  . If we're trying to add some spacing, that should be done with CSS.

It's not valid to have a <ul> inside another <ul>. It needs to be in <li>.

Looking good!

This should be:
datagrid_cls = UserPageReviewRequestDataGrid

Then below, you can just do:
datagrid = datagrid_cls(request, .....)

Indentation got a bit wacky here.

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    reviewboard/datagrids/views.py
    reviewboard/datagrids/columns.py
    reviewboard/datagrids/tests.py
    reviewboard/datagrids/urls.py
    reviewboard/extensions/hooks.py
    reviewboard/datagrids/grids.py
    reviewboard/reviews/managers.py
  Ignored Files:
    reviewboard/static/rb/css/common.less
    reviewboard/templates/datagrids/review_request_listview.html

Col: 17
 E126 continuation line over-indented for hanging indent

Col: 5
 E303 too many blank lines (2)

Col: 21
 E128 continuation line under-indented for visual indent

This is a review from Review Bot.
  Tool: Pyflakes
  Processed Files:
    reviewboard/datagrids/views.py
    reviewboard/datagrids/columns.py
    reviewboard/datagrids/tests.py
    reviewboard/datagrids/urls.py
    reviewboard/extensions/hooks.py
    reviewboard/datagrids/grids.py
    reviewboard/reviews/managers.py
  Ignored Files:
    reviewboard/static/rb/css/common.less
    reviewboard/templates/datagrids/review_request_listview.html

 local variable 'review1' is assigned to but never used

 local variable 'review2' is assigned to but never used

 'SignalHook' imported but unused

 'TemplateHook' imported but unused

 'URLHook' imported but unused

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    reviewboard/datagrids/views.py
    reviewboard/datagrids/columns.py
    reviewboard/datagrids/tests.py
    reviewboard/datagrids/urls.py
    reviewboard/extensions/hooks.py
    reviewboard/datagrids/grids.py
    reviewboard/reviews/managers.py
  Ignored Files:
    reviewboard/static/rb/css/common.less
    reviewboard/templates/datagrids/review_request_listview.html

Col: 38
 E127 continuation line over-indented for visual indent

Col: 38
 E127 continuation line over-indented for visual indent

Col: 21
 E128 continuation line under-indented for visual indent

This is a review from Review Bot.
  Tool: Pyflakes
  Processed Files:
    reviewboard/datagrids/views.py
    reviewboard/datagrids/columns.py
    reviewboard/datagrids/tests.py
    reviewboard/datagrids/urls.py
    reviewboard/extensions/hooks.py
    reviewboard/datagrids/grids.py
    reviewboard/reviews/managers.py
  Ignored Files:
    reviewboard/static/rb/css/common.less
    reviewboard/templates/datagrids/review_request_listview.html

 'SignalHook' imported but unused

 'TemplateHook' imported but unused

 'URLHook' imported but unused

A few small things. This is pretty great! Just a couple trivial style changes, and we'll be able to put this in for 2.1 :)

Needs a trailing period.

For each of these, you should have a trailing comma after the value in the dictionary, and the }) should be aligned with the opening statement on its own line, like:

self.queryset = self.queryset.filter(**{
    ...
});

These can probably just be constants on the class, right?

I don't think this is needed.

This code is pretty similar to what's in ReviewRequestManager. I don't expect you to worry about refactoring all this right now, but we should probably do this at some point.
Can you put a TODO comment above all this code saying it should be consolidated with the queries in ReviewRequestManager?