Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in

Issue 27102: Fixing 8010: Missing options from Options dlg.... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
8 years, 6 months ago by Finnur
6 years, 2 months ago


Fixing 8010: Missing options from Options dlg. This should also fix the hang in the interactive_ui_tests. The problem is in a one line change in where we return early from a Layout call. BUG=8010 Committed:

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1 line) Patch
M chrome/views/ View 1 chunk +0 lines, -1 line 0 comments Download
Trybot results:
Commit queue not available (can’t edit this change).


Total messages: 2 (0 generated)
8 years, 6 months ago (2009-02-24 21:41:23 UTC) #1
Ben Goodger (Google)
8 years, 6 months ago (2009-02-24 21:43:45 UTC) #2
OK - can you add a comment saying that we think we should return here since the
layout manager should be handling things but it causes regressions?
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld b40b6558b