New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: bump chromium to 0e4ca9c0a63d7a39bd910997ad4c6 (master) #24687
Conversation
0c29a76
to
8147002
Compare
692c662
to
f5d3182
Compare
@@ -166,28 +166,28 @@ index a87c1992018071ecce8e4a640abebfeaf4e742eb..80ac0ab3b0edb12ca1846b8960143659 | |||
|
|||
DCHECK_EQ(GL_TEXTURE0, GetActiveTextureUnit(gl_)); | |||
gl_->BindTexture(GL_TEXTURE_EXTERNAL_OES, lock.texture_id()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fyi @poiru
f8288d9
to
60224be
Compare
db23f61
to
692c74a
Compare
26df6e0
to
372e966
Compare
9843a94
to
37d22f8
Compare
7cec63a
to
b10689c
Compare
const content::ConsoleMessage& message) { | ||
v8::Isolate* isolate = JavascriptEnvironment::GetIsolate(); | ||
v8::HandleScope scope(isolate); | ||
v8::HandleScope handle_scope(isolate); | ||
Emit("console-message", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add scope
to the event object?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like we do now https://github.com/electron/electron/blob/master/shell/common/gin_helper/event_emitter.h#L63 , can remove this in a follow-up PR.
widget_host->Send(new WidgetMsg_EnableDeviceEmulation( | ||
widget_host->GetRoutingID(), params)); | ||
auto* widget_host_impl = | ||
frame_host ? static_cast<content::RenderWidgetHostImpl*>( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oof, i don't love casting to the Impl
. Can we do this without reaching into the internals?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately, the emulation apis are only reachable via the Impl
class https://source.chromium.org/chromium/chromium/src/+/master:content/browser/devtools/protocol/emulation_handler.cc;l=553-563
this should be safe to cast https://source.chromium.org/chromium/chromium/src/+/master:content/browser/renderer_host/render_widget_host_impl.cc;l=510-512
else if (screen_position == "desktop") | ||
out->screen_position = blink::WebDeviceEmulationParams::kDesktop; | ||
std::string screen_type; | ||
if (dict.Get("screenPosition", &screen_type)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"screen position" is a weird name for this, but probably not worth it to change it.
Failing woa test is unrelated, merging. |
No Release Notes |
Updating Chromium to 0e4ca9c0a63d7a39bd910997ad4c6 (master).
See all changes in 1a093e6a0cb5e72ba78990fe39824..0e4ca9c0a63d7a39bd910997ad4c6
Notes: no-notes