Commit 9f12ac29 authored by sky@chromium.org's avatar sky@chromium.org

Adds a comment to GridLayout to explain what DCHECK does

BUG=none
TEST=none
R=finnur@chromium.org

Review URL: https://chromiumcodereview.appspot.com/23464036

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222055 0039d316-1c4b-4281-b951-d872f2087c98
parent e3068b73
...@@ -995,6 +995,9 @@ void GridLayout::AddViewState(ViewState* view_state) { ...@@ -995,6 +995,9 @@ void GridLayout::AddViewState(ViewState* view_state) {
void GridLayout::AddRow(Row* row) { void GridLayout::AddRow(Row* row) {
current_row_++; current_row_++;
remaining_row_span_--; remaining_row_span_--;
// GridLayout requires that if you add a View with a row span you use the same
// column set for each of the rows the view lands it. This DCHECK verifies
// that.
DCHECK(remaining_row_span_ <= 0 || DCHECK(remaining_row_span_ <= 0 ||
row->column_set() == NULL || row->column_set() == NULL ||
row->column_set() == GetLastValidColumnSet()); row->column_set() == GetLastValidColumnSet());
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment