Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(822)

Issue 217283006: Force software layer to redraw when it is resized. (Closed)

Created:
6 years, 8 months ago by ccameron
Modified:
6 years, 8 months ago
CC:
chromium-reviews, yusukes+watch_chromium.org, yukishiino+watch_chromium.org, jam, penghuang+watch_chromium.org, nona+watch_chromium.org, darin-cc_chromium.org, James Su, miu+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Force software layer to redraw when it is resized. Consider the following sequence of events: 1. The view is initialized with a size of 0x0 2. A frame comes in at a size of 50x50. Nothing is drawn. 3. The view is resized to 50x50. We should re-draw the full frame at step 3, but we don't. This patch makes it so that we do. BUG=358450 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=261465

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -1 line) Patch
M content/browser/renderer_host/render_widget_host_view_mac.mm View 1 chunk +6 lines, -1 line 0 comments Download

Messages

Total messages: 10 (0 generated)
ccameron
Quick fix for one of the CA bugs found during the extra QA coverage.
6 years, 8 months ago (2014-04-02 23:17:02 UTC) #1
Ken Russell (switch to Gerrit)
lgtm
6 years, 8 months ago (2014-04-03 01:10:55 UTC) #2
ccameron
Thanks!!
6 years, 8 months ago (2014-04-03 08:26:39 UTC) #3
ccameron
The CQ bit was checked by ccameron@chromium.org
6 years, 8 months ago (2014-04-03 08:26:42 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ccameron@chromium.org/217283006/1
6 years, 8 months ago (2014-04-03 08:27:26 UTC) #5
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 8 months ago (2014-04-03 12:21:09 UTC) #6
commit-bot: I haz the power
Commit queue rejected this change because the description was changed between the time the change ...
6 years, 8 months ago (2014-04-03 12:21:09 UTC) #7
ccameron
The CQ bit was checked by ccameron@chromium.org
6 years, 8 months ago (2014-04-03 17:26:14 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ccameron@chromium.org/217283006/1
6 years, 8 months ago (2014-04-03 17:27:02 UTC) #9
commit-bot: I haz the power
6 years, 8 months ago (2014-04-03 17:29:04 UTC) #10
Message was sent while issue was closed.
Change committed as 261465

Powered by Google App Engine
This is Rietveld 408576698