Commit ccf405ad authored by John Z Wu's avatar John Z Wu Committed by Commit Bot

Check CanCallJavaScriptFunction() in SharedPasswordController

The check was there before, but I accidentally removed it as part of
the refactoring in:
https://chromium-review.googlesource.com/c/chromium/src/+/2254351

Change-Id: I4c57334ead28dd94fa624656b6fc21041bb10a1d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2310845
Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org>
Reviewed-by: default avatarMohamed Amir Yosef <mamir@chromium.org>
Reviewed-by: default avatarMike Dougherty <michaeldo@chromium.org>
Reviewed-by: default avatarMaria Kazinova <kazinova@google.com>
Cr-Commit-Position: refs/heads/master@{#790948}
parent 00c27eb1
...@@ -213,6 +213,8 @@ NSString* const kSuggestionSuffix = @" ••••••••"; ...@@ -213,6 +213,8 @@ NSString* const kSuggestionSuffix = @" ••••••••";
frameDidBecomeAvailable:(web::WebFrame*)web_frame { frameDidBecomeAvailable:(web::WebFrame*)web_frame {
DCHECK_EQ(_webState, webState); DCHECK_EQ(_webState, webState);
DCHECK(web_frame); DCHECK(web_frame);
if (!web_frame->CanCallJavaScriptFunction())
return;
UniqueIDTabHelper* uniqueIDTabHelper = UniqueIDTabHelper* uniqueIDTabHelper =
UniqueIDTabHelper::FromWebState(_webState); UniqueIDTabHelper::FromWebState(_webState);
uint32_t nextAvailableRendererID = uint32_t nextAvailableRendererID =
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment