Revert of DevTools: Decouple OverridesView from InspectorView with "main.show-drawer-view" action (https://chromiumcodereview.appspot.com/372733002/)

Reason for revert:
Landed by CQ in error, as the offline decision was to use a different approach

Original issue's description:
> DevTools: Implement showing drawer views through "main.show-drawer-view" action
> 
> In particular, this patch is required to decouple the OverridesView from InspectorView.
> 
> R=dgozman, vsevik
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=177584

TBR=dgozman@chromium.org,vsevik@chromium.org,pfeldman@chromium.org
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/365333005

git-svn-id: svn://svn.chromium.org/blink/trunk@177587 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Timeline: show painted pictures on layer view in Timeline

- gather paint events as part of Frame data in TimelineFrameModel;
- load pictures for paint events in TimelineLayerTreeView;
- change Layers3DView to operate over PaintProfilerSnapshots, not
    image URLs, so it may later control bitmap sizes depending on zoom;
- (drive by) fix viewport rendering in Layers3DView.

BUG=

Review URL: https://codereview.chromium.org/369913002

git-svn-id: svn://svn.chromium.org/blink/trunk@177588 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: speculative fix of inspector/console/console-viewport-selection.html

This patch speculatively fixes flakiness of
inspector/console/console-viewport-selection.html test. The fix adds
viewport.refresh() methods to explicitly satisfy preconditions for every
method that does reads from viewport control model.

BUG=390204
TBR=vsevik

Review URL: https://codereview.chromium.org/370223004

git-svn-id: svn://svn.chromium.org/blink/trunk@177589 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: NetworkPanel: get rid of node <-> request mapping.

BUG=

Review URL: https://codereview.chromium.org/377513004

git-svn-id: svn://svn.chromium.org/blink/trunk@177590 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Embed CSS/JS under Source/web/resources/ as GRD resources

Embed the CSS and JS used for the color chooser and date/time chooser
as GRD resources instead of as C++ arrays using make-file-array.py.

BUG=312586

Review URL: https://codereview.chromium.org/364323002

git-svn-id: svn://svn.chromium.org/blink/trunk@177591 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Fix imported stylesheets editing.

BUG=385035

Review URL: https://codereview.chromium.org/373753004

git-svn-id: svn://svn.chromium.org/blink/trunk@177592 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Do not rely on the precalculated event level when building tracing timeline flame chart

Review URL: https://codereview.chromium.org/371963002

git-svn-id: svn://svn.chromium.org/blink/trunk@177593 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Unflake layout test loader/data-uri-images-load-when-loading-images-automatically-disabled.html

Re-arranged the test to avoid using img.onload which doesn't seem to be
reliable enough.

Verified that the test still would detect the condition of data: uri images
being blocked by loadsImagesAutomatically.

BUG=387101,391388

Review URL: https://codereview.chromium.org/377513002

git-svn-id: svn://svn.chromium.org/blink/trunk@177594 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Directly update opacity

This CL expands the codepath that directly updates transforms to directly
update opacity. We don't need to do anything clever for opacity because having
opacity doesn't affect the early stages of the compositing pipeline.

I've also added some comments explaining why we bail out from a direct update
in a couple places.

R=esprehn@chromium.org

Review URL: https://codereview.chromium.org/372433002

git-svn-id: svn://svn.chromium.org/blink/trunk@177595 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove Platform::requestProcessMemorySizes

Minor refactoring. I couldn't find its usage anywhere.
It is not implemented nor called.

BUG=None

Review URL: https://codereview.chromium.org/378503002

git-svn-id: svn://svn.chromium.org/blink/trunk@177596 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: fix String.prototype.hashCode() to produce a SMI, not a double

String.prototype.hashCode() used to return Infinity for large strings due
to numbers promoted to double and hence not subject to modulo 2^32
arithmetics that hashCode() apparently expected.

BUG=

Review URL: https://codereview.chromium.org/375443003

git-svn-id: svn://svn.chromium.org/blink/trunk@177597 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove dirtyLinesFromChangedChild from updateMarkerLocation

This patch removes the unnecessary dirtyLinesFromChangedChild call
from RenderListItem::updateMarkerLocation.

No new tests as this was not needed.

R=leviw
BUG=391009

Review URL: https://codereview.chromium.org/375433003

git-svn-id: svn://svn.chromium.org/blink/trunk@177598 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Be consistent when applying outline style to anonymous block continuation

Anonymous block continuation should use its continuation's style for
outline.

However, previously we only use continuation's style during
invalidation, but not when painting outline.

Rename RenderObject::outlineStyleForPaintInvalidation() to
RenderObject::outlineStyle(), and use it when applicable.

BUG=388799
TEST=fast/css/anonymous-block-continuation-outline.html

Review URL: https://codereview.chromium.org/363083003

git-svn-id: svn://svn.chromium.org/blink/trunk@177599 bbb929c8-8fbe-4397-9dbb-9b2b20218538
HTML Imports: Make CustomElementSyncMicrotaskQueue per context.

This change introduces per-CustomElementRegistrationContext queue for
blockable microtask queue (CustomElementSyncMicrotaskQueue).

Before this change, the queue is singleton. So one slow import from
a frame can intefere another document's microtask execution.
For example, a microtask is blocked from execution by an import
in other document (frame).

BUG=384516
R=dglazkov@chromium.org, dominicc@chromium.org
TEST=import-upgrade-iframe.html

Review URL: https://codereview.chromium.org/339593004

git-svn-id: svn://svn.chromium.org/blink/trunk@177600 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Removing unused variables for parent in FocusCandidate

The variables for parent ie parentDistance and parentAlignment in
FocusCandidate are not used anymore. So removing these unused variables.

These changes has been taken from following WebKit patch:
http://trac.webkit.org/changeset/152465

Review URL: https://codereview.chromium.org/371443002

git-svn-id: svn://svn.chromium.org/blink/trunk@177602 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Add builder grid to unexpected failures tab.

R=abarth@chromium.org,ojan@chromium.org,esprehn@chromium.org
NOTRY=true

Review URL: https://codereview.chromium.org/367853002

git-svn-id: svn://svn.chromium.org/blink/trunk@177604 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Empty inline elements always get a line box.

CSS 2.1 (http://www.w3.org/TR/CSS2/visudet.html#leading) says: "Empty inline 
elements generate empty inline boxes, but these boxes still have margins, padding, 
borders and a line height, and thus influence [line-height calculation] just 
like elements with content."

We weren't creating lineboxes for empty inline elements when they occurred
between a single trailing space and a single leading space - this is because
our linebox creation was based on the too-narrow premise that we were already
ignoring spaces. However, if an empty inline element comes after a trailing 
space but before one or more leading spaces then it will be ignored as part 
of the collapsed space between the trailing space and the next non-collapsible 
space - so we should force the creation of a linebox in such cases.

This allows us to pass Hixie's empty inline test: 
http://www.hixie.ch/tests/evil/mixed/emptyinline.html

BUG=328939

Review URL: https://codereview.chromium.org/300853007

git-svn-id: svn://svn.chromium.org/blink/trunk@177605 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Bring back result counter in search across all files.

BUG=368169

Review URL: https://codereview.chromium.org/374573004

git-svn-id: svn://svn.chromium.org/blink/trunk@177606 bbb929c8-8fbe-4397-9dbb-9b2b20218538
When updating a font family setting, only mark style and
layout as dirty if the font family was actually changed.

BUG=390799

Review URL: https://codereview.chromium.org/371853006

git-svn-id: svn://svn.chromium.org/blink/trunk@177607 bbb929c8-8fbe-4397-9dbb-9b2b20218538
In RenderLayerRepainter::repaintRectIncludingNonCompositingDescendants, issue
paint invalidations with the previous paint invalidation rect, rather than
computing a new one.

In all of the call sites, the expectation is to issue a special paint invalidation before making
 a destructive update to the render layer/compositing tree.

It also mirrors the logic in RenderLayerRepainter::repaintRectIncludingNonCompositingDescendants.

BUG=391036

Review URL: https://codereview.chromium.org/371913005

git-svn-id: svn://svn.chromium.org/blink/trunk@177608 bbb929c8-8fbe-4397-9dbb-9b2b20218538