|
|
Update Mac plugin IME handling to match updated spec
This changes IME handling per the recent NPAPI spec clarification, and adds the new bool to indicate conformance to the new model.
Also:
- Fixes an existing bug where the initial key down would be sent to the wrong IME system (since it happens before plugin IME starts) by having the render widget track whether a plugin is focused.
- Changes ComplexTextInputPanel to match recent upstream changes.
BUG= 70427
TEST=None (plugins have not yet implemented Cocoa IME).
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72190
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+248 lines, -121 lines) |
Patch |
 |
M |
chrome/browser/renderer_host/render_widget_host.h
|
View
|
1
2
3
|
1 chunk |
+2 lines, -1 line |
0 comments
|
Download
|
|
M |
chrome/browser/renderer_host/render_widget_host.cc
|
View
|
1
2
3
|
2 chunks |
+10 lines, -4 lines |
0 comments
|
Download
|
|
M |
chrome/browser/renderer_host/render_widget_host_view.h
|
View
|
|
1 chunk |
+5 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/browser/renderer_host/render_widget_host_view_mac.h
|
View
|
|
4 chunks |
+13 lines, -7 lines |
0 comments
|
Download
|
|
M |
chrome/browser/renderer_host/render_widget_host_view_mac.mm
|
View
|
1
2
3
|
10 chunks |
+52 lines, -32 lines |
0 comments
|
Download
|
|
M |
chrome/browser/renderer_host/test/test_render_view_host.h
|
View
|
|
1 chunk |
+2 lines, -1 line |
0 comments
|
Download
|
|
M |
chrome/browser/renderer_host/test/test_render_view_host.cc
|
View
|
|
1 chunk |
+5 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/common/plugin_messages_internal.h
|
View
|
|
2 chunks |
+5 lines, -3 lines |
0 comments
|
Download
|
|
M |
chrome/common/render_messages_internal.h
|
View
|
1
2
3
|
2 chunks |
+8 lines, -5 lines |
0 comments
|
Download
|
|
M |
chrome/plugin/webplugin_delegate_stub.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
chrome/plugin/webplugin_delegate_stub.cc
|
View
|
|
2 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
|
M |
chrome/plugin/webplugin_proxy.h
|
View
|
1
2
3
|
1 chunk |
+3 lines, -1 line |
0 comments
|
Download
|
|
M |
chrome/plugin/webplugin_proxy.cc
|
View
|
1
2
3
|
1 chunk |
+7 lines, -2 lines |
0 comments
|
Download
|
|
M |
chrome/renderer/render_view.h
|
View
|
1
2
3
|
2 chunks |
+6 lines, -3 lines |
0 comments
|
Download
|
|
M |
chrome/renderer/render_view.cc
|
View
|
1
2
3
|
3 chunks |
+13 lines, -8 lines |
0 comments
|
Download
|
|
M |
chrome/renderer/webplugin_delegate_proxy.h
|
View
|
|
2 chunks |
+5 lines, -3 lines |
0 comments
|
Download
|
|
M |
chrome/renderer/webplugin_delegate_proxy.cc
|
View
|
1
2
3
|
3 chunks |
+14 lines, -7 lines |
0 comments
|
Download
|
|
M |
third_party/mozilla/ComplexTextInputPanel.h
|
View
|
|
1 chunk |
+2 lines, -1 line |
0 comments
|
Download
|
|
M |
third_party/mozilla/ComplexTextInputPanel.mm
|
View
|
|
2 chunks |
+10 lines, -6 lines |
0 comments
|
Download
|
|
M |
third_party/mozilla/README.chromium
|
View
|
1
2
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
webkit/plugins/npapi/plugin_host.cc
|
View
|
|
1 chunk |
+10 lines, -0 lines |
0 comments
|
Download
|
|
M |
webkit/plugins/npapi/webplugin.h
|
View
|
|
1 chunk |
+5 lines, -2 lines |
0 comments
|
Download
|
|
M |
webkit/plugins/npapi/webplugin_delegate_impl.h
|
View
|
1
2
3
|
4 chunks |
+9 lines, -4 lines |
0 comments
|
Download
|
|
M |
webkit/plugins/npapi/webplugin_delegate_impl_mac.mm
|
View
|
1
|
8 chunks |
+57 lines, -21 lines |
0 comments
|
Download
|
Total messages: 2 (0 generated)
|
Chromium Code Reviews
Issue
6259013:
Update Mac plugin IME handling to match updated spec (Closed)
