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

Issue 177923011: Chromium side of moving clearContextMenu from WebViewClient to WebFrameClient. (Closed)

Created:
6 years, 9 months ago by jam
Modified:
6 years, 9 months ago
Reviewers:
jamesr
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, site-isolation-reviews_chromium.org
Visibility:
Public.

Description

Chromium side of moving clearContextMenu from WebViewClient to WebFrameClient. BUG=304341 R=jamesr@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255479

Patch Set 1 #

Total comments: 1

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -0 lines) Patch
M content/renderer/render_frame_impl.h View 1 chunk +1 line, -0 lines 0 comments Download
M content/renderer/render_frame_impl.cc View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
jam
6 years, 9 months ago (2014-03-06 21:36:41 UTC) #1
jamesr
lgtm but you should be perfectly fine landing this without the #if guards. There's a ...
6 years, 9 months ago (2014-03-06 21:59:26 UTC) #2
jam
On 2014/03/06 21:59:26, jamesr wrote: > lgtm but you should be perfectly fine landing this ...
6 years, 9 months ago (2014-03-06 22:01:07 UTC) #3
jamesr
6 years, 9 months ago (2014-03-06 22:02:59 UTC) #4
cool :)

Powered by Google App Engine
This is Rietveld 408576698