New issue
Advanced search Search tips

Issue 701501 link

Starred by 3 users

Issue metadata

Status: ExternalDependency
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 674593


Show other hotlists

Hotlists containing this issue:
Non-Standard-IDL


Sign in to add a comment

Standardize or remove baseNode/baseOffset/extentNode/extentOffset

Project Member Reported by lunalu@chromium.org, Mar 14 2017

Issue description

Comment 1 by tkent@chromium.org, Mar 17 2017

Labels: Hotlist-Interop
Status: ExternalDependency (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 19 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1c0152d0297a8341498a04087587b26ee59fbdda

commit 1c0152d0297a8341498a04087587b26ee59fbdda
Author: Yoshifumi Inoue <yosin@chromium.org>
Date: Mon Mar 19 06:51:38 2018

Make Selection#{base,extent}{Node,Offset} as aliases of {anchor,focus}{Node,Offset}

This patch makes |Selection#{base,extent}{Node,Offset}| as aliases of
|Selection#{anchor,focus}{Node,Offset}| instead of getting position from
|VisibleSelection| which we don't want to expose to JavaScript.

Note: Selection#{base,extent}{Node,Offset} are not standard API and there
are some discussion for standardizing them[1].

[1] https://github.com/w3c/selection-api/issues/34
Specify baseNode/baseOffset/extentNode/extentOffset?

Bug: 701501
Change-Id: I34fcdb4e80d5cef49bc1e0bdaea3cdec5c8b3758
Reviewed-on: https://chromium-review.googlesource.com/965663
Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543982}
[modify] https://crrev.com/1c0152d0297a8341498a04087587b26ee59fbdda/third_party/WebKit/Source/core/editing/DOMSelection.cpp

Sign in to add a comment