Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1)

Issue 11519012: Ensure that the Inspect Element operation when performed on Windows 8 Metro ASH opens a docked dev … (Closed)

Created:
8 years ago by ananta
Modified:
8 years ago
Reviewers:
sky, sky, pfeldman
CC:
chromium-reviews, vsevik, yurys, pfeldman
Visibility:
Public.

Description

Ensure that the Inspect Element operation when performed on Windows 8 Metro ASH opens a docked dev tools window. Currently if this operation is performed on Windows 8 ASH we end up opening the Dev tools window in desktop mode which is incorrect. This happens because we attempt to find the inspected WebContents instance in the desktop browser list. Fix is to find the inspected webcontents in the desktop and the ASH browser lists as they can both be active concurrently for Windows 8. BUG=151718 R=sky Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=172360

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+36 lines, -2 lines) Patch
M chrome/browser/debugger/devtools_window.h View 1 2 chunks +8 lines, -0 lines 0 comments Download
M chrome/browser/debugger/devtools_window.cc View 1 2 2 chunks +28 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
ananta
8 years ago (2012-12-10 23:54:44 UTC) #1
pfeldman
8 years ago (2012-12-11 06:40:40 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698