[webkit-changes] [WebKit/WebKit] c412d3: Remove rendererWillBeDestroyed() from HTMLTextArea...

Ahmad Saleem noreply at github.com
Tue Jul 4 08:10:36 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c412d338b0081d855291ea6a50724861f97daa1e
      https://github.com/WebKit/WebKit/commit/c412d338b0081d855291ea6a50724861f97daa1e
  Author: Ahmad Saleem <ahmad.saleem792+github at gmail.com>
  Date:   2023-07-04 (Tue, 04 Jul 2023)

  Changed paths:
    M Source/WebCore/html/HTMLTextAreaElement.h
    M Source/WebCore/rendering/RenderTextControlMultiLine.cpp
    M Source/WebCore/rendering/RenderTextControlMultiLine.h

  Log Message:
  -----------
  Remove rendererWillBeDestroyed() from HTMLTextAreaElement.h and RenderTextControlMultiLine.cpp

https://bugs.webkit.org/show_bug.cgi?id=258785

Reviewed by Tim Nguyen.

Merge: https://chromium.googlesource.com/chromium/blink/+/10c9bbc0ea2c10efe497897cbd12bb842f7b0ee2

This code was added in 2006 in https://trac.webkit.org/changeset/13051
and the test that was added with it fast/forms/textarea-setvalue-submit.html
still passes without the code now. It doesn't make any sense that form
controls would need to do this update in the destructor of the renderer,
especially not only `textarea` since none of the other controls do this.

* Source/WebCore/html/HTMLTextAreaElement.h: Remove 'rendererWillBeDestroyed'
* Source/WebCore/rendering/RenderTextControlMultiLine.cpp:
(RenderTextControlMultiLine::willBeDestroyed): Deleted
* Source/WebCore/rendering/RenderTextControlMultiLine.h: Delete 'willBeDestroyed' definition

Canonical link: https://commits.webkit.org/265755@main




More information about the webkit-changes mailing list