Remove platform/Widget.h, it is now fully renamed to platform/FrameViewBase.h

Remove platform/Widget.h, it is now fully renamed to platform/FrameViewBase.h


BUG=697351
Review-Url: https://codereview.chromium.org/2724003002
Cr-Commit-Position: refs/heads/master@{#454239}
Committed: https://chromium.googlesource.com/chromium/src/+/0a7e111977232fbb33ad8db8bad857bbb3c68580

[joelhockey, 2017-03-01 10:47:48]

joelhockey@chromium.org changed reviewers:
+ haraken@chromium.org, sashab@chromium.org, slangley@chromium.org

[joelhockey, 2017-03-01 10:52:15]

Haraken, Now that the rename of the class is done, I am planning to also rename
any methods, and fields that refer to Widget and make them FrameViewBase.

Things such as:
platform/FrameViewBase.convertToContainingWidget
core/layout/LayoutPart.widget
core/frame/FrameView.setNeedsUpdateWidgetGeometries
... and many others

Is this worthwhile?

[haraken, 2017-03-01 10:55:30]

LGTM

On 2017/03/01 10:52:15, joelhockey wrote:
> Haraken, Now that the rename of the class is done, I am planning to also
rename
> any methods, and fields that refer to Widget and make them FrameViewBase.
> 
> Things such as:
> platform/FrameViewBase.convertToContainingWidget
> core/layout/LayoutPart.widget
> core/frame/FrameView.setNeedsUpdateWidgetGeometries
> ... and many others
> 
> Is this worthwhile?

Yeah, I think we should drop "Widget" from the code base.

[joelhockey, 2017-03-02 03:09:24]

The CQ bit was checked by joelhockey@chromium.org

[commit-bot: I haz the power, 2017-03-02 03:10:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[joelhockey, 2017-03-02 03:26:37]

Description was changed from

==========
Remove platform/Widget.h, it is now fully renamed to platform/FrameViewBase.h


Rename platform/Widget to platform/FrameViewBase in platform.

BUG=697351
==========

to

==========
Remove platform/Widget.h, it is now fully renamed to platform/FrameViewBase.h


BUG=697351
==========

[commit-bot: I haz the power, 2017-03-02 07:05:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-02 07:05:34]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[joelhockey, 2017-03-02 12:03:10]

The CQ bit was checked by joelhockey@chromium.org

[commit-bot: I haz the power, 2017-03-02 12:03:21]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-02 12:46:54]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488456190316970,
"parent_rev": "cb1950ec6514d9d4d9d5505a99300c363b30c469", "commit_rev":
"0a7e111977232fbb33ad8db8bad857bbb3c68580"}

[commit-bot: I haz the power, 2017-03-02 12:47:29]

Description was changed from

==========
Remove platform/Widget.h, it is now fully renamed to platform/FrameViewBase.h


BUG=697351
==========

to

==========
Remove platform/Widget.h, it is now fully renamed to platform/FrameViewBase.h


BUG=697351

Review-Url: https://codereview.chromium.org/2724003002
Cr-Commit-Position: refs/heads/master@{#454239}
Committed:
https://chromium.googlesource.com/chromium/src/+/0a7e111977232fbb33ad8db8bad8...
==========

[commit-bot: I haz the power, 2017-03-02 12:47:30]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/0a7e111977232fbb33ad8db8bad8...