Chromium Code Reviews
Help | Chromium Project | Sign in
(1)

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years ago by Finnur
Modified:
5 years, 8 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

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 view.cc where we return early from a Layout call. BUG=8010 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=10294

Patch Set 1 #

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

Messages

Total messages: 2 (0 generated)
Finnur
8 years ago (2009-02-24 21:41:23 UTC) #1
Ben Goodger (Google)
8 years 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 f8e48bd