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

Issue 10544060: Merge 140030 - Revert 124453 - WebWidgetClient::screenInfo() no longer does a synchronous IPC. (Closed)

Created:
8 years, 6 months ago by varunjain
Modified:
8 years, 6 months ago
Reviewers:
varunjain
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Merge 140030 - Revert 124453 - WebWidgetClient::screenInfo() no longer does a synchronous IPC. Instead it grabs the necessary info from a cached WebScreenInfo factory. BUG=111401, 126586 TEST=Manually Review URL: http://codereview.chromium.org/9302022 TBR=fsamuel@chromium.org,jam@chromium.org Review URL: https://chromiumcodereview.appspot.com/10441101 TBR=varunjain@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=141086

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -40 lines) Patch
M content/browser/renderer_host/render_message_filter.h View 1 chunk +2 lines, -0 lines 0 comments Download
M content/browser/renderer_host/render_message_filter.cc View 1 chunk +1 line, -0 lines 0 comments Download
M content/browser/renderer_host/render_message_filter_win.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M content/browser/renderer_host/render_view_host_impl.cc View 1 chunk +0 lines, -13 lines 0 comments Download
M content/browser/renderer_host/render_widget_host_impl.h View 1 chunk +2 lines, -0 lines 0 comments Download
M content/browser/renderer_host/render_widget_host_impl.cc View 2 chunks +9 lines, -0 lines 0 comments Download
M content/common/view_messages.h View 2 chunks +7 lines, -3 lines 0 comments Download
M content/renderer/render_thread_impl.cc View 1 chunk +0 lines, -1 line 0 comments Download
M content/renderer/render_view_impl.h View 2 chunks +0 lines, -2 lines 0 comments Download
M content/renderer/render_view_impl.cc View 5 chunks +2 lines, -6 lines 0 comments Download
M content/renderer/render_widget.h View 3 chunks +1 line, -6 lines 0 comments Download
M content/renderer/render_widget.cc View 4 chunks +15 lines, -6 lines 0 comments Download
M content/renderer/render_widget_fullscreen.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/test/render_view_test.cc View 2 chunks +0 lines, -2 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
varunjain
8 years, 6 months ago (2012-06-07 21:25:48 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698